improved code coverage

Author: bitcoder

pom.xml

@@ -208,7 +208,9 @@
 					</excludes>
 
 					<testFailureIgnore>false</testFailureIgnore>
-					<disableXmlReport>true</disableXmlReport>
+                    <statelessTestsetReporter implementation="org.apache.maven.plugin.surefire.extensions.junit5.JUnit5Xml30StatelessReporter">
+                        <disable>true</disable>
+                    </statelessTestsetReporter>
 					<useFile>false</useFile>
 				</configuration>
 			</plugin>

src/test/java/com/sergiofreire/xray/tutorials/springboot/UserRestControllerIT.java

@@ -1,6 +1,5 @@
 package com.sergiofreire.xray.tutorials.springboot;
 
-import org.json.JSONException;
 import org.json.JSONObject;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;

src/test/java/com/sergiofreire/xray/tutorials/springboot/UserServiceUnitTest.java

@@ -45,10 +45,28 @@ void setUp() {
 
 
         Mockito.when(userRepository.findById(john.getId())).thenReturn(Optional.of(john));
+        Mockito.when(userRepository.findByUsername(john.getUsername())).thenReturn(john);
         Mockito.when(userRepository.findAll()).thenReturn(allUsers);
         Mockito.when(userRepository.findById(-99L)).thenReturn(Optional.empty());
     }
 
+    @Test
+     void getUserByUsernameReturnsUserIfExisting() {
+        Optional<User> user = userService.getUserByUsername("johndoe");
+
+        Mockito.verify(userRepository, VerificationModeFactory.times(1)).findByUsername(Mockito.anyString());
+        assertThat(user.get().getName()).isEqualTo("John Doe");
+        assertThat(user.get().getUsername()).isEqualTo("johndoe");
+    }
+
+    @Test
+     void getUserByUsernameReturnsEmptyIfNonExisting() {
+        Optional<User> user = userService.getUserByUsername("unknown");
+
+        Mockito.verify(userRepository, VerificationModeFactory.times(1)).findByUsername(Mockito.anyString());
+        assertThat(user).isNotPresent();
+    }
+
     @Test
      void getUserDetailsReturnsUserIfExisting() {
         Optional<User> user = userService.getUserDetails(999L);
@@ -66,6 +84,22 @@ void getUserDetailsReturnsEmptyIfNonExisting() {
         assertThat(user).isNotPresent();
     }
 
+    @Test
+     void existsReturnsTrueIfExisting() {
+        boolean exists = userService.exists("johndoe");
+
+        Mockito.verify(userRepository, VerificationModeFactory.times(1)).findByUsername(Mockito.anyString());
+        assertThat(exists).isTrue();
+    }
+
+    @Test
+     void existsReturnsFalseIfNonExisting() {
+        boolean exists = userService.exists("unknown");
+
+        Mockito.verify(userRepository, VerificationModeFactory.times(1)).findByUsername(Mockito.anyString());
+        assertThat(exists).isFalse();
+    }
+
     @Test
      void getAllUsersReturnsAllExistingUsers() {
         User john = new User("John Doe", "johndoe", "dummypassword");