-
Notifications
You must be signed in to change notification settings - Fork 73
Add config to use custom phpunit.humbug.xml (e.g for symfony project) #197
base: master
Are you sure you want to change the base?
Conversation
tabbabi
commented
Dec 27, 2016
•
edited by theofidry
edited by theofidry
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While I think this is a nice feature, there is besides the comment two things to consider:
- Ensure that this config file is overridable with an argument when invoking humbug
- I'm not sure about the entry key if we add support for say phpSpec for example (as you could very well use both PHPUnit and phpSpec)
tests/Adapter/PhpunitTest.php
Outdated
{ | ||
$container = m::mock('\Humbug\Container'); | ||
$container->shouldReceive([ | ||
'getSourceList' => $this->root, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you remove the unnecessary whitespace between the arrows please?
tests/Adapter/PhpunitTest.php
Outdated
]); | ||
; | ||
$adapter = new Phpunit; | ||
$process = $adapter->getProcess( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this can be a one liner
Also needs to be documented. It's relatively simple, so I might look at it later over the weekend. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update to README required.
126c15d
to
67be9e2
Compare
67be9e2
to
0a6af39
Compare
@tabbabi could you add the missing doc? After that I think it will be ready to be merged :) |