Skip to content

Commit

Permalink
Remove broken test
Browse files Browse the repository at this point in the history
  • Loading branch information
michael-o committed Jan 4, 2024
1 parent 8ef1395 commit 335f829
Showing 1 changed file with 0 additions and 17 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,6 @@
import org.apache.maven.surefire.its.fixture.SurefireJUnit4IntegrationTestCase;
import org.junit.Test;

import static org.apache.maven.surefire.its.fixture.HelperAssertions.assumeJavaMaxVersion;

/**
* Test TestNG setup and teardown ordering with parallelism
*
Expand Down Expand Up @@ -58,19 +56,4 @@ public void testNgParallelOrderingWithEarlyVersion6() {
.executeTest()
.verifyErrorFree(12);
}

// TestNG 5.13+ already has XmlClass.m_index field, but doesn't have XmlClass.setIndex method.
// Note that the problem of wrong setup methods ordering (SUREFIRE-1967) was not observed on that version.
// This is likely because SUREFIRE-1967 is related to a change in TestNG 6.3, where preserve-order became true by
// default (https://github.com/cbeust/testng/commit/8849b3406ef2184ceb6002768a2d087d7a8de8d5).
@Test
public void testNgParallelOrderingWithVersion5() {
// TestNG 5.13 does not work with Java 17
assumeJavaMaxVersion(16);

unpack("surefire-1967-testng-method-parallel-ordering")
.sysProp("testNgVersion", "5.13")
.executeTest()
.verifyErrorFree(12);
}
}

0 comments on commit 335f829

Please sign in to comment.