From 727f5447dc8dcd6077a9034e50511bbc5b025293 Mon Sep 17 00:00:00 2001 From: ShyamKrishna33 Date: Thu, 14 Mar 2024 23:08:53 +0800 Subject: [PATCH 1/8] Create JUnit Test Case for parseTransaction --- src/test/java/parser/ParserTest.java | 21 +++++++++++++++++++++ src/test/java/seedu/duke/DukeTest.java | 12 ------------ 2 files changed, 21 insertions(+), 12 deletions(-) create mode 100644 src/test/java/parser/ParserTest.java delete mode 100644 src/test/java/seedu/duke/DukeTest.java diff --git a/src/test/java/parser/ParserTest.java b/src/test/java/parser/ParserTest.java new file mode 100644 index 0000000000..e12d52c543 --- /dev/null +++ b/src/test/java/parser/ParserTest.java @@ -0,0 +1,21 @@ +package parser; +import org.junit.jupiter.api.Test; +import transactions.Transaction; + +import java.time.LocalDate; +import java.time.format.DateTimeFormatter; + +import static org.junit.jupiter.api.Assertions.*; + +public class ParserTest { + + @Test + public void testParseTransaction() { + Parser parser = new Parser(); + Transaction transaction = parser.parseTransaction("add /n/Shopping /$/50 /d/14-03-2024 /c/Personal"); + assertEquals("Shopping", transaction.getDescription()); + assertEquals(50.0f, transaction.getAmount(), 0.001); + assertEquals(LocalDate.parse("14-03-2024", DateTimeFormatter.ofPattern("dd-MM-yyyy")), transaction.getDate()); + assertEquals("Personal", transaction.getCategory()); + } +} \ No newline at end of file diff --git a/src/test/java/seedu/duke/DukeTest.java b/src/test/java/seedu/duke/DukeTest.java deleted file mode 100644 index 2dda5fd651..0000000000 --- a/src/test/java/seedu/duke/DukeTest.java +++ /dev/null @@ -1,12 +0,0 @@ -package seedu.duke; - -import static org.junit.jupiter.api.Assertions.assertTrue; - -import org.junit.jupiter.api.Test; - -class DukeTest { - @Test - public void sampleTest() { - assertTrue(true); - } -} From 83951294da9038215201ee95b9bd01397e4f7eef Mon Sep 17 00:00:00 2001 From: ShyamKrishna33 Date: Thu, 14 Mar 2024 23:14:08 +0800 Subject: [PATCH 2/8] fix: Replace * import to specific package --- src/test/java/parser/ParserTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/java/parser/ParserTest.java b/src/test/java/parser/ParserTest.java index e12d52c543..9a3725584e 100644 --- a/src/test/java/parser/ParserTest.java +++ b/src/test/java/parser/ParserTest.java @@ -5,7 +5,7 @@ import java.time.LocalDate; import java.time.format.DateTimeFormatter; -import static org.junit.jupiter.api.Assertions.*; +import static org.junit.jupiter.api.Assertions.assertEquals; public class ParserTest { From 77a1eeb325d80773070d7e67a9fc238bc6521c57 Mon Sep 17 00:00:00 2001 From: ShyamKrishna33 Date: Thu, 14 Mar 2024 23:17:38 +0800 Subject: [PATCH 3/8] fix: Add newline at EndOfFile --- src/test/java/parser/ParserTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/java/parser/ParserTest.java b/src/test/java/parser/ParserTest.java index 9a3725584e..9d4a300339 100644 --- a/src/test/java/parser/ParserTest.java +++ b/src/test/java/parser/ParserTest.java @@ -18,4 +18,4 @@ public void testParseTransaction() { assertEquals(LocalDate.parse("14-03-2024", DateTimeFormatter.ofPattern("dd-MM-yyyy")), transaction.getDate()); assertEquals("Personal", transaction.getCategory()); } -} \ No newline at end of file +} From ac9abbb18638779632bb1bac955fc689b755bc17 Mon Sep 17 00:00:00 2001 From: Isaac Eng <> Date: Fri, 15 Mar 2024 00:02:04 +0800 Subject: [PATCH 4/8] Create class TransactionListTest --- .../java/financemanager/TransactionListTest.java | 4 ++++ src/test/java/seedu/duke/DukeTest.java | 12 ------------ 2 files changed, 4 insertions(+), 12 deletions(-) create mode 100644 src/test/java/financemanager/TransactionListTest.java delete mode 100644 src/test/java/seedu/duke/DukeTest.java diff --git a/src/test/java/financemanager/TransactionListTest.java b/src/test/java/financemanager/TransactionListTest.java new file mode 100644 index 0000000000..9399dad38c --- /dev/null +++ b/src/test/java/financemanager/TransactionListTest.java @@ -0,0 +1,4 @@ +package financemanager; + +public class TransactionListTest { +} diff --git a/src/test/java/seedu/duke/DukeTest.java b/src/test/java/seedu/duke/DukeTest.java deleted file mode 100644 index 2dda5fd651..0000000000 --- a/src/test/java/seedu/duke/DukeTest.java +++ /dev/null @@ -1,12 +0,0 @@ -package seedu.duke; - -import static org.junit.jupiter.api.Assertions.assertTrue; - -import org.junit.jupiter.api.Test; - -class DukeTest { - @Test - public void sampleTest() { - assertTrue(true); - } -} From 08c74a0a9c2b3bc342a4a1dada612417695df5a5 Mon Sep 17 00:00:00 2001 From: Isaac Eng <> Date: Fri, 15 Mar 2024 00:03:20 +0800 Subject: [PATCH 5/8] Add JUnit test for TransactionList class --- .../financemanager/TransactionListTest.java | 33 ++++++++++++++++++- 1 file changed, 32 insertions(+), 1 deletion(-) diff --git a/src/test/java/financemanager/TransactionListTest.java b/src/test/java/financemanager/TransactionListTest.java index 9399dad38c..32accb1873 100644 --- a/src/test/java/financemanager/TransactionListTest.java +++ b/src/test/java/financemanager/TransactionListTest.java @@ -1,4 +1,35 @@ package financemanager; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.BeforeEach; +import transactions.Transaction; + +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertThrows; + public class TransactionListTest { -} + + private TransactionList transactionList; + + @BeforeEach + public void setUp() { + transactionList = new TransactionList(); + } + + @Test + public void getTransactions_initiallyEmpty() { + assertEquals(0, transactionList.getTransactions().size()); + } + + @Test + public void processTransaction_addsTransaction() { + Transaction testTransaction = new Transaction("Test", 200,"Personal", "14-03-2024"); + transactionList.processTransaction("add /n/Test /$/200 /d/14-03-2024 /c/Personal"); + + assertEquals(1, transactionList.getTransactions().size()); + assertEquals(testTransaction.getDescription(), transactionList.getTransactions().get(0).getDescription()); + assertEquals(testTransaction.getAmount(), transactionList.getTransactions().get(0).getAmount()); + assertEquals(testTransaction.getCategory(), transactionList.getTransactions().get(0).getCategory()); + assertEquals(testTransaction.getDate(), transactionList.getTransactions().get(0).getDate()); + } +} \ No newline at end of file From 624e2d8a3c2dfb1154a8573a68fcc80f7d242e9b Mon Sep 17 00:00:00 2001 From: Vavinan Date: Fri, 15 Mar 2024 00:15:47 +0800 Subject: [PATCH 6/8] Add: JUnit test for Transaction class --- src/test/java/seedu/duke/DukeTest.java | 12 ----- .../java/transactions/TransactionTest.java | 52 +++++++++++++++++++ 2 files changed, 52 insertions(+), 12 deletions(-) delete mode 100644 src/test/java/seedu/duke/DukeTest.java create mode 100644 src/test/java/transactions/TransactionTest.java diff --git a/src/test/java/seedu/duke/DukeTest.java b/src/test/java/seedu/duke/DukeTest.java deleted file mode 100644 index 2dda5fd651..0000000000 --- a/src/test/java/seedu/duke/DukeTest.java +++ /dev/null @@ -1,12 +0,0 @@ -package seedu.duke; - -import static org.junit.jupiter.api.Assertions.assertTrue; - -import org.junit.jupiter.api.Test; - -class DukeTest { - @Test - public void sampleTest() { - assertTrue(true); - } -} diff --git a/src/test/java/transactions/TransactionTest.java b/src/test/java/transactions/TransactionTest.java new file mode 100644 index 0000000000..8371e4bb9a --- /dev/null +++ b/src/test/java/transactions/TransactionTest.java @@ -0,0 +1,52 @@ +package transactions; + +import org.junit.jupiter.api.Test; + +import java.time.LocalDate; +import java.time.format.DateTimeFormatter; + +import static org.junit.jupiter.api.Assertions.assertEquals; + +public class TransactionTest { + @Test + public void testTransactionConstructor() { + Transaction transaction = new Transaction("Groceries", 50.0f, "Food", "14-03-2024"); + assertEquals("Groceries", transaction.getDescription()); + assertEquals(50.0f, transaction.getAmount(), 0.001); + assertEquals("Food", transaction.getCategory()); + LocalDate date = LocalDate.parse("14-03-2024", DateTimeFormatter.ofPattern("dd-MM-yyyy")); + assertEquals(date, transaction.getDate()); + } + + @Test + public void testGetDescription() { + Transaction transaction = new Transaction("Groceries", 50.0f, "Food", "14-03-2024"); + assertEquals("Groceries", transaction.getDescription()); + } + + @Test + public void testGetAmount() { + Transaction transaction = new Transaction("Groceries", 50.0f, "Food", "14-03-2024"); + assertEquals(50.0f, transaction.getAmount(), 0.001); + } + + @Test + public void testGetCategory() { + Transaction transaction = new Transaction("Groceries", 50.0f, "Food", "14-03-2024"); + assertEquals("Food", transaction.getCategory()); + } + + @Test + public void testGetDate() { + Transaction transaction = new Transaction("Groceries", 50.0f, "Food", "14-03-2024"); + LocalDate date = LocalDate.parse("14-03-2024", DateTimeFormatter.ofPattern("dd-MM-yyyy")); + assertEquals(date, transaction.getDate()); + } + + @Test + public void testToString() { + Transaction transaction = new Transaction("Groceries", 50.0f, "Food", "14-03-2024"); + String expected = " Description: Groceries | Date: 2024-03-14 | Amount: 50.0 | Category: Food"; + assertEquals(expected, transaction.toString()); + } +} From a18c37c2bec1836eb2ec7ad89050d21540f02573 Mon Sep 17 00:00:00 2001 From: Isaac Eng <> Date: Fri, 15 Mar 2024 00:21:16 +0800 Subject: [PATCH 7/8] Remove unused import --- src/test/java/financemanager/TransactionListTest.java | 1 - 1 file changed, 1 deletion(-) diff --git a/src/test/java/financemanager/TransactionListTest.java b/src/test/java/financemanager/TransactionListTest.java index 32accb1873..986630227a 100644 --- a/src/test/java/financemanager/TransactionListTest.java +++ b/src/test/java/financemanager/TransactionListTest.java @@ -5,7 +5,6 @@ import transactions.Transaction; import static org.junit.jupiter.api.Assertions.assertEquals; -import static org.junit.jupiter.api.Assertions.assertThrows; public class TransactionListTest { From 7ae090f34a7b99b17fda676c6eafb76ae0de7fa0 Mon Sep 17 00:00:00 2001 From: Isaac Eng <> Date: Fri, 15 Mar 2024 00:23:24 +0800 Subject: [PATCH 8/8] Add newline at end of file --- src/test/java/financemanager/TransactionListTest.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/test/java/financemanager/TransactionListTest.java b/src/test/java/financemanager/TransactionListTest.java index 986630227a..6efb5572f2 100644 --- a/src/test/java/financemanager/TransactionListTest.java +++ b/src/test/java/financemanager/TransactionListTest.java @@ -31,4 +31,5 @@ public void processTransaction_addsTransaction() { assertEquals(testTransaction.getCategory(), transactionList.getTransactions().get(0).getCategory()); assertEquals(testTransaction.getDate(), transactionList.getTransactions().get(0).getDate()); } -} \ No newline at end of file + +}