don't declare two classes in one file

[tflori]

No description provided.

[coveralls]

Coverage remained the same at 77.913% when pulling 8340d61 on tflori:fix-phpunitIntegration into 7e9a844 on padraic:master.

[davedevelopment]

I didn't know about the class in 5.4 that you mentioned, so if we can agree to drop support for PHPUnit < 5.4, we can skip the separate class definitions.

I definitely do not want to remove the trait completely, that's a backwards step for me.

[tflori]

https://github.com/sebastianbergmann/phpunit/blob/5.4/src/ForwardCompatibility/TestCase.php

For me: yes but I'm not maintainer of mockery. mockery requires php >= 5.6 and in this versions you can use the still supported phpunit 5.7. I don't know a reason for using php 4.8 other than dealing with php 5.3.

btw.: phpunit stops supporting php5.6 in february

[coveralls]

Coverage remained the same at 77.913% when pulling 35de8e8 on tflori:fix-phpunitIntegration into 7e9a844 on padraic:master.

[davedevelopment]

Thanks :)