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

[SUREFIRE-839] Tests not matching categories would fail build #12

Closed
wants to merge 1 commit into from

Conversation

agudian
Copy link
Member

@agudian agudian commented Dec 8, 2012

Fixed in a way that does not break forkMode=onceperthread (enhanced the IT for that), refined the logic of JUnitCoreProvider.canRunClass (to also detect the BasicTest class from the IT)

Fixed in a way that does not break forkMode=onceperthread (enhanced the IT for that), refined the logic of JUnitCoreProvider.canRunClass (to also detect the BasicTest class from the IT)
@Tibor17
Copy link
Contributor

Tibor17 commented Dec 8, 2012

This is behaviour of JUnit Categories.

Instead of patching surefire, thinking the JUnit would report an issue and get an answer.

If this behavior is intended in JUnit, then this is right solution.

@krosenvold
Copy link
Contributor

@Tibor17 I'm unsure about this; it seems like the category parsing inside junit is slightly broken:

If you send a single class to junit core and that has no matches on the categories it fails miserably. Does that seem right to you ?

@agudian
Copy link
Member Author

agudian commented Dec 8, 2012

Oops, seems like I have broken six other ITs with this commit... I close it now and check what went wrong there...

@agudian agudian closed this Dec 8, 2012
@Tibor17
Copy link
Contributor

Tibor17 commented Dec 8, 2012

@krosenvold
No it's not right according to me.

I think, the right appoach would be to ask the author in Junit, if this corner behavior was intended.
The reason is that too much code is place for mistakes and bugs. It looks like 2.12.x versions were such case.

Any way, this behaiour is not mentioned in JUnit documentation.

There are other such corner behaviors, the only thing is to get voters.

@Tibor17
Copy link
Contributor

Tibor17 commented Dec 9, 2012

@krosenvold
Sorry for my question, but there is no forum page.

I would like to make a progress about Parallel Computer, but I have problems with pushing a new pull.

Every time I try to push a new branch the github gives it up, even the checkout fails.
Should I push to other repository, and not here? If yes, how to get the rights there?

Thx a lot.

@krosenvold
Copy link
Contributor

Normally, you would clone the apache github repo to your own account and push to there; subsequently send a pull request.

You can send me an email at krosenvold at apache.org if there's something to help. Out mailing list is dev@maven.apache.org.

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.

None yet

3 participants