Fix OR condition in scoped attribute selector

smithy-model/src/main/java/software/amazon/smithy/model/selector/ScopedAttributeSelector.java

@@ -96,12 +96,18 @@ private boolean compareWithScope(AttributeValue scope) {
         // Ensure that each assertion matches, and provide them the scope.
         for (Assertion assertion : assertions) {
             AttributeValue lhs = assertion.lhs.create(scope);
+            boolean matchedOneRhs = false;
             for (ScopedFactory factory : assertion.rhs) {
                 AttributeValue rhs = factory.create(scope);
-                if (!assertion.comparator.compare(lhs, rhs, assertion.caseInsensitive)) {
-                    return false;
+                if (assertion.comparator.compare(lhs, rhs, assertion.caseInsensitive)) {
+                    matchedOneRhs = true;
+                    break;
                 }
             }
+
+            if (!matchedOneRhs) {
+                return false;
+            }
         }
 
         return true;

smithy-model/src/test/java/software/amazon/smithy/model/selector/SelectorTest.java

@@ -736,6 +736,24 @@ public void evaluatesScopedAttributes() {
         assertThat(shapes2, equalTo(shapes1));
     }
 
+    @Test
+    public void evaluatesScopedAttributesWithOr() {
+        final Model enumModel = Model.assembler()
+                .addImport(SelectorTest.class.getResource("enums.smithy"))
+                .assemble()
+                .unwrap();
+
+        Set<String> shapes = ids(
+            enumModel, "[@trait|enum|(values): @{name} ^= DIA, CLU]");
+
+        assertThat(shapes, containsInAnyOrder("smithy.example#Suit"));
+
+        shapes = ids(
+            enumModel, "[@trait|enum|(values): @{name} ^= DIA, BLA]");
+
+        assertThat(shapes, containsInAnyOrder("smithy.example#Color", "smithy.example#Suit"));
+    }
+
     @Test
     public void evaluatesScopedAttributesWithProjections() {
         // Note that the projection can be on either side.

smithy-model/src/test/resources/software/amazon/smithy/model/selector/enums.smithy

@@ -0,0 +1,17 @@
+$version: "1.0"
+
+namespace smithy.example
+
+@enum([
+    { name: "DIAMOND", value: "diamond"},
+    { name: "CLUB", value: "club"},
+    { name: "HEART", value: "heart"},
+    { name: "SPADE", value: "spade"},
+])
+string Suit
+
+@enum([
+    { name: "RED", value: "red"}
+    { name: "BLACK", value: "black"}
+])
+string Color