Skip to content

Commit f3405e0

Browse files
authored
Merge pull request #88 from navalcode/gd_sr_postService_getPost
PostService GetPost and modify comment methods other classes
2 parents 056b60c + d1f6760 commit f3405e0

4 files changed

Lines changed: 76 additions & 1 deletion

File tree

src/test/java/com/sopromadze/blogapi/controllerTest/userController/CurrentUser.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import com.sopromadze.blogapi.security.UserPrincipal;
1010
import com.sopromadze.blogapi.service.UserService;
1111
import lombok.extern.java.Log;
12+
import org.junit.jupiter.api.DisplayName;
1213
import org.junit.jupiter.api.Test;
1314
import org.springframework.beans.factory.annotation.Autowired;
1415
import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
@@ -46,6 +47,7 @@ public class CurrentUser {
4647
*/
4748
@Test
4849
@WithMockUser(authorities = {"ROLE_USER"})
50+
@DisplayName ("Current user successfully")
4951
void currentUser_success() throws Exception{
5052

5153
User user = new User();
@@ -76,6 +78,7 @@ void currentUser_success() throws Exception{
7678
Salida esperada: Test exitoso, codigo de respuesta correcto (403)
7779
*/
7880
@Test
81+
@DisplayName ("Error 403 current user")
7982
void currentUser_successWhen403() throws Exception{
8083

8184
User user = new User();

src/test/java/com/sopromadze/blogapi/controllerTest/userController/TakeAdmin.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
import com.sopromadze.blogapi.payload.ApiResponse;
1010
import com.sopromadze.blogapi.service.UserService;
1111
import lombok.extern.java.Log;
12+
import org.junit.jupiter.api.DisplayName;
1213
import org.junit.jupiter.api.Test;
1314
import org.springframework.beans.factory.annotation.Autowired;
1415
import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
@@ -47,6 +48,7 @@ public class TakeAdmin {
4748
*/
4849
@Test
4950
@WithMockUser (authorities = {"ROLE_ADMIN"})
51+
@DisplayName ("Take admin successfully")
5052
void takeAdmin_success() throws Exception{
5153

5254
User user = new User();
@@ -78,6 +80,7 @@ void takeAdmin_success() throws Exception{
7880
*/
7981
@Test
8082
@WithMockUser (authorities = {"ROLE_USER"}) // o sin autorización
83+
@DisplayName ("Error code 403 Take admin")
8184
void takeAdmin_successWhen403() throws Exception{
8285

8386
User user = new User();

src/test/java/com/sopromadze/blogapi/serviceTest/photo/addPhoto.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ void initTest() {
9393
*/
9494
@Test
9595
@DisplayName ("Add photo successfully")
96-
void addPhoto_succes(){
96+
void addPhoto_success(){
9797
when(albumRepository.findById(photoRequest.getAlbumId())).thenReturn(Optional.of(album));
9898
when(photoRepository.findById(photo.getId())).thenReturn(Optional.of(photo));
9999
assertEquals(photo.getAlbum().getUser().getId(),user.getId());
Lines changed: 69 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,69 @@
1+
package com.sopromadze.blogapi.serviceTest.post;
2+
3+
4+
import com.sopromadze.blogapi.exception.ResourceNotFoundException;
5+
import com.sopromadze.blogapi.model.Post;
6+
import com.sopromadze.blogapi.repository.PostRepository;
7+
import com.sopromadze.blogapi.repository.UserRepository;
8+
import com.sopromadze.blogapi.service.impl.PostServiceImpl;
9+
import org.junit.jupiter.api.BeforeEach;
10+
import org.junit.jupiter.api.DisplayName;
11+
import org.junit.jupiter.api.Test;
12+
import org.junit.jupiter.api.extension.ExtendWith;
13+
import org.mockito.InjectMocks;
14+
import org.mockito.Mock;
15+
import org.mockito.junit.jupiter.MockitoExtension;
16+
import org.mockito.junit.jupiter.MockitoSettings;
17+
import org.mockito.quality.Strictness;
18+
19+
import static org.junit.jupiter.api.Assertions.assertEquals;
20+
import static org.junit.jupiter.api.Assertions.assertThrows;
21+
import static org.mockito.Mockito.when;
22+
23+
@ExtendWith (MockitoExtension.class)
24+
@MockitoSettings (strictness = Strictness.LENIENT)
25+
public class GetPost {
26+
27+
@Mock
28+
private PostRepository postRepository;
29+
@InjectMocks
30+
private PostServiceImpl postService;
31+
32+
Post post;
33+
34+
@BeforeEach
35+
void initTest() {
36+
post = new Post();
37+
post.setId(1L);
38+
post.setTitle("Titulo");
39+
40+
}
41+
42+
/*
43+
Test: Obtener un post
44+
Entrada: id(post)
45+
Salida esperada: El test se realiza con exito cuando obtiene el post por su id
46+
*/
47+
@Test
48+
@DisplayName ("Get post successfully")
49+
void getPost_success(){
50+
when(postRepository.findById(1L)).thenReturn(java.util.Optional.of(post));
51+
assertEquals(post,postService.getPost(post.getId()));
52+
}
53+
54+
/*
55+
Test: Al obtener un post, salte excepcion ResourceNotFoundException
56+
Entrada: id(post)
57+
Salida esperada: El test se realiza con exito cuando obtiene la excepcion en la peticion
58+
*/
59+
@Test
60+
@DisplayName ("Get post exception ResourceNotFoundException")
61+
void getPost_failException(){
62+
when(postRepository.findById(1L)).thenReturn(java.util.Optional.of(post));
63+
assertEquals(post,postService.getPost(post.getId()));
64+
65+
assertThrows(ResourceNotFoundException.class, () -> postService.getPost(0L));
66+
67+
}
68+
69+
}

0 commit comments

Comments
 (0)