Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

[SUREFIRE-876] avoid using Description.getTestClass() #4

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

Description.getTestClass() uses Class.forName() which
breaks classloader isolation [1,2].
Refactor JUnitCoreRunListener where we only
need the class name anyway to avoid calling
Description.getTestClass().

[1] https://github.com/KentBeck/junit/issues/364
[2] https://bugs.eclipse.org/bugs/show_bug.cgi?id=318299#c8

[SUREFIRE-876] avoid using Description.getTestClass()
Description.getTestClass() uses Class.forName() which
breaks classloader isolation [1,2].
Refactor JUnitCoreRunListener where we only
need the class name anyway to avoid calling
Description.getTestClass().

[1] https://github.com/KentBeck/junit/issues/364
[2] https://bugs.eclipse.org/bugs/show_bug.cgi?id=318299#c8
Member

olamy commented Jul 11, 2012

committed.

thanks!

@jsievers jsievers closed this Jul 11, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment