Apache
/
maven
mirror of https://github.com/apache/maven.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[MNG-7970] Remove extra space in MavenSimpleLogger stack trace formatter (#1342) 

Regular formatted stack traces do not have a space before the file/line
section.

before:
at java.util.zip.ZipFile$Source.get (ZipFile.java:1426)

after:
at java.util.zip.ZipFile$Source.get(ZipFile.java:1426)
This commit is contained in:
Michael Bien 2023-12-15 07:39:42 +01:00 committed by GitHub
parent 2cccc0a476
commit 3c65873b58
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
maven-slf4j-provider/src/main/java/org/slf4j/impl/MavenSimpleLogger.java
View File

@ -83,7 +83,7 @@ public class MavenSimpleLogger extends SimpleLogger {
builder.a(e.getClassName()); builder.a(e.getClassName());
builder.a("."); builder.a(".");
builder.a(e.getMethodName()); builder.a(e.getMethodName());
builder.a(" ("); builder.a("(");
builder.strong(getLocation(e)); builder.strong(getLocation(e));
builder.a(")"); builder.a(")");
stream.println(builder); stream.println(builder);

2
maven-slf4j-provider/src/test/java/org/slf4j/impl/MavenSimpleLoggerTest.java
View File

@ -51,7 +51,7 @@ class MavenSimpleLoggerTest {
Class<?> testClass = testInfo.getTestClass().get(); Class<?> testClass = testInfo.getTestClass().get();
String testMethodName = testInfo.getTestMethod().get().getName(); String testMethodName = testInfo.getTestMethod().get().getName();
String testClassStackTraceLinePattern = "at " + testClass.getName() + "." + testMethodName + " \\(" String testClassStackTraceLinePattern = "at " + testClass.getName() + "." + testMethodName + "\\("
+ testClass.getSimpleName() + ".java:\\d+\\)"; + testClass.getSimpleName() + ".java:\\d+\\)";
List<String> expectedLines = Arrays.asList( List<String> expectedLines = Arrays.asList(
"java.lang.RuntimeException: top-level", "java.lang.RuntimeException: top-level",