Upgrades tests and path variable names

jesperancinha · Jan 9, 2024 · 3a32c66 · 3a32c66
1 parent 7e4659e
commit 3a32c66
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/...in/java/org/jesperancinha/std/flash17/cipher/controllers/Flash17CSRFCreateController.java b/...in/java/org/jesperancinha/std/flash17/cipher/controllers/Flash17CSRFCreateController.java
@@ -15,7 +15,7 @@
 @Profile("test")
 public class Flash17CSRFCreateController {
 
-    private JdbcUserDetailsManager jdbcUserDetailsManager;
+    private final JdbcUserDetailsManager jdbcUserDetailsManager;
 
     private final PasswordEncoder passwordEncoder;
 
@@ -26,11 +26,11 @@ public Flash17CSRFCreateController(final JdbcUserDetailsManager jdbcUserDetailsM
 
     @GetMapping("/open/create/{name}/{password}/{role}")
     public void createUseViaGet(
-            @PathVariable
+            @PathVariable("name")
                     String name,
-            @PathVariable
+            @PathVariable("password")
                     String password,
-            @PathVariable
+            @PathVariable("role")
                     String role
     ) {
         jdbcUserDetailsManager.createUser(User.withUsername(name).password(passwordEncoder.encode(password)).roles(role).build());

diff --git a/...ava/org/jesperancinha/std/flash17/cipher/controllers/Flash17CSRFCreateControllerTest.java b/...ava/org/jesperancinha/std/flash17/cipher/controllers/Flash17CSRFCreateControllerTest.java
@@ -52,7 +52,7 @@ class Flash17CSRFCreateControllerTest {
 
     @Test
     @WithMockUser(roles = "ADMIN")
-    void testCreateUserViaGeWhenCreateUserViaGetThenOk() throws Exception {
+    void testCreateUserViaGetWhenCreateUserViaGetThenOk() throws Exception {
         mockMvc.perform(MockMvcRequestBuilders.get("/open/create/admin/password/ADMIN"))
                 .andExpect(status().isOk());