Claudenw commented on code in PR #264:
URL: https://github.com/apache/creadur-rat/pull/264#discussion_r1626949174
##########
apache-rat-core/src/test/java/org/apache/rat/analysis/license/AbstractLicenseTest.java:
##########
@@ -110,8 +107,7 @@ public void testEmbeddedStrings(String id, String family,
String name, String no
for (String fmt : formats) {
boolean found = processText(license, String.format(fmt,
target[TEXT]));
license.reset();
- assertTrue(found, () -> String.format("%s %s did not match
pattern '%s' for target string %s", id,
Review Comment:
You could generate the message string and just use it. It will have some
performance impact. Or you could just leave the junit 5 format
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
