Do not require phpunit on production

[soullivaneuh]

PHPUnit is for your tests.

Lot of projects may not need it because of not using phpunit or use global version.

[stof]

no, phpunit is not for our tests. The code of the WebApiContext currently depends on PHPUnit assertions, so this is a hard requirement of the library (unfortunately the assertions are not yet standalone). PHPUnit code needs to be autoloadable from behat runs.

See #38 which is about removing this dependency.

[soullivaneuh]

The code of the WebApiContext currently depends on PHPUnit assertions

Yes, but what about global using of phpunit? I'm using .phar and I don't want to have a second phpunit to override it and have useless dependencies.

PHPUnit code needs to be autoloadable from behat runs.

What do you mean?

See #38 which is about removing this dependency.

Subscribed, thanks.

[stof]

Yes, but what about global using of phpunit?

the code inside your global PHPUnit phar won't be accessible by your behat suite to use the PHPUnit codebase

[soullivaneuh]

depends on PHPUnit assertions

AFAIK PHPUnit 6 use webmozart/assert.

Can't this be separated with PHPUnit 6 compatibility?

This is overkill for app to require the hole phpunit dependency.

[stof]

Well, PHPUnit assertions are still there in PHPUnit, and are not using webmozart/assert

[soullivaneuh]

@stof Indeed I was wrong.

Still, do you have any reason to use only PHPUnit assertion and not another library?

If not, you may migrate to a lighter one like webmozart/assert to get rid of the full phpunit requirement.

WDYT?