Skip to content

Commit

Permalink
Test: Adding test for price change. Small bugfix
Browse files Browse the repository at this point in the history
  • Loading branch information
martin committed Apr 20, 2026
1 parent 31965c8 commit 8d15ad4
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/main/java/millions/model/Stock.java
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ public BigDecimal getLatestPriceChange() {
}

BigDecimal currentPrice = this.prices.getLast();
BigDecimal lastPrice = this.prices.get(this.prices.size() - 1);
BigDecimal lastPrice = this.prices.get(this.prices.size() - 2);

return currentPrice.subtract(lastPrice);
}
Expand Down
11 changes: 10 additions & 1 deletion src/test/java/millions/StockTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
import java.math.BigDecimal;
import java.util.ArrayList;
import java.util.Arrays;

import java.util.List;
import millions.model.Stock;
import org.junit.jupiter.api.Test;

Expand All @@ -27,6 +27,15 @@ public void settersAndGetters() {
assertEquals("Nvadia", stock.getCompany());
}

@Test
public void testGetPriceChange() {
ArrayList prices = new ArrayList<>(List.of(BigDecimal.valueOf(100), BigDecimal.valueOf(125)));
Stock stock = new Stock("AAPL", "Apple", prices);
assertEquals(BigDecimal.valueOf(25), stock.getLatestPriceChange());
stock.addNewSalesPrice(BigDecimal.valueOf(155));
assertEquals(BigDecimal.valueOf(30), stock.getLatestPriceChange());
}

@Test
public void testNullsAndInvalid() {

Expand Down

0 comments on commit 8d15ad4

Please sign in to comment.