Fix: Renamed method in Portfolio

Team-40-IDATT2003 · May 25, 2026 · d6d29ed · d6d29ed
1 parent 9310cae
commit d6d29ed
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/src/main/java/edu/ntnu/idi/idatt2003/g40/mappe/model/Portfolio.java b/src/main/java/edu/ntnu/idi/idatt2003/g40/mappe/model/Portfolio.java
@@ -178,7 +178,7 @@ public BigDecimal getNetWorth() {
    * @return        BigDecimal representing total quantity of all
    *                shares of this symbol.
    * */
-  public BigDecimal getTotalSharesBySymbol(final String symbol) {
+  public BigDecimal getTotalShareQuantityBySymbol(final String symbol) {
     if (symbol == null) {
       return BigDecimal.ZERO;
     }

diff --git a/...ain/java/edu/ntnu/idi/idatt2003/g40/mappe/view/widgets/dashboard/DashBoardController.java b/...ain/java/edu/ntnu/idi/idatt2003/g40/mappe/view/widgets/dashboard/DashBoardController.java
@@ -116,7 +116,7 @@ private void populateStockList(final String filter) {
 
         getViewElement().setOnStockAction(stockBtn, s, (Stock stock) -> {
           BigDecimal amountOfSharesOwned = player.getPortfolio()
-                  .getTotalSharesBySymbol(s.getSymbol());
+                  .getTotalShareQuantityBySymbol(s.getSymbol());
           handleStockSelection(stock, amountOfSharesOwned.floatValue());
         });
       }

diff --git a/src/test/java/edu/ntnu/idi/idatt2003/g40/mappe/model/PortfolioTest.java b/src/test/java/edu/ntnu/idi/idatt2003/g40/mappe/model/PortfolioTest.java
@@ -131,21 +131,21 @@ void containsThrowsExceptionOnIllegalArgument() {
   void getTotalSharesBySymbolReturnsCorrectValues() {
     assertEquals(0,
             BigDecimal.ZERO.compareTo(
-                    testPortfolio.getTotalSharesBySymbol("AAPL")
+                    testPortfolio.getTotalShareQuantityBySymbol("AAPL")
             )
     );
 
     assertEquals(0,
             BigDecimal.ZERO.compareTo(
-                    testPortfolio.getTotalSharesBySymbol(null)
+                    testPortfolio.getTotalShareQuantityBySymbol(null)
             )
     );
 
     testPortfolio.addShare(testShare);
 
     assertEquals(0,
             new BigDecimal("2.0").compareTo(
-                    testPortfolio.getTotalSharesBySymbol("AAPL")
+                    testPortfolio.getTotalShareQuantityBySymbol("AAPL")
             )
     );
   }