Update Test Cases for Deck Related Commands and Home Command.

src/main/java/seedu/flashnotes/logic/commands/AddDeckCommand.java

@@ -44,4 +44,11 @@ public CommandResult execute(Model model) throws CommandException {
 
         return new CommandResult(String.format(MESSAGE_SUCCESS, toAdd.getDeckName()));
     }
+
+    @Override
+    public boolean equals(Object other) {
+        return other == this // short circuit if same object
+                || (other instanceof AddDeckCommand // instanceof handles nulls
+                && toAdd.equals(((AddDeckCommand) other).toAdd));
+    }
 }

src/main/java/seedu/flashnotes/logic/commands/HomeCommand.java

@@ -22,6 +22,6 @@ public CommandResult execute(Model model) {
 
     @Override
     public boolean equals(Object other) {
-        return other == this; // short circuit if same object
+        return other == this || (other instanceof HomeCommand);
     }
 }

src/main/java/seedu/flashnotes/model/FlashNotes.java

@@ -187,8 +187,8 @@ public void addDeck(Deck card) {
      */
     public void setDeck(Deck target, Deck editedDeck) {
         requireNonNull(editedDeck);
-
         decks.setDeck(target, editedDeck);
+        //TODO update the flashcards if there are existing flashcardsun
     }
 
     /**
@@ -260,8 +260,6 @@ public ObservableList<Flashcard> getFlashcardList() {
 
     @Override
     public ObservableList<Deck> getDeckList() {
-        //todo read the tags and update
-        //todo change when we have decklist implementation up
         return decks.asUnmodifiableObservableList();
     }
 

src/main/java/seedu/flashnotes/model/ModelManager.java

@@ -115,7 +115,6 @@ public boolean hasDeck(Deck deck) {
 
     @Override
     public void deleteDeck(Deck target) {
-        //todo delete deck - PX
         flashNotes.removeDeck(target);
         updateFilteredDeckList(PREDICATE_SHOW_ALL_DECKS);
     }
@@ -130,7 +129,6 @@ public void addDeck(Deck deck) {
     @Override
     public void setDeck(Deck target, Deck editedDeck) {
         flashNotes.setDeck(target, editedDeck);
-
     }
 
     @Override

src/main/java/seedu/flashnotes/model/deck/Deck.java

@@ -3,7 +3,9 @@
 
 import java.util.Objects;
 
-
+/**
+ * Deck is the tag which is given to every card.
+ */
 public class Deck {
     public static final String MESSAGE_CONSTRAINTS =
             "DeckName should not be blank";
@@ -32,10 +34,12 @@ public String getResultStatistics() {
     }
 
     public void setDeckName(String deckName) {
+        requireNonNull(deckName);
         this.deckName = deckName;
     }
 
     public void setResultStatistics(String resultStatistics) {
+        requireNonNull(resultStatistics);
         this.resultStatistics = resultStatistics;
     }
 
@@ -56,7 +60,6 @@ public boolean isSameDeck(Deck otherDeck) {
 
         return otherDeck != null
                 && otherDeck.getDeckName().equals(getDeckName());
-        //&& (otherDeck.getResultStatistics().equals(getResultStatistics()));
     }
     @Override
     public boolean equals(Object o) {
@@ -67,11 +70,11 @@ public boolean equals(Object o) {
             return false;
         }
         Deck deck = (Deck) o;
-        return Objects.equals(deckName, deck.deckName); //&& Objects.equals(resultStatistics, deck.resultStatistics);
+        return Objects.equals(deckName, deck.deckName);
     }
 
     @Override
     public int hashCode() {
-        return Objects.hash(deckName); //, resultStatistics);
+        return Objects.hash(deckName);
     }
 }

src/test/java/seedu/flashnotes/logic/commands/AddCommandTest.java

@@ -146,17 +146,17 @@ public boolean hasDeck(Deck deck) {
 
         @Override
         public void deleteDeck(Deck target) {
-
+            throw new AssertionError("This method should not be called.");
         }
 
         @Override
         public void addDeck(Deck deck) {
-
+            throw new AssertionError("This method should not be called.");
         }
 
         @Override
         public void setDeck(Deck target, Deck editedDeck) {
-
+            throw new AssertionError("This method should not be called.");
         }
 
         @Override
@@ -166,17 +166,17 @@ public boolean getIsInDeck() {
 
         @Override
         public void setIsInDeckTrue() {
-
+            throw new AssertionError("This method should not be called.");
         }
 
         @Override
         public void setIsInDeckFalse() {
-
+            throw new AssertionError("This method should not be called.");
         }
 
         @Override
         public void setCurrentDeckName(String deckName) {
-
+            throw new AssertionError("This method should not be called.");
         }
 
         @Override
@@ -191,7 +191,7 @@ public ObservableList<Deck> getFilteredDeckList() {
 
         @Override
         public void updateFilteredDeckList(Predicate<Deck> predicate) {
-
+            throw new AssertionError("This method should not be called.");
         }
 
         @Override
@@ -276,6 +276,10 @@ public void addFlashcard(Flashcard flashcard) {
             flashcardsAdded.add(flashcard);
         }
 
+        @Override
+        public void addDeck(Deck deck) {
+            requireNonNull(deck);
+        }
         @Override
         public ReadOnlyFlashNotes getFlashNotes() {
             return new FlashNotes();