badoo / badoo/soft-mocks

Bug with SplFileInfo

Open
#7 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
PHP
Stars
308
Forks
38
PR merge metrics
No merged PRs in 30d

Description

Hi!

I use php 5.6+ and take next bug.

https://github.com/badoo/soft-mocks/blob/bc5af0de85d5d2ad5fe2fe233dc2ecbc3406b973/src/QA/SoftMocks.php#L683

SplFileInfo can not use as array!

Fix should be something like that:
......
if (strpos($file, '/') != 0) {
.....

Contributor guide

No contributing guide indexed for this repository

Research direction

Inspect src/QA/SoftMocks.php around line 683 and reproduce the problem with PHP 5.6+ using a SplFileInfo value. Follow the issue's suggested path check, then verify that SplFileInfo is handled without being treated as an array and that the existing mocking behavior remains intact.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
testing
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.