8000 added Tester\FileMock by dg · Pull Request #137 · nette/tester · GitHub
[go: up one dir, main page]
Skip to content

added Tester\FileMock#137

Merged
dg merged 1 commit intonette:masterfrom
dg:pull-filemock
May 29, 2014
Merged

added Tester\FileMock#137
dg merged 1 commit intonette:masterfrom
dg:pull-filemock

Conversation

@dg
Copy link
Member
@dg dg commented May 29, 2014

Allowes to test functions like parse_ini_file():

Assert::same(
    array('key' => 'value'), 
    parse_ini_file(FileMock::create('key=value'))
);

@mishak87
Copy link
mishak87 commented May 29, 2014

This is neat!

@stepansvoboda
Copy link
stepansvoboda commented May 29, 2014

Super :)

@fprochazka
Copy link
Contributor
fprochazka commented May 29, 2014

👏

@Vrtak-CZ
Copy link
Contributor
Vrtak-CZ commented May 29, 2014

🆒

dg added a commit that referenced this pull request May 29, 2014
@dg
Merge pull request #137 from dg/pull-filemock
e4a4938 
added Tester\FileMock
@dg dg merged commit e4a4938 into nette:master May 29, 2014
@dg dg deleted the pull-filemock branch May 29, 2014 14:44
@janmarek
Copy link
Contributor
janmarek commented Jun 6, 2014

Why not just use http://vfs.bovigo.org/ ? Is it somehow better?

@dg
Copy link
Member Author
dg commented Jun 6, 2014

I did not know vfsStream. It seems powerful, FileMock mocks just single files.

@janmarek
Copy link
Contributor
janmarek commented Jun 6, 2014

This problem can't be solved better than what they did in vfsStream. It mostly works, but some functions like realpath can't be mocked.

@dg
Copy link
Member Author
dg commented Jun 6, 2014

Ok.
A769

@dg
Copy link
Member Author
dg commented Jun 6, 2014

Btw, in PHP exists stream data: which can emulate files too:

$file = 'data:text/plain,'.urlencode('Nette Tester');
echo file_get_contents($file); // Nette Tester

but it doesn't work for example with parse_ini_file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@dg @mishak87 @stepansvoboda @fprochazka @Vrtak-CZ @janmarek
0