search

ArcBees / GWTP

A complete model-view-presenter framework to simplify your next GWT project.
Other
335 stars 132 forks source link

Make sure tests always run in the correct order #726

Closed rcpaul closed 9 years ago

rcpaul commented 9 years ago

This pull request addresses issue #725. With the added surefire plugin configuration, we make sure tests are always run in the same order which in this case (alphabetical) also happens to be the 'correct' order.

christiangoudreau commented 9 years ago

I'm just wondering, is there a better way to ensure that the tests are run in the correct order? What if we rename them?

rcpaul commented 9 years ago

Surefire defaults to executing tests in the order they are reported by the file system and there is no global rule that defines this order. There might be another way of telling Surefire to run PlaceManagerImpl2Test before PlaceManagerImplTest that I'm unaware of however. The best way of course would be to make all tests independent but I think that requires some more work.

olafleur commented 9 years ago

I agree with @christiangoudreau that it's not future-proof. We should investigate and make the tests independent.

rcpaul commented 9 years ago

Done. See #727. Closing this pull request since it has become irrelevant.