Convert test to inline expectations

owen-mc · owen-mc · commit b5b252b10f7f · 2025-04-10T11:47:15.000+01:00
diff --git a/java/ql/test/query-tests/StringReplaceAllWithNonRegex/StringReplaceAllWithNonRegex.qlref b/java/ql/test/query-tests/StringReplaceAllWithNonRegex/StringReplaceAllWithNonRegex.qlref
@@ -1 +1,2 @@
-Performance/StringReplaceAllWithNonRegex.ql
+query: Performance/StringReplaceAllWithNonRegex.ql
+postprocess: utils/test/InlineExpectationsTestQuery.ql
diff --git a/java/ql/test/query-tests/StringReplaceAllWithNonRegex/Test.java b/java/ql/test/query-tests/StringReplaceAllWithNonRegex/Test.java
@@ -1,7 +1,7 @@
 public class Test {
     void f() {
         String s1 = "test";
-        s1 = s1.replaceAll("t", "x"); // NON_COMPLIANT
-        s1 = s1.replaceAll(".*", "x"); // COMPLIANT
+        s1 = s1.replaceAll("t", "x"); // $ Alert
+        s1 = s1.replaceAll(".*", "x");
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -1 +1,2 @@`
`1`		`-Performance/StringReplaceAllWithNonRegex.ql`
	`1`	`+query: Performance/StringReplaceAllWithNonRegex.ql`
	`2`	`+postprocess: utils/test/InlineExpectationsTestQuery.ql`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`public class Test {`
`2`	`2`	`void f() {`
`3`	`3`	`String s1 = "test";`
`4`		`- s1 = s1.replaceAll("t", "x"); // NON_COMPLIANT`
`5`		`- s1 = s1.replaceAll(".*", "x"); // COMPLIANT`
	`4`	`+ s1 = s1.replaceAll("t", "x"); // $ Alert`
	`5`	`+ s1 = s1.replaceAll(".*", "x");`
`6`	`6`	`}`
`7`	`7`	`}`