Make AssertTrue and AssertFalse mutually exclusive

Co-authored-by: Carsten Wickner <11309681+CarstenWickner@users.noreply.github.com>
victools · Jul 12, 2024 · caca6b5 · caca6b5
1 parent 00b9426
commit caca6b5
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/...ava/com/github/victools/jsonschema/module/jakarta/validation/JakartaValidationModule.java b/...ava/com/github/victools/jsonschema/module/jakarta/validation/JakartaValidationModule.java
@@ -512,17 +512,15 @@ protected BigDecimal resolveNumberExclusiveMaximum(MemberScope<?, ?> member) {
      * @see AssertFalse
      */
     protected List<Object> resolveEnum(MemberScope<?, ?> member) {
-        List<Object> values = new ArrayList<>();
-
+        List<Object> values;
         if (this.getAnnotationFromFieldOrGetter(member, AssertTrue.class, AssertTrue::groups) != null) {
-            values.add(true);
-        }
-
-        if (this.getAnnotationFromFieldOrGetter(member, AssertFalse.class, AssertFalse::groups) != null) {
-            values.add(false);
+            values = Collections.singletonList(true);
+        } else if (this.getAnnotationFromFieldOrGetter(member, AssertFalse.class, AssertFalse::groups) != null) {
+            values = Collections.singletonList(false);
+        } else {
+            values = null;
         }
-
-        return values.isEmpty() ? null : values;
+        return values;
     }
 
     /**