diff --git a/REQUIREMENTS.md b/REQUIREMENTS.md index a820e88..612345b 100644 --- a/REQUIREMENTS.md +++ b/REQUIREMENTS.md @@ -172,9 +172,9 @@ Disse inluderer: - [Monocle](https://wiki.openjdk.java.net/display/OpenJFX/Monocle) - [Jacoco](https://www.eclemma.org/jacoco/) -På grunn av hvordan noen av klassene er skrevet statisk, så mangler vi to tester for [`FileOperations`][] og [`DialogBoxes`][] +På grunn av hvordan noen av klassene er skrevet statisk, så mangler vi to tester for [`FileOperations`][FileOperations-file] og [`DialogBoxes`][DialogBoxes-file] -Grunnen til at vi ikke fikk testet disse er fordi at Mockito selv ikke kan teste kode som inneholder `static`/`final` properties, eller inline bruk av konstruktør. For disse brukstilfellene har det vært vanlig å bruke [Powermock][powermock], men dette støttet ikke JUnit5 enda. I [`FileOperationsTest.java`][] finner dere kommentert ut kode som ville vært hvordan vi hadde skrevet koden om powermock var tilgjengelig for JUnit5 +Grunnen til at vi ikke fikk testet disse er fordi at Mockito selv ikke kan teste kode som inneholder `static`/`final` properties, eller inline bruk av konstruktør. For disse brukstilfellene har det vært vanlig å bruke [Powermock][powermock], men dette støttet ikke JUnit5 enda. I [`FileOperationsTest.java`][FileOperations-file] finner dere kommentert ut kode som ville vært hvordan vi hadde skrevet koden om powermock var tilgjengelig for JUnit5 Se [aggarwal-rohan17.medium.com](https://aggarwal-rohan17.medium.com/simplifying-junit-mockito-and-powermock-d1392059ce87#48d4) for detaljer @@ -196,6 +196,8 @@ Se [aggarwal-rohan17.medium.com](https://aggarwal-rohan17.medium.com/simplifying [ThemeChangedEvent-file]: https://gitlab.stud.idi.ntnu.no/oysteikt/h20-tdt4100-project/-/blob/master/src/main/java/app/events/ThemeChangedEvent.java [FileOperations-file]: https://gitlab.stud.idi.ntnu.no/oysteikt/h20-tdt4100-project/-/blob/master/src/main/java/app/service/FileOperations.java [FileTreeOperations-file]: https://gitlab.stud.idi.ntnu.no/oysteikt/h20-tdt4100-project/-/blob/master/src/main/java/app/service/FiletreeOperations.java +[DialogBoxes-file]: https://gitlab.stud.idi.ntnu.no/oysteikt/h20-tdt4100-project/-/blob/master/src/main/java/app/service/DialogBoxes.java +[powermock]: https://github.com/powermock/powermock diff --git a/src/main/java/app/Main.java b/src/main/java/app/Main.java index 52b94ec..dee3ccf 100644 --- a/src/main/java/app/Main.java +++ b/src/main/java/app/Main.java @@ -4,7 +4,6 @@ import java.io.IOException; import java.util.Optional; import javafx.application.Application; -import javafx.application.Platform; import javafx.fxml.FXMLLoader; import javafx.scene.Parent; import javafx.scene.Scene;