From 8554f63a10a7cf6bbbeaae7b8a888eb741efc6e0 Mon Sep 17 00:00:00 2001
From: Elisabeth Berg <elisab3@stud.ntnu.no>
Date: Sun, 24 May 2026 22:33:18 +0200
Subject: [PATCH] Added more tests to PurchaseTest

---
 src/test/java/PurchaseTest.java | 93 +++++++++++++++++++++++++++------
 1 file changed, 77 insertions(+), 16 deletions(-)

diff --git a/src/test/java/PurchaseTest.java b/src/test/java/PurchaseTest.java
index 2f87b39..d41e71a 100644
--- a/src/test/java/PurchaseTest.java
+++ b/src/test/java/PurchaseTest.java
@@ -1,4 +1,4 @@
-import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.*;
 
 import java.math.BigDecimal;
 
@@ -11,32 +11,93 @@
 import Model.Stock;
 
 public class PurchaseTest {
-    Stock stock;
-    Share share;
-    Purchase purchase;
+
+    private Stock stock;
+    private Share share;
+    private Purchase purchase;
+    private Player player;
 
     @BeforeEach
     void setUp() {
-        this.stock = new Stock("symbol", "company", new BigDecimal(100));
-        this.share = new Share(this.stock, new BigDecimal(20), new BigDecimal(50));
-        this.purchase = new Purchase(this.share, 18);
+        stock = new Stock("AAPL", "Apple", new BigDecimal("100"));
+        share = new Share(stock, new BigDecimal("20"), new BigDecimal("50"));
+        purchase = new Purchase(share, 1);
+        player = new Player("Jane", new BigDecimal("500000"));
     }
 
+    // ---- Positive tests ----
+
     @Test
-    void commit_validPurchase_updatesPlayer() {
+    void testCommitDeductsMoneyFromPlayer() {
+        BigDecimal before = player.getMoney();
+        purchase.commit(player);
+        assertTrue(player.getMoney().compareTo(before) < 0);
+    }
 
-        // Arrange
-        Player player = new Player("Jane", new BigDecimal(500000));
-        BigDecimal startingMoney = player.getMoney();
+    @Test
+    void testCommitAddsShareToPortfolio() {
+        purchase.commit(player);
+        assertTrue(player.getPortfolio().contains(share));
+    }
+
+    @Test
+    void testCommitRecordedInArchive() {
+        purchase.commit(player);
+        assertTrue(player.getTransactionArchive().getAllTransactions().contains(purchase));
+    }
 
-        // Act
-        this.purchase.commit(player);
+    @Test
+    void testCommitSetsCommittedFlag() {
+        purchase.commit(player);
+        assertTrue(purchase.isCommitted());
+    }
 
-        // Assert
-        assertEquals(1, startingMoney.subtract(player.getMoney()).signum());
+    @Test
+    void testGetShareReturnsCorrectShare() {
+        assertEquals(share, purchase.getShare());
+    }
 
+    @Test
+    void testGetWeekReturnsCorrectWeek() {
+        assertEquals(1, purchase.getWeek());
     }
 
+    // ---- Negative tests ----
 
-    
+    @Test
+    void testCommitNullPlayerThrows() {
+        assertThrows(IllegalArgumentException.class, () ->
+            purchase.commit(null)
+        );
+    }
+
+    @Test
+    void testCommitInsufficientFundsThrows() {
+        Player poorPlayer = new Player("Broke", new BigDecimal("1"));
+        assertThrows(IllegalStateException.class, () ->
+            purchase.commit(poorPlayer)
+        );
+    }
+
+    @Test
+    void testCommitAlreadyCommittedThrows() {
+        purchase.commit(player);
+        assertThrows(IllegalStateException.class, () ->
+            purchase.commit(player)
+        );
+    }
+
+    @Test
+    void testConstructorWeekZeroThrows() {
+        assertThrows(IllegalArgumentException.class, () ->
+            new Purchase(share, 0)
+        );
+    }
+
+    @Test
+    void testConstructorNegativeWeekThrows() {
+        assertThrows(IllegalArgumentException.class, () ->
+            new Purchase(share, -5)
+        );
+    }
 }