Update test results (remove SPURIOUS annotations)

owen-mc · owen-mc · commit cf73d96c9df4 · 2026-02-16T12:03:02.000Z
diff --git a/java/ql/test/query-tests/security/CWE-117/LogInjectionTest.expected b/java/ql/test/query-tests/security/CWE-117/LogInjectionTest.expected
@@ -8129,6 +8129,3 @@ nodes
 | LogInjectionTest.java:2151:38:2151:54 | (...)... | semmle.label | (...)... |
 | LogInjectionTest.java:2151:47:2151:54 | source(...) : Object | semmle.label | source(...) : Object |
 subpaths
-testFailures
-| LogInjectionTest.java:198:44:198:63 | // $ SPURIOUS: Alert | Fixed spurious result: Alert |
-| LogInjectionTest.java:199:41:199:60 | // $ SPURIOUS: Alert | Fixed spurious result: Alert |
diff --git a/java/ql/test/query-tests/security/CWE-117/LogInjectionTest.java b/java/ql/test/query-tests/security/CWE-117/LogInjectionTest.java
@@ -195,8 +195,8 @@ public void testGuards() {
             logger.debug(source); // $ MISSING: $ Alert
         }
 
-        logger.debug(validatedInputField); // $ SPURIOUS: Alert
-        logger.debug(validatedInput()); // $ SPURIOUS: Alert
+        logger.debug(validatedInputField);
+        logger.debug(validatedInput());
     }
 
     public void test() {

Original file line number	Diff line number	Diff line change
`@@ -195,8 +195,8 @@ public void testGuards() {`
`195`	`195`	`logger.debug(source); // $ MISSING: $ Alert`
`196`	`196`	`}`
`197`	`197`
`198`		`- logger.debug(validatedInputField); // $ SPURIOUS: Alert`
`199`		`- logger.debug(validatedInput()); // $ SPURIOUS: Alert`
	`198`	`+ logger.debug(validatedInputField);`
	`199`	`+ logger.debug(validatedInput());`
`200`	`200`	`}`
`201`	`201`
`202`	`202`	`public void test() {`