[ https://issues.apache.org/jira/browse/SUREFIRE-2298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17940920#comment-17940920 ]
Olivier Lamy commented on SUREFIRE-2298: ---------------------------------------- I have started a fix with this PR [https://github.com/apache/maven-surefire/pull/828] > @Nested test classes are not reported properly > ---------------------------------------------- > > Key: SUREFIRE-2298 > URL: https://issues.apache.org/jira/browse/SUREFIRE-2298 > Project: Maven Surefire > Issue Type: Bug > Affects Versions: 3.5.3 > Reporter: Leonard Brünings > Assignee: Olivier Lamy > Priority: Major > > Given this example test you can see that the {{@Nested}} test classes are not > reported correctly. In 3.5.2 and earlier all executions land in the > {{TEST-BaseNestedTest$Inner.xml}} and with the changes in 3.5.3 they are > distributed in all files. However, the nested classes belong to the enclosing > class and the reporting should reflect that, so it would be more accurate to > report the nested executions as part of the enclosing tests (see below). > [~marcphilipp] please chime in. > > {code:java} > import org.junit.jupiter.api.*; > abstract class BaseNestedTest { > @Test > void outerTest() { > } > > @Nested > class Inner { > @Test > void innerTest() { > } > } > } > class FirstNestedTest extends BaseNestedTest { > } > class SecondNestedTest extends BaseNestedTest { > }{code} > This was most likely caused by > [https://github.com/apache/maven-surefire/commit/5aeca19716dedc4cbcab8d68dfc74a99f50ce9ee] > 3.5.2 > {code:java|title=TEST-FirstNestedTest.xml} > <?xml version="1.0" encoding="UTF-8"?> > <testsuite xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; > xsi:noNamespaceSchemaLocation="https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd"; > version="3.0.2" name="FirstNestedTest" time="0.019" tests="0" errors="0" > skipped="0" failures="0"> > </testsuite> > {code} > {code:java|title=TEST-SecondNestedTest.xml} > <?xml version="1.0" encoding="UTF-8"?> > <testsuite xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; > xsi:noNamespaceSchemaLocation="https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd"; > version="3.0.2" name="SecondNestedTest" time="0.004" tests="0" errors="0" > skipped="0" failures="0"> > </testsuite> > {code} > {code:java|title=TEST-BaseNestedTest$Inner.xml} > <?xml version="1.0" encoding="UTF-8"?> > <testsuite xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; > xsi:noNamespaceSchemaLocation="https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd"; > version="3.0.2" name="BaseNestedTest$Inner" time="0.001" tests="2" > errors="0" skipped="0" failures="0"> > <testcase name="outerTest" classname="FirstNestedTest" time="0.007"/> > <testcase name="innerTest" classname="BaseNestedTest$Inner" time="0.0"/> > <testcase name="innerTest" classname="BaseNestedTest$Inner" time="0.0"/> > <testcase name="outerTest" classname="SecondNestedTest" time="0.001"/> > </testsuite> > {code} > ------------------- > 3.5.3 > {code:java|title=TEST-FirstNestedTest.xml} > <?xml version="1.0" encoding="UTF-8"?> > <testsuite xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; > xsi:noNamespaceSchemaLocation="https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd"; > version="3.0.2" name="FirstNestedTest" time="0.019" tests="1" errors="0" > skipped="0" failures="0"> > <testcase name="outerTest" classname="FirstNestedTest" time="0.007"/> > </testsuite> > {code} > {code:java|title=TEST-SecondNestedTest.xml} > <?xml version="1.0" encoding="UTF-8"?> > <testsuite xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; > xsi:noNamespaceSchemaLocation="https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd"; > version="3.0.2" name="SecondNestedTest" time="0.003" tests="1" errors="0" > skipped="0" failures="0"> > </properties> > <testcase name="outerTest" classname="SecondNestedTest" time="0.001"/> > </testsuite> > {code} > {code:java|title=TEST-BaseNestedTest$Inner.xml} > <?xml version="1.0" encoding="UTF-8"?> > <testsuite xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; > xsi:noNamespaceSchemaLocation="https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd"; > version="3.0.2" name="BaseNestedTest$Inner" time="0.001" tests="1" > errors="0" skipped="0" failures="0"> > <testcase name="innerTest" classname="BaseNestedTest$Inner" time="0.001"/> > <testcase name="innerTest" classname="BaseNestedTest$Inner" time="0.0"/> > </testsuite> > {code} > > ------------------- > Expected reporting > > {code:java|title=TEST-FirstNestedTest.xml} > <?xml version="1.0" encoding="UTF-8"?> > <testsuite xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; > xsi:noNamespaceSchemaLocation="https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd"; > version="3.0.2" name="FirstNestedTest" time="0.019" tests="1" errors="0" > skipped="0" failures="0"> > <testcase name="outerTest" classname="FirstNestedTest" time="0.007"/> > <testcase name="innerTest" classname="BaseNestedTest$Inner" time="0.001"/> > </testsuite> > {code} > {code:java|title=TEST-SecondNestedTest.xml} > <?xml version="1.0" encoding="UTF-8"?> > <testsuite xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; > xsi:noNamespaceSchemaLocation="https://maven.apache.org/surefire/maven-surefire-plugin/xsd/surefire-test-report.xsd"; > version="3.0.2" name="SecondNestedTest" time="0.003" tests="1" errors="0" > skipped="0" failures="0"> > </properties> > <testcase name="outerTest" classname="SecondNestedTest" time="0.001"/> > <testcase name="innerTest" classname="BaseNestedTest$Inner" time="0.0"/> > </testsuite> > {code} > -- This message was sent by Atlassian Jira (v8.20.10#820010)