Skip to content
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

Functionality and unit-test for execute test methods in the specified order are added #941

Closed
wants to merge 4 commits into from

Conversation

otereshin
Copy link

Functionality and unit-test for execute test methods in the specified order are added. Please look at this.

@kcooney
Copy link
Member

kcooney commented Jun 28, 2014

Thanks for the contribution, @otereshin

I'm personally leaning against accepting this pull. I personally think we should instead get #882 to the point where it could be merged. That pull would allow you to write your own sorting mechanism, so you could do what you have here without further changes to the core JUnit code.

@junit-team/junit-committers what do you guys think?

@marcphilipp
Copy link
Member

+1

@otereshin
Copy link
Author

Hello Kevin,
Will it useful if I'll add this functionality for #882 ?

@kcooney
Copy link
Member

kcooney commented Jun 29, 2014

@otereshin if you think you could make your own coopy of #882 and make it ready for merging, please do!

@kcooney kcooney closed this Sep 6, 2014
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.

3 participants