Skip to content

Conversation

@remicollet
Copy link
Contributor

Without this, using PHPUnit 5.4.4

There was 1 failure:

1) EasyMock\Test\EasyMockTest::should_allow_to_spy_method_calls
Failed asserting that 'Expectation failed for method name is equal to <string:foo> when invoked at least once.
Expected invocation at least once but it never occurred.
' contains "Expected invocation at least once but it never occured".

/tmp/phpunit-easymock/tests/EasyMockTest.php:141

@remicollet
Copy link
Contributor Author

@mnapoli
Copy link
Owner

mnapoli commented Jun 13, 2016

Thanks! Tests are run against the latest versions of the dependencies as well as the lowest (allowed by Composer), which explains why the build fails.

Could you simply remove the word ("occurred") from the assertion? That way it should be good to go for all versions.

Thanks again

@remicollet
Copy link
Contributor Author

I have removed the trailing "red", (so both occured and occurred should work)

@mnapoli mnapoli merged commit 023279d into mnapoli:master Jun 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants