Refactor tests to get rid of a parameterized test

[#163096443] Signed-off-by: Natalie Bennett <nbennett@pivotal.io> Co-authored-by: Natalie Bennett <nbennett@pivotal.io>
cloudfoundry · Feb 1, 2019 · 84b0c13 · 84b0c13
1 parent 67b3925
commit 84b0c13
Show file tree

Hide file tree

Showing 3 changed files with 222 additions and 214 deletions.
diff --git a/server/src/main/java/org/cloudfoundry/identity/uaa/scim/endpoints/ScimUserEndpoints.java b/server/src/main/java/org/cloudfoundry/identity/uaa/scim/endpoints/ScimUserEndpoints.java
@@ -615,7 +615,7 @@ public void setStatuses(Map<Class<? extends Exception>, HttpStatus> statuses) {
         this.statuses = statuses;
     }
 
-    private void throwWhenInvalidSelfEdit(@RequestBody ScimUser user, @PathVariable String userId, HttpServletRequest request, Authentication authentication) {
+    private void throwWhenInvalidSelfEdit(ScimUser user, String userId, HttpServletRequest request, Authentication authentication) {
         boolean isSelfEdit = isSelfCheck.isUserSelf(request, 1);
         if (!isSelfEdit) {
             return;