Surefire doesn't work if testng is in the classpath and a suitexml isn't being used

Bascially, not tests will be run. I was able to fix this in the latest surefire code, by changing this code in TestNGDirectoryTestSuite.java:

try

{ Class junitClass = Class.forName( "junit.framework.Test" ); xmlTest.setJUnit( testSet.getTestClass().isAssignableFrom( junitClass ) ); }

catch ( ClassNotFoundException e )
{
}

to this code:

if ( !TestNGClassFinder.isTestNGClass( testSet.getTestClass(), annotationFinder ) )

{ xmlTest.setJUnit( true ); }