New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Travis matrix build & bump up Composer dependencies #61
Conversation
462b5b2
to
8442fd4
Compare
8442fd4
to
c8e5665
Compare
$wrongParameterValue = 'some other value'; | ||
|
||
return array( | ||
// the container does not have the parameter | ||
array($emptyContainer, $parameterName, $parameterValue, true, false), | ||
// the container has the parameter but the values don't match | ||
array($containerWithParameter, $parameterName, $wrongParameterValue, true, false), | ||
array($this->createMockContainerWithParameters(array($parameterName => $parameterValue)), $parameterName, $wrongParameterValue, true, false), |
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.
Mocked method was available only for the first call, I haven't used PhpUnit mocks yet so this is the best I came up with :)
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.
Looks reasonable to me as it doesn't make much sense to reuse the mocked object.
Ping @matthiasnoback 🎉 Anyway, I'm using these test libraries heavily for a while and if you're looking for someone to maintain it, just drop me a line :) |
It's quite a blocker, since I can't install new release of ConfigTest along with DependencyInjectionTest. |
Thanks, sorry for letting you wait. |
"symfony/config": "^2.0.5|~3.0", | ||
"sebastian/exporter": "~1" | ||
"matthiasnoback/symfony-config-test": "^1.0|^2.0", | ||
"phpunit/phpunit": "^4.0|^5.0", |
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.
This has nothing to do here and was especially removed on #47.
@matthiasnoback Could you please revert this back?
It was sneakily moved back to require in SymfonyTest#61, without any explanation. Also see SymfonyTest#47, SymfonyTest#28.
Similar to SymfonyTest/SymfonyConfigTest#32 - drops unsupported Symfony versions and PhpUnit
3.7
. Moreover, drops support for SymfonyConfigTest0.*
and allows for2.0
to be installed.