refactor: code

Author: ThisAster

src/main/java/com/thisaster/testtask/subscription/dto/SubscriptionDTO.java

@@ -16,7 +16,7 @@ public class SubscriptionDTO {
 
     @JsonInclude(JsonInclude.Include.NON_NULL)
     @JsonProperty(access = JsonProperty.Access.READ_ONLY)
-    private Long id;
+    private long id;
     @NotBlank
     private String name;
 }

src/main/java/com/thisaster/testtask/subscription/entity/Subscription.java

@@ -22,7 +22,7 @@
 public class Subscription {
     @Id
     @GeneratedValue(strategy = GenerationType.IDENTITY)
-    private Long id;
+    private long id;
     @Column(unique = true, nullable = false)
     private String name;
 }

src/main/java/com/thisaster/testtask/user/dto/UserDTO.java

@@ -34,7 +34,7 @@ public class UserDTO {
     @JsonProperty(access = JsonProperty.Access.READ_WRITE)
     private Set<SubscriptionDTO> subscriptions;
 
-    @JsonProperty(access = JsonProperty.Access.READ_WRITE)
+    @JsonProperty(access = JsonProperty.Access.WRITE_ONLY)
 
     @NotBlank(message = "Role cannot be blank")
     @Pattern(regexp = "^(ADMIN|USER|SUPERVISOR)$", message = "Role must be either admin or user")

src/test/java/com/thisaster/testtask/user/controller/UserControllerTest.java

@@ -127,7 +127,6 @@ void shouldUpdateUser(@Value("classpath:user/updateUser.json") Resource json) th
                 .andExpect(jsonPath("$.subscriptions[*].name", containsInAnyOrder(
                         "Premium Plan", "Newsletter", "Yandex Premium"
                 )))
-                .andExpect(jsonPath("$.role", is("SUPERVISOR")))
                 .andReturn();
 
         System.out.println("Result : " + result.getResponse().getContentAsString());