Fix formatting of inline expectation test comment

owen-mc · owen-mc · commit 098b732937ba · 2024-06-14T14:04:42.000+01:00
diff --git a/java/ql/test/query-tests/security/CWE-807/semmle/tests/ConditionalBypassTest.java b/java/ql/test/query-tests/security/CWE-807/semmle/tests/ConditionalBypassTest.java
@@ -38,7 +38,7 @@ public static void main(HttpServletRequest request) throws Exception {
 		}
 
 		// FALSE NEGATIVE: we have no way of telling that the skipped method is sensitive
-		if (adminCookie.getValue() == "false") // $ MISSING: $ hasConditionalBypassTest
+		if (adminCookie.getValue() == "false") // $ MISSING: hasConditionalBypassTest
 			doReallyImportantSecurityWork();
 
 		InetAddress local = InetAddress.getLocalHost();

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public static void main(HttpServletRequest request) throws Exception {`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`// FALSE NEGATIVE: we have no way of telling that the skipped method is sensitive`
`41`		`- if (adminCookie.getValue() == "false") // $ MISSING: $ hasConditionalBypassTest`
	`41`	`+ if (adminCookie.getValue() == "false") // $ MISSING: hasConditionalBypassTest`
`42`	`42`	`doReallyImportantSecurityWork();`
`43`	`43`
`44`	`44`	`InetAddress local = InetAddress.getLocalHost();`