[ https://issues.apache.org/jira/browse/SUREFIRE-2298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17948545#comment-17948545 ]
Ulli Hafner commented on SUREFIRE-2298: --------------------------------------- I am wondering why your expectations are that the outer test results should be reported in the nested class XML and not in the corresponding class XML? From my perspective this expectation is wrong for 3.5.2 and 3.5.3, or am I missing something? Or is my use case a completely different thing because I do not inheritance? I have a test like: {code} class SourcePrinterTest { @Test void testOuter() { } @Nested class ColumnMarkerTest { @Test void testInner() { }. } {code} This results in 0 tests in {{SourcePrinterTest.xml }} but 2 tests in {{SourcePrinterTest$ColumnMarkerTest.xml}}. I would expect to have 1 test in {{SourcePrinterTest.xml }} and 1 test in {{SourcePrinterTest$ColumnMarkerTest.xml}}. > @Nested test classes are not reported properly > ---------------------------------------------- > > Key: SUREFIRE-2298 > URL: https://issues.apache.org/jira/browse/SUREFIRE-2298 > Project: Maven Surefire (Moved to GitHub) > 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)