This is an automated email from the ASF dual-hosted git repository.
michaelo pushed a commit to branch backport
in repository https://gitbox.apache.org/repos/asf/maven-surefire.git
commit 009c45c058aae5f218c14e155a6fedd2c2ea21a8
Author: Michael Osipov <micha...@apache.org>
AuthorDate: Fri May 19 21:25:19 2023 +0200
Use typographically correct en dash approximations
https://en.wikipedia.org/wiki/Dash#Approximating_the_em_dash_with_two_or_three_hyphens
---
.../extensions/junit5/JUnit5StatelessTestsetInfoReporter.java | 2 +-
.../java/org/apache/maven/plugin/surefire/report/TestSetStats.java | 2 +-
.../org/apache/maven/plugin/surefire/report/WrappedReportEntry.java | 2 +-
.../apache/maven/plugin/surefire/report/WrappedReportEntryTest.java | 6 +++---
.../maven/surefire/extensions/StatelessTestsetInfoReporterTest.java | 2 +-
.../java/org/apache/maven/surefire/its/JUnit47ConcurrencyIT.java | 2 +-
.../java/org/apache/maven/surefire/its/JUnitPlatformEnginesIT.java | 2 +-
.../maven/surefire/its/jiras/Surefire1185DoNotSpawnTestsIT.java | 4 ++--
.../its/jiras/Surefire747MethodParallelWithSuiteCountIT.java | 4 +++-
9 files changed, 14 insertions(+), 12 deletions(-)
diff --git
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/junit5/JUnit5StatelessTestsetInfoReporter.java
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/junit5/JUnit5StatelessTestsetInfoReporter.java
index ef44a7755..77987e690 100644
---
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/junit5/JUnit5StatelessTestsetInfoReporter.java
+++
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/extensions/junit5/JUnit5StatelessTestsetInfoReporter.java
@@ -56,7 +56,7 @@ public class JUnit5StatelessTestsetInfoReporter extends
SurefireStatelessTestset
/**
* Phrased class name of test case in the log (see xxx)
- * <em>Tests run: ., Failures: ., Errors: ., Skipped: ., Time elapsed: .
s, - in xxx</em>.
+ * <em>Tests run: ., Failures: ., Errors: ., Skipped: ., Time elapsed: .
s, -- in xxx</em>.
* {@code false} by default.
* <br>
* This action takes effect only in JUnit5 provider together with a test
class annotated <em>DisplayName</em>.
diff --git
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java
index 6e79000dc..96b4fb451 100644
---
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java
+++
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/TestSetStats.java
@@ -41,7 +41,7 @@ public class TestSetStats {
private static final String ERRORS = "Errors: ";
private static final String SKIPPED = "Skipped: ";
private static final String FAILURE_MARKER = " <<< FAILURE!";
- private static final String IN_MARKER = " - in ";
+ private static final String IN_MARKER = " -- in ";
private static final String COMMA = ", ";
private final Queue<WrappedReportEntry> reportEntries = new
ConcurrentLinkedQueue<>();
diff --git
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
index c3633f556..6f2073040 100644
---
a/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
+++
b/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
@@ -180,7 +180,7 @@ public class WrappedReportEntry implements
TestSetReportEntry {
public String getElapsedTimeSummary() {
String description = getName() == null ? getSourceName() :
getClassMethodName();
- return description + " " + getElapsedTimeVerbose();
+ return description + " -- " + getElapsedTimeVerbose();
}
public boolean isErrorOrFailure() {
diff --git
a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/WrappedReportEntryTest.java
b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/WrappedReportEntryTest.java
index 97e1b4cca..6d337467a 100644
---
a/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/WrappedReportEntryTest.java
+++
b/maven-surefire-common/src/test/java/org/apache/maven/plugin/surefire/report/WrappedReportEntryTest.java
@@ -76,9 +76,9 @@ public class WrappedReportEntryTest extends TestCase {
assertTrue(wr.isErrorOrFailure());
assertFalse(wr.isSkipped());
assertNull(wr.getStackTraceWriter());
- assertEquals("surefire.testcase.JunitParamsTest.testSum Time elapsed:
0.012 s", wr.getElapsedTimeSummary());
+ assertEquals("surefire.testcase.JunitParamsTest.testSum -- Time
elapsed: 0.012 s", wr.getElapsedTimeSummary());
assertEquals(
- "surefire.testcase.JunitParamsTest.testSum Time elapsed:
0.012 s <<< ERROR!", wr.getOutput(false));
+ "surefire.testcase.JunitParamsTest.testSum -- Time elapsed:
0.012 s <<< ERROR!", wr.getOutput(false));
assertEquals("exception", wr.getMessage());
}
@@ -147,6 +147,6 @@ public class WrappedReportEntryTest extends TestCase {
ReportEntry reportEntry = new SimpleReportEntry(NORMAL_RUN, 1L,
className, null, null, null);
WrappedReportEntry wr = new WrappedReportEntry(reportEntry, null, 12,
null, null);
String elapsedTimeSummary = wr.getElapsedTimeSummary();
- assertEquals("[0] 1, 2, 3 (testSum) Time elapsed: 0.012 s",
elapsedTimeSummary);
+ assertEquals("[0] 1, 2, 3 (testSum) -- Time elapsed: 0.012 s",
elapsedTimeSummary);
}
}
diff --git
a/maven-surefire-common/src/test/java/org/apache/maven/surefire/extensions/StatelessTestsetInfoReporterTest.java
b/maven-surefire-common/src/test/java/org/apache/maven/surefire/extensions/StatelessTestsetInfoReporterTest.java
index a7bd1b0ac..0c2a98a50 100644
---
a/maven-surefire-common/src/test/java/org/apache/maven/surefire/extensions/StatelessTestsetInfoReporterTest.java
+++
b/maven-surefire-common/src/test/java/org/apache/maven/surefire/extensions/StatelessTestsetInfoReporterTest.java
@@ -193,7 +193,7 @@ public class StatelessTestsetInfoReporterTest {
.hasSize(2)
.containsSequence(
"Tests run: 1, Failures: 1, Errors: 0, Skipped: 0,
Time elapsed: 1.03 s "
- + "<<< FAILURE! - in pkg.MyTest",
+ + "<<< FAILURE! -- in pkg.MyTest",
"pkg.MyTest failed");
verifyNoMoreInteractions(consoleLogger);
}
diff --git
a/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnit47ConcurrencyIT.java
b/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnit47ConcurrencyIT.java
index fd9266e9b..681fa0080 100644
---
a/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnit47ConcurrencyIT.java
+++
b/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnit47ConcurrencyIT.java
@@ -51,6 +51,6 @@ public class JUnit47ConcurrencyIT extends
SurefireJUnit4IntegrationTestCase {
containsString("Time elapsed: 1."),
containsString("Time elapsed: 1 s"),
containsString("Time elapsed: 0.9")));
- assertThat(result, endsWith(" s - in
concurrentjunit47.src.test.java.junit47.BasicTest"));
+ assertThat(result, endsWith(" s -- in
concurrentjunit47.src.test.java.junit47.BasicTest"));
}
}
diff --git
a/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformEnginesIT.java
b/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformEnginesIT.java
index ee4e7fa36..d6513b167 100644
---
a/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformEnginesIT.java
+++
b/surefire-its/src/test/java/org/apache/maven/surefire/its/JUnitPlatformEnginesIT.java
@@ -381,7 +381,7 @@ public class JUnitPlatformEnginesIT extends
SurefireJUnit4IntegrationTestCase {
validator
.getSurefireReportsFile("junitplatformenginejupiter.DisplayNameTest.txt", UTF_8)
- .assertContainsText(" - in << ✨ >>");
+ .assertContainsText(" -- in << ✨ >>");
validator
.getSurefireReportsFile("junitplatformenginejupiter.DisplayNameTest-output.txt",
UTF_8)
diff --git
a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1185DoNotSpawnTestsIT.java
b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1185DoNotSpawnTestsIT.java
index 898c2d152..9579d721a 100644
---
a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1185DoNotSpawnTestsIT.java
+++
b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire1185DoNotSpawnTestsIT.java
@@ -32,9 +32,9 @@ import static org.hamcrest.Matchers.is;
* Example, UnlistedTest is the problem here because it runs with filtered out
methods:
*
* Running pkg.UnlistedTest
- * Tests run: 0, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 sec -
in pkg.UnlistedTest
+ * Tests run: 0, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 sec
-- in pkg.UnlistedTest
* Running pkg.RunningTest
- * Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 sec -
in pkg.RunningTest
+ * Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 sec
-- in pkg.RunningTest
*
* Results:
*
diff --git
a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire747MethodParallelWithSuiteCountIT.java
b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire747MethodParallelWithSuiteCountIT.java
index 6005dec78..122071331 100644
---
a/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire747MethodParallelWithSuiteCountIT.java
+++
b/surefire-its/src/test/java/org/apache/maven/surefire/its/jiras/Surefire747MethodParallelWithSuiteCountIT.java
@@ -115,7 +115,9 @@ public class Surefire747MethodParallelWithSuiteCountIT
extends SurefireJUnit4Int
containsString("Time elapsed: " + delayMax)));
assertThat(
line,
- anyOf(endsWith(" s - in surefire747.SuiteTest1"),
endsWith(" s - in surefire747.SuiteTest2")));
+ anyOf(
+ endsWith(" s -- in surefire747.SuiteTest1"),
+ endsWith(" s -- in surefire747.SuiteTest2")));
}
}
}
