FIX: Tests now work

src/Server/src/main/java/greenify/server/service/UserService.java

@@ -87,7 +87,7 @@ public class UserService {
     public void addFriend(Long id, String name, String friend) {
         User user = userRepository.findByName(name);
         User add = userRepository.findByName(friend);
-        user.addFriend(user);
+        user.addFriend(add);
         userRepository.save(user);
         logger.info("Added friend to user(id=" + user.getId()
                 + ", name=" + user.getName() + ")");

src/Server/src/test/java/greenify/server/service/UserServiceTest.java

@@ -42,14 +42,17 @@ public class UserServiceTest {
      */
     @Before
     public void setUp() {
-        User alex = new User(1L, "alex", "password", 0);
+        User user = new User(1L, "user", "password", 0);
-        when(userRepository.findByName(alex.getName()))
+        when(userRepository.findByName(user.getName()))
-                .thenReturn(alex);
+                .thenReturn(user);
+        User friend = new User(2L, "friend", "password", 0);
+        when(userRepository.findByName(friend.getName()))
+                .thenReturn(friend);
     }
 
     @Test
     public void validLoginTest() {
-        String name = "alex";
+        String name = "user";
         String password = "password";
         UserDto found = userService.loginUser(name, password);
         assertEquals(found.getName(), name);
@@ -57,8 +60,8 @@ public class UserServiceTest {
 
     @Test
     public void userRegisterTest() {
-        User user = new User(1L, "name", "password", 0);
+        User test = new User(1L, "name", "password", 0);
-        UserDto registered = userService.registerUser(user.getName(), user.getPassword());
+        UserDto registered = userService.registerUser(test.getName(), test.getPassword());
         assertEquals(registered.getName(), "name");
     }
 
@@ -69,7 +72,6 @@ public class UserServiceTest {
 
     @Test
     public void invalidLoginTest() {
-        User user = null;
         assertThrows(ApplicationException.class, () -> {
             userService.loginUser(null, null);
         });
@@ -77,20 +79,10 @@ public class UserServiceTest {
 
     @Test
     public void addFriendTest() {
-        User user = new User(1l,"Merel", "password", 0);
+        userService.addFriend(1L,"user", "friend");
-        User friend = new User(2l, "Ellis", "pass", 0);
+        List<User> test = new ArrayList<User>();
-        userService.registerUser("Merel", "password");
+        test.add(userRepository.findByName("friend"));
-        userService.registerUser("Ellis", "pass");
+        assertEquals(userRepository.findByName("user").getFriends(), test);
-        assertEquals(user.getFriends(), new ArrayList<User>());
-
-        //STILL TELLS ME NULL
-        System.out.println(userRepository.findByName("Merel"));
-
-
-//        userService.addFriend(1l,"Merel", "Ellis");
-//        List<User> ellis = new ArrayList<User>();
-//        ((ArrayList) ellis).add(friend);
-//        assertEquals(user.getFriends(), ellis);
     }
 
 }