Merge pull request #38944 from FBibonne
* pr/38944: Polish "Fix error mark position for PatternParseException" Fix error mark position for PatternParseException Closes gh-38944
This commit is contained in:
commit
a0b034dfc8
|
|
@ -30,7 +30,7 @@ class PatternParseFailureAnalyzer extends AbstractFailureAnalyzer<PatternParseEx
|
|||
|
||||
@Override
|
||||
protected FailureAnalysis analyze(Throwable rootFailure, PatternParseException cause) {
|
||||
return new FailureAnalysis("Invalid mapping pattern detected: " + cause.toDetailedString(),
|
||||
return new FailureAnalysis("Invalid mapping pattern detected:\n" + cause.toDetailedString(),
|
||||
"Fix this pattern in your application or switch to the legacy parser implementation with "
|
||||
+ "'spring.mvc.pathmatch.matching-strategy=ant_path_matcher'.",
|
||||
cause);
|
||||
|
|
|
|||
|
|
@ -36,7 +36,8 @@ class PatternParseFailureAnalyzerTests {
|
|||
@Test
|
||||
void patternParseFailureQuotesPattern() {
|
||||
FailureAnalysis failureAnalysis = performAnalysis("/spring/**/framework");
|
||||
assertThat(failureAnalysis.getDescription()).contains("Invalid mapping pattern detected: /spring/**/framework");
|
||||
assertThat(failureAnalysis.getDescription())
|
||||
.contains("Invalid mapping pattern detected:\n" + "/spring/**/framework\n" + " ^");
|
||||
assertThat(failureAnalysis.getAction())
|
||||
.contains("Fix this pattern in your application or switch to the legacy parser"
|
||||
+ " implementation with 'spring.mvc.pathmatch.matching-strategy=ant_path_matcher'.");
|
||||
|
|
|
|||
Loading…
Reference in New Issue