fix test with line numbers...

Signed-off-by: Olivier Lamy <olamy@apache.org>
apache · Feb 26, 2023 · 39eb36b · 39eb36b
1 parent aa84e94
commit 39eb36b
Showing 1 changed file with 14 additions and 14 deletions.
diff --git a/...ommon-java5/src/test/java/org/apache/maven/surefire/report/SmartStackTraceParserTest.java b/...ommon-java5/src/test/java/org/apache/maven/surefire/report/SmartStackTraceParserTest.java
@@ -45,7 +45,7 @@ public void testGetString() {
             SmartStackTraceParser smartStackTraceParser =
                     new SmartStackTraceParser(ATestClass.class.getName(), e, null);
             String res = smartStackTraceParser.getString();
-            assertEquals("ATestClass.failInAssert:33 X is not Z", res);
+            assertEquals("ATestClass.failInAssert:30 X is not Z", res);
         }
     }
 
@@ -57,7 +57,7 @@ public void testGetStringFromNested() {
             SmartStackTraceParser smartStackTraceParser =
                     new SmartStackTraceParser(ATestClass.class.getName(), e, null);
             String res = smartStackTraceParser.getString();
-            assertEquals("ATestClass.failInAssert:33 X is not Z", res);
+            assertEquals("ATestClass.failInAssert:30 X is not Z", res);
         }
     }
 
@@ -81,7 +81,7 @@ public void testNestedFailure() {
             SmartStackTraceParser smartStackTraceParser =
                     new SmartStackTraceParser(ATestClass.class.getName(), e, null);
             String res = smartStackTraceParser.getString();
-            assertEquals("ATestClass.nestedFailInAssert:38->failInAssert:33 X is not Z", res);
+            assertEquals("ATestClass.nestedFailInAssert:34->failInAssert:30 X is not Z", res);
         }
     }
 
@@ -93,7 +93,7 @@ public void testNestedNpe() {
             SmartStackTraceParser smartStackTraceParser =
                     new SmartStackTraceParser(ATestClass.class.getName(), e, null);
             String res = smartStackTraceParser.getString();
-            assertEquals("ATestClass.nestedNpe:48->npe:43 NullPointer It was null", res);
+            assertEquals("ATestClass.nestedNpe:42->npe:38 NullPointer It was null", res);
         }
     }
 
@@ -105,7 +105,7 @@ public void testNestedNpeOutsideTest() {
             SmartStackTraceParser smartStackTraceParser =
                     new SmartStackTraceParser(ATestClass.class.getName(), e, null);
             String res = smartStackTraceParser.getString();
-            assertEquals("ATestClass.nestedNpeOutsideTest:58->npeOutsideTest:53 » NullPointer", res);
+            assertEquals("ATestClass.nestedNpeOutsideTest:50->npeOutsideTest:46 » NullPointer", res);
         }
     }
 
@@ -117,7 +117,7 @@ public void testLongMessageHandling() {
             SmartStackTraceParser smartStackTraceParser =
                     new SmartStackTraceParser(ATestClass.class.getName(), e, null);
             String res = smartStackTraceParser.getString();
-            assertEquals("ATestClass.aLongTestErrorMessage:63 Runtime " + e.getMessage(), res);
+            assertEquals("ATestClass.aLongTestErrorMessage:54 Runtime " + e.getMessage(), res);
         }
     }
 
@@ -128,7 +128,7 @@ public void testFailureInBaseClass() {
         } catch (NullPointerException e) {
             SmartStackTraceParser smartStackTraceParser = new SmartStackTraceParser(ASubClass.class.getName(), e, null);
             String res = smartStackTraceParser.getString();
-            assertEquals("ASubClass>ABaseClass.npe:29 » NullPointer It was null", res);
+            assertEquals("ASubClass>ABaseClass.npe:45 » NullPointer It was null", res);
         }
     }
 
@@ -140,7 +140,7 @@ public void testClassThatWillFail() {
             SmartStackTraceParser smartStackTraceParser =
                     new SmartStackTraceParser(CaseThatWillFail.class.getName(), e, null);
             String res = smartStackTraceParser.getString();
-            assertEquals("CaseThatWillFail.testThatWillFail:29 expected:<[abc]> but was:<[def]>", res);
+            assertEquals("CaseThatWillFail.testThatWillFail:25 expected:<[abc]> but was:<[def]>", res);
         }
     }
 
@@ -172,7 +172,7 @@ public void testAssertionWithNoMessage() {
             SmartStackTraceParser smartStackTraceParser =
                     new SmartStackTraceParser(AssertionNoMessage.class.getName(), e, null);
             String res = smartStackTraceParser.getString();
-            assertEquals("AssertionNoMessage.testThrowSomething:29 expected:<[abc]> but was:<[xyz]>", res);
+            assertEquals("AssertionNoMessage.testThrowSomething:25 expected:<[abc]> but was:<[xyz]>", res);
         }
     }
 
@@ -183,7 +183,7 @@ public void testFailWithFail() {
             SmartStackTraceParser smartStackTraceParser =
                     new SmartStackTraceParser(FailWithFail.class.getName(), e, null);
             String res = smartStackTraceParser.getString();
-            assertEquals("FailWithFail.testThatWillFail:29 abc", res);
+            assertEquals("FailWithFail.testThatWillFail:25 abc", res);
         }
     }
 
@@ -259,12 +259,12 @@ public void testStackTraceWithFocusOnClassAsString() {
 
             assertEquals(
                     "java.lang.RuntimeException: java.lang.IllegalStateException: java.io.IOException: I/O error\n"
-                            + "\tat org.apache.maven.surefire.report.StackTraceFocusedOnClass$B.b(StackTraceFocusedOnClass.java:65)\n"
+                            + "\tat org.apache.maven.surefire.report.StackTraceFocusedOnClass$B.b(StackTraceFocusedOnClass.java:49)\n"
                             + "Caused by: java.lang.IllegalStateException: java.io.IOException: I/O error\n"
-                            + "\tat org.apache.maven.surefire.report.StackTraceFocusedOnClass$B.b(StackTraceFocusedOnClass.java:61)\n"
+                            + "\tat org.apache.maven.surefire.report.StackTraceFocusedOnClass$B.b(StackTraceFocusedOnClass.java:47)\n"
                             + "Caused by: java.io.IOException: I/O error\n"
-                            + "\tat org.apache.maven.surefire.report.StackTraceFocusedOnClass$B.abs(StackTraceFocusedOnClass.java:73)\n"
-                            + "\tat org.apache.maven.surefire.report.StackTraceFocusedOnClass$B.b(StackTraceFocusedOnClass.java:61)\n",
+                            + "\tat org.apache.maven.surefire.report.StackTraceFocusedOnClass$B.abs(StackTraceFocusedOnClass.java:55)\n"
+                            + "\tat org.apache.maven.surefire.report.StackTraceFocusedOnClass$B.b(StackTraceFocusedOnClass.java:47)\n",
                     trace);
         }
     }