Skip to content
This repository has been archived by the owner on Apr 6, 2024. It is now read-only.

Commit

Permalink
Fix test
Browse files Browse the repository at this point in the history
  • Loading branch information
conradsoon committed Nov 6, 2023
1 parent a4308c7 commit 22cf1e8
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 7 deletions.
5 changes: 0 additions & 5 deletions src/main/java/seedu/address/ui/NotesWindow.java
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
package seedu.address.ui;

import java.util.List;
import java.util.logging.Logger;

import javafx.collections.FXCollections;
Expand Down Expand Up @@ -69,10 +68,6 @@ public void hide() {
getRoot().hide();
}

public void focus() {
getRoot().requestFocus();
}

private void populateListView(ObservableList<Note> notes) {
ObservableList<Note> notesObservableList = FXCollections.observableArrayList(notes);
notesListView.setItems(notesObservableList);
Expand Down
11 changes: 9 additions & 2 deletions src/test/java/seedu/address/ui/NotesWindowTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,10 @@
import org.testfx.framework.junit5.ApplicationTest;

import javafx.application.Platform;
import javafx.collections.ObservableList;
import javafx.scene.input.KeyCode;
import javafx.stage.Stage;
import seedu.address.model.person.Note;
import seedu.address.model.person.Person;

public class NotesWindowTest extends ApplicationTest {
Expand Down Expand Up @@ -43,9 +45,14 @@ public void setUp() {
@Test
public void displayNotes_correctNumberOfItemsAndContent_displaysExpectedNotes() {
interact(() -> {
List<String> expectedNotes = Arrays.asList("Likes to swim", "Likes to run", "Is a chad");
List<Note> expectedNotes = Arrays.asList(
new Note("Likes to swim"),
new Note("Likes to run"),
new Note("Is a chad")
);
verifyThat("#notesListView", hasItems(3));
assertTrue(notesWindow.getNotesListView().getItems().containsAll(expectedNotes));
ObservableList<Note> actualNotes = notesWindow.getNotesListView().getItems();
assertTrue(actualNotes.containsAll(expectedNotes));
});
}

Expand Down

0 comments on commit 22cf1e8

Please sign in to comment.