delete unused endpoints

ita-social-projects · Sep 12, 2023 · 0cb4ae1 · 0cb4ae1
1 parent b5f4967
commit 0cb4ae1
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 61 deletions.
diff --git a/core/src/main/java/greencity/controller/UserController.java b/core/src/main/java/greencity/controller/UserController.java
@@ -477,40 +477,6 @@ public UserAndAllFriendsWithOnlineStatusDto getUserAndAllFriendsWithOnlineStatus
         return userService.getAllFriendsWithTheOnlineStatus(userId, pageable);
     }
 
-    /**
-     * Method find user by principal.
-     *
-     * @return {@link ResponseEntity}.
-     * @author Orest Mamchuk
-     */
-    @ApiOperation(value = "Find current user by principal")
-    @ApiResponses(value = {
-        @ApiResponse(code = 200, message = HttpStatuses.OK),
-        @ApiResponse(code = 400, message = HttpStatuses.BAD_REQUEST),
-        @ApiResponse(code = 401, message = HttpStatuses.UNAUTHORIZED),
-    })
-    @GetMapping("/findByEmail")
-    public ResponseEntity<UserVO> findByEmail(@RequestParam String email) {
-        return ResponseEntity.status(HttpStatus.OK).body(userService.findByEmail(email));
-    }
-
-    /**
-     * Get {@link UserVO} by id.
-     *
-     * @return {@link UserUpdateDto}.
-     * @author Orest Mamchuk
-     */
-    @ApiOperation(value = "Get User by id")
-    @ApiResponses(value = {
-        @ApiResponse(code = 200, message = HttpStatuses.OK),
-        @ApiResponse(code = 400, message = HttpStatuses.BAD_REQUEST),
-        @ApiResponse(code = 401, message = HttpStatuses.UNAUTHORIZED),
-    })
-    @GetMapping("/findById")
-    public ResponseEntity<UserVO> findById(@RequestParam Long id) {
-        return ResponseEntity.status(HttpStatus.OK).body(userService.findById(id));
-    }
-
     /**
      * Method that allow you to find {@link UserVO} by Id.
      *

diff --git a/core/src/test/java/greencity/controller/UserControllerTest.java b/core/src/test/java/greencity/controller/UserControllerTest.java
@@ -48,8 +48,6 @@
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 
-import static org.mockito.ArgumentMatchers.any;
-import static org.mockito.ArgumentMatchers.anyLong;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
@@ -406,31 +404,6 @@ void searchTest() throws Exception {
         verify(userService).search(pageable, userViewDto);
     }
 
-    @Test
-    void findByEmailTest() throws Exception {
-        UserVO userVO = ModelUtils.getUserVO();
-        when(userService.findByEmail(TestConst.EMAIL)).thenReturn(userVO);
-        mockMvc.perform(get(userLink + "/findByEmail")
-            .param("email", TestConst.EMAIL))
-            .andExpect(status().isOk())
-            .andExpect(jsonPath("$.id").value(1L))
-            .andExpect(jsonPath("$.name").value(TestConst.NAME))
-            .andExpect(jsonPath("$.email").value(TestConst.EMAIL));
-    }
-
-    @Test
-    void findByIdTest() throws Exception {
-        UserVO userVO = ModelUtils.getUserVO();
-        when(userService.findById(1L)).thenReturn(userVO);
-        mockMvc.perform(get(userLink + "/findById")
-            .param("id", "1"))
-            .andExpect(status().isOk())
-            .andExpect(content().contentType("application/json;charset=UTF-8"))
-            .andExpect(jsonPath("$.id").value(1L))
-            .andExpect(jsonPath("$.name").value(TestConst.NAME))
-            .andExpect(jsonPath("$.email").value(TestConst.EMAIL));
-    }
-
     @Test
     void findUserForAchievementTest() throws Exception {
         UserVOAchievement userVOAchievement = UserVOAchievement.builder()