fixup 5

andrewwilsonnew · andrewwilsonnew · commit 128a926f119a · 2025-04-28T12:02:07.000+01:00
diff --git a/modules/openapi-generator/src/test/java/org/openapitools/codegen/AbstractOptionsTest.java b/modules/openapi-generator/src/test/java/org/openapitools/codegen/AbstractOptionsTest.java
@@ -62,7 +62,7 @@ public void checkOptionsHelp() {
             Assert.fail(String.format(Locale.ROOT, "These options weren't checked: %s.", StringUtils.join(skipped, ", ")));
         }
         final Set<String> undocumented = new HashSet<String>(testOptions);
-        cliOptions.forEach(undocumented::remove);
+        undocumented.removeAll(cliOptions);
         if (!undocumented.isEmpty()) {
             Assert.fail(String.format(Locale.ROOT, "These options weren't documented: %s. Are you expecting base options and calling cliOptions.clear()?", StringUtils.join(undocumented, ", ")));
         }

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ public void checkOptionsHelp() {`
`62`	`62`	`Assert.fail(String.format(Locale.ROOT, "These options weren't checked: %s.", StringUtils.join(skipped, ", ")));`
`63`	`63`	`}`
`64`	`64`	`final Set<String> undocumented = new HashSet<String>(testOptions);`
`65`		`- cliOptions.forEach(undocumented::remove);`
	`65`	`+ undocumented.removeAll(cliOptions);`
`66`	`66`	`if (!undocumented.isEmpty()) {`
`67`	`67`	`Assert.fail(String.format(Locale.ROOT, "These options weren't documented: %s. Are you expecting base options and calling cliOptions.clear()?", StringUtils.join(undocumented, ", ")));`
`68`	`68`	`}`