ottlinger commented on code in PR #264:
URL: https://github.com/apache/creadur-rat/pull/264#discussion_r1626628632
##########
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:
@Claudenw it seems that assertJ has no equivalent to this supplier construct.
Do you have an idea how to transform these kind of tests into assertJ or
should we leave the other assertTrue() - WDYT?
--
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: dev-unsubscr...@creadur.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
