Don't replace web annotation names that aren't camel cased

Author: jkschneider

src/main/java/org/openrewrite/spring/ImplicitWebAnnotationNames.java

@@ -100,6 +100,7 @@ private Optional<J.Literal> nameArgumentValue(J.Annotation annotation) {
                         .findAny()
                         .flatMap(arg -> arg.whenType(J.Assign.class)
                                 .map(assign -> assign.getAssignment().whenType(J.Literal.class))
-                                .orElse(arg.whenType(J.Literal.class)));
+                                .orElse(arg.whenType(J.Literal.class)))
+                        .filter(value -> value.getValue().toString().matches("[a-z][A-Za-z0-9]*"));
     }
 }

src/test/kotlin/org/openrewrite/spring/ImplicitWebAnnotationNamesTest.kt

@@ -128,4 +128,29 @@ class ImplicitWebAnnotationNamesTest {
             }
         """)
     }
+
+    @Test
+    fun onlyDropCamelCasedNames() {
+        val controller = jp.parse("""
+            import org.springframework.web.bind.annotation.*;
+            
+            public class UsersController {
+                public ResponseEntity<String> getUser(@PathVariable("id") Long id,
+                                                      @PathVariable(value = "another_name") Long anotherName) {
+                }
+            }
+        """.trimIndent())
+
+        val fixed = controller.refactor().visit(ImplicitWebAnnotationNames()).fix().fixed
+
+        assertRefactored(fixed, """
+            import org.springframework.web.bind.annotation.*;
+            
+            public class UsersController {
+                public ResponseEntity<String> getUser(@PathVariable Long id,
+                                                      @PathVariable(value = "another_name") Long anotherName) {
+                }
+            }
+        """)
+    }
 }