Fix test

joefarebrother · joefarebrother · commit c419e8d24a60 · 2023-06-26T18:05:11.000+01:00
diff --git a/csharp/ql/test/query-tests/Security Features/CWE-285/MissingAccessControl/MVCTests/MissingAccessControl.expected b/csharp/ql/test/query-tests/Security Features/CWE-285/MissingAccessControl/MVCTests/MissingAccessControl.expected
@@ -1 +1 @@
-| ProfileController.cs:8:25:8:31 | Delete1 | This action is missing an authorization check. |
+| ProfileController.cs:9:25:9:31 | Delete1 | This action is missing an authorization check. |
diff --git a/csharp/ql/test/query-tests/Security Features/CWE-285/MissingAccessControl/MVCTests/ProfileController.cs b/csharp/ql/test/query-tests/Security Features/CWE-285/MissingAccessControl/MVCTests/ProfileController.cs
@@ -23,7 +23,7 @@ public ActionResult Delete2(int id) {
     // GOOD: The Authorize attribute is used.
     [Authorize]
     public ActionResult Delete3(int id) {
-        doThings()
+        doThings();
         return View();
     }
 

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-\| ProfileController.cs:8:25:8:31 \| Delete1 \| This action is missing an authorization check. \|`
	`1`	`+\| ProfileController.cs:9:25:9:31 \| Delete1 \| This action is missing an authorization check. \|`
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ public ActionResult Delete2(int id) {`
`23`	`23`	`// GOOD: The Authorize attribute is used.`
`24`	`24`	`[Authorize]`
`25`	`25`	`public ActionResult Delete3(int id) {`
`26`		`- doThings()`
	`26`	`+ doThings();`
`27`	`27`	`return View();`
`28`	`28`	`}`
`29`	`29`