Suggestions

PicnicSupermarket · Aug 19, 2022 · cd337d8 · cd337d8
1 parent 3d01f1e
commit cd337d8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/...in/java/tech/picnic/errorprone/bugpatterns/LexicographicalAnnotationAttributeListing.java b/...in/java/tech/picnic/errorprone/bugpatterns/LexicographicalAnnotationAttributeListing.java
@@ -63,6 +63,7 @@ public final class LexicographicalAnnotationAttributeListing extends BugChecker
           "io.swagger.annotations.ApiImplicitParams#value",
           "io.swagger.v3.oas.annotations.Parameters#value",
           "javax.xml.bind.annotation.XmlType#propOrder",
+          "org.springframework.context.annotation.PropertySource#value",
           "org.springframework.test.context.TestPropertySource#locations",
           "org.springframework.test.context.TestPropertySource#value");
   private static final String FLAG_PREFIX = "LexicographicalAnnotationAttributeListing:";

diff --git a/...ava/tech/picnic/errorprone/bugpatterns/LexicographicalAnnotationAttributeListingTest.java b/...ava/tech/picnic/errorprone/bugpatterns/LexicographicalAnnotationAttributeListingTest.java
@@ -35,6 +35,7 @@ void identification() {
             "import io.swagger.v3.oas.annotations.Parameters;",
             "import java.math.RoundingMode;",
             "import javax.xml.bind.annotation.XmlType;",
+            "import org.springframework.context.annotation.PropertySource;",
             "import org.springframework.test.context.TestPropertySource;",
             "",
             "interface A {",
@@ -145,13 +146,16 @@ void identification() {
             "  A secondEndpoint();",
             "",
             "  @XmlType(propOrder = {\"field2\", \"field1\"})",
-            "  class FirstDummy {}",
+            "  class XmlTypeDummy {}",
+            "",
+            "  @PropertySource({\"field2\", \"field1\"})",
+            "  class PropertySourceDummy {}",
             "",
             "  @TestPropertySource(locations = {\"field2\", \"field1\"})",
-            "  class SecondDummy {}",
+            "  class FirstTestPropertySourceDummy {}",
             "",
             "  @TestPropertySource({\"field2\", \"field1\"})",
-            "  class ThirdDummy {}",
+            "  class SecondTestPropertySourceDummy {}",
             "}")
         .doTest();
   }