Fixed bugs.

FabriDeCastelli · Aug 9, 2023 · b79db39 · b79db39
1 parent 26bcf32
commit b79db39
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/main/java/server/controller/RequestHandler.java b/src/main/java/server/controller/RequestHandler.java
@@ -113,7 +113,7 @@ public void register(@NotNull User user) {
             serverResponse = new ServerResponse(-1, "Registration failed.");
         }
 
-        if (sendResponse(serverResponse)) {
+        if (!sendResponse(serverResponse)) {
             throw new IllegalStateException("Cannot send response to client.");
         }
     }

diff --git a/src/test/java/service/AuthenticationServiceTests.java b/src/test/java/service/AuthenticationServiceTests.java
@@ -3,6 +3,7 @@
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
+import client.service.PasswordHashingService;
 import java.util.Optional;
 import model.User;
 import org.junit.jupiter.api.BeforeAll;
@@ -75,7 +76,7 @@ public void testAddAlreadyRegisteredUser() {
             assertThrows(
                     IllegalArgumentException.class,
                     () -> authenticationService.add(
-                            new User("fabry", "fabry")));
+                            new User("f", PasswordHashingService.getInstance().hashPassword("f"))));
         }
     }