From b617333ee94fc60c8365d20318fcd95c49eac4db Mon Sep 17 00:00:00 2001
From: paulgries <pgries@cs.toronto.edu>
Date: Sun, 15 Oct 2023 13:11:46 -0400
Subject: [PATCH 1/2] Signup and CommonUser unit tests, exception demo test

---
 CACoding.iml                                  | 17 +++++
 test/entity/CommonUserTest.java               | 27 ++++++++
 test/entity/ExceptionDemoTest.java            | 28 ++++++++
 .../use_case/signup/SignupInteractorTest.java | 65 +++++++++++++++++++
 4 files changed, 137 insertions(+)
 create mode 100644 test/entity/CommonUserTest.java
 create mode 100644 test/entity/ExceptionDemoTest.java
 create mode 100644 test/use_case/signup/SignupInteractorTest.java

diff --git a/CACoding.iml b/CACoding.iml
index 69623856..b3e930f2 100644
--- a/CACoding.iml
+++ b/CACoding.iml
@@ -4,9 +4,26 @@
     <exclude-output />
     <content url="file://$MODULE_DIR$">
       <sourceFolder url="file://$MODULE_DIR$/src" isTestSource="false" />
+      <sourceFolder url="file://$MODULE_DIR$/test" isTestSource="true" />
     </content>
     <orderEntry type="inheritedJdk" />
     <orderEntry type="sourceFolder" forTests="false" />
+    <orderEntry type="module-library" scope="TEST">
+      <library name="JUnit5.8.1">
+        <CLASSES>
+          <root url="jar://$MAVEN_REPOSITORY$/org/junit/jupiter/junit-jupiter/5.8.1/junit-jupiter-5.8.1.jar!/" />
+          <root url="jar://$MAVEN_REPOSITORY$/org/junit/jupiter/junit-jupiter-api/5.8.1/junit-jupiter-api-5.8.1.jar!/" />
+          <root url="jar://$MAVEN_REPOSITORY$/org/opentest4j/opentest4j/1.2.0/opentest4j-1.2.0.jar!/" />
+          <root url="jar://$MAVEN_REPOSITORY$/org/junit/platform/junit-platform-commons/1.8.1/junit-platform-commons-1.8.1.jar!/" />
+          <root url="jar://$MAVEN_REPOSITORY$/org/apiguardian/apiguardian-api/1.1.2/apiguardian-api-1.1.2.jar!/" />
+          <root url="jar://$MAVEN_REPOSITORY$/org/junit/jupiter/junit-jupiter-params/5.8.1/junit-jupiter-params-5.8.1.jar!/" />
+          <root url="jar://$MAVEN_REPOSITORY$/org/junit/jupiter/junit-jupiter-engine/5.8.1/junit-jupiter-engine-5.8.1.jar!/" />
+          <root url="jar://$MAVEN_REPOSITORY$/org/junit/platform/junit-platform-engine/1.8.1/junit-platform-engine-1.8.1.jar!/" />
+        </CLASSES>
+        <JAVADOC />
+        <SOURCES />
+      </library>
+    </orderEntry>
   </component>
   <component name="SonarLintModuleSettings">
     <option name="uniqueId" value="ce2f91cc-4cf6-4286-a14e-4c3d4f923b96" />
diff --git a/test/entity/CommonUserTest.java b/test/entity/CommonUserTest.java
new file mode 100644
index 00000000..763a3496
--- /dev/null
+++ b/test/entity/CommonUserTest.java
@@ -0,0 +1,27 @@
+package entity;
+
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import java.time.LocalDateTime;
+import static org.junit.jupiter.api.Assertions.*;
+
+class CommonUserTest {
+
+    private CommonUser user;
+
+    @BeforeEach
+    void init() {
+        user = new CommonUser("Paul", "password", LocalDateTime.now());
+    }
+
+    @Test
+    void getNameTest() {
+        assertEquals("Paul", user.getName());
+    }
+
+    @Test
+    void getPasswordTest() {
+        assertEquals("password", user.getPassword());
+    }
+
+}
\ No newline at end of file
diff --git a/test/entity/ExceptionDemoTest.java b/test/entity/ExceptionDemoTest.java
new file mode 100644
index 00000000..ef17367d
--- /dev/null
+++ b/test/entity/ExceptionDemoTest.java
@@ -0,0 +1,28 @@
+package entity;
+
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+
+import java.time.LocalDateTime;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+
+class Calculator {
+    public void divide(int i, int j) {
+        int result = i / j;
+    }
+}
+class ExceptionDemoTest {
+    @Test
+    void exceptionTest() {
+        Calculator calculator = new Calculator();
+        // Assert that the calculator.divide call throws an exception
+        Exception exception = assertThrows(
+                ArithmeticException.class,
+                // This is an anonymous method that gets called by the assertThrows method
+                () -> calculator.divide(1, 0)
+        );
+        assertEquals("/ by zero", exception.getMessage());
+    }
+}
\ No newline at end of file
diff --git a/test/use_case/signup/SignupInteractorTest.java b/test/use_case/signup/SignupInteractorTest.java
new file mode 100644
index 00000000..4b3ef640
--- /dev/null
+++ b/test/use_case/signup/SignupInteractorTest.java
@@ -0,0 +1,65 @@
+package use_case.signup;
+
+import data_access.InMemoryUserDataAccessObject;
+import entity.CommonUserFactory;
+import entity.User;
+import entity.UserFactory;
+import interface_adapter.signup.SignupPresenter;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+
+import java.time.LocalDateTime;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+class SignupInteractorTest {
+
+    @Test
+    void successTest() {
+        SignupInputData inputData = new SignupInputData("Paul", "password", "password");
+        SignupUserDataAccessInterface userRepository = new InMemoryUserDataAccessObject();
+
+        // This creates a successPresenter that tests whether the test case is as we expect.
+        SignupOutputBoundary successPresenter = new SignupOutputBoundary() {
+            @Override
+            public void prepareSuccessView(SignupOutputData user) {
+                // 2 things to check: the output data is correct, and the user has been created in the DAO.
+                assertEquals("Paul", user.getUsername());
+                assertNotNull(user.getCreationTime()); // any creation time is fine.
+                assertTrue(userRepository.existsByName("Paul"));
+            }
+
+            @Override
+            public void prepareFailView(String error) {
+                fail("Use case failure is unexpected.");
+            }
+        };
+
+        SignupInputBoundary interactor = new SignupInteractor(userRepository, successPresenter, new CommonUserFactory());
+        interactor.execute(inputData);
+    }
+
+    void failureTest() {
+        SignupInputData inputData = new SignupInputData("Paul", "password", "wrong");
+        SignupUserDataAccessInterface userRepository = new InMemoryUserDataAccessObject();
+
+        // This creates a presenter that tests whether the test case is as we expect.
+        SignupOutputBoundary failurePresenter = new SignupOutputBoundary() {
+            @Override
+            public void prepareSuccessView(SignupOutputData user) {
+                // 2 things to check: the output data is correct, and the user has been created in the DAO.
+                fail("Use case success is unexpected.");
+            }
+
+            @Override
+            public void prepareFailView(String error) {
+                assertEquals("Passwords don't match.", error);
+            }
+        };
+
+        SignupInputBoundary interactor = new SignupInteractor(userRepository, failurePresenter, new CommonUserFactory());
+        interactor.execute(inputData);
+    }
+
+}
\ No newline at end of file

From 5833c8d08630994ce5fe595ec129d6e7dd8ca341 Mon Sep 17 00:00:00 2001
From: paulgries <pgries@cs.toronto.edu>
Date: Sun, 15 Oct 2023 13:16:50 -0400
Subject: [PATCH 2/2] Added test for user already exists

---
 .../use_case/signup/SignupInteractorTest.java | 33 ++++++++++++++++---
 1 file changed, 29 insertions(+), 4 deletions(-)

diff --git a/test/use_case/signup/SignupInteractorTest.java b/test/use_case/signup/SignupInteractorTest.java
index 4b3ef640..d2c2b36e 100644
--- a/test/use_case/signup/SignupInteractorTest.java
+++ b/test/use_case/signup/SignupInteractorTest.java
@@ -4,9 +4,6 @@
 import entity.CommonUserFactory;
 import entity.User;
 import entity.UserFactory;
-import interface_adapter.signup.SignupPresenter;
-import org.junit.jupiter.api.AfterEach;
-import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
 import java.time.LocalDateTime;
@@ -40,7 +37,8 @@ public void prepareFailView(String error) {
         interactor.execute(inputData);
     }
 
-    void failureTest() {
+    @Test
+    void failurePasswordMismatchTest() {
         SignupInputData inputData = new SignupInputData("Paul", "password", "wrong");
         SignupUserDataAccessInterface userRepository = new InMemoryUserDataAccessObject();
 
@@ -62,4 +60,31 @@ public void prepareFailView(String error) {
         interactor.execute(inputData);
     }
 
+    @Test
+    void failureUserExistsTest() {
+        SignupInputData inputData = new SignupInputData("Paul", "password", "wrong");
+        SignupUserDataAccessInterface userRepository = new InMemoryUserDataAccessObject();
+
+        // Add Paul to the repo so that when we check later they already exist
+        UserFactory factory = new CommonUserFactory();
+        User user = factory.create("Paul", "pwd",  LocalDateTime.now());
+        userRepository.save(user);
+
+        // This creates a presenter that tests whether the test case is as we expect.
+        SignupOutputBoundary failurePresenter = new SignupOutputBoundary() {
+            @Override
+            public void prepareSuccessView(SignupOutputData user) {
+                // 2 things to check: the output data is correct, and the user has been created in the DAO.
+                fail("Use case success is unexpected.");
+            }
+
+            @Override
+            public void prepareFailView(String error) {
+                assertEquals("User already exists.", error);
+            }
+        };
+
+        SignupInputBoundary interactor = new SignupInteractor(userRepository, failurePresenter, new CommonUserFactory());
+        interactor.execute(inputData);
+    }
 }
\ No newline at end of file