From 95e1c32d096543f2cd7edc9fc5c0b141a56a3359 Mon Sep 17 00:00:00 2001 From: h7x4 Date: Mon, 26 Apr 2021 22:49:45 +0200 Subject: [PATCH] Update requirements --- REQUIREMENTS.md | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/REQUIREMENTS.md b/REQUIREMENTS.md index 9480aa7..a820e88 100644 --- a/REQUIREMENTS.md +++ b/REQUIREMENTS.md @@ -158,6 +158,8 @@ Dette finner dere i [service/FileOperations.java][FileOperations-file] Dere finner testene i [src/test/java/app][tests-folder] +**OBS: se nederst angående `FileOperations` og `DialogBoxes`** + 2. **Enhetstestene skal skrives i JUnit 5.** Alle testene er skrevet i JUnit 5, med hjelp fra et par andre biblioteker og plugins som bygger på toppen av JUnit. @@ -170,6 +172,12 @@ 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`][] + +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 + +Se [aggarwal-rohan17.medium.com](https://aggarwal-rohan17.medium.com/simplifying-junit-mockito-and-powermock-d1392059ce87#48d4) for detaljer + [Model-file]: https://gitlab.stud.idi.ntnu.no/oysteikt/h20-tdt4100-project/-/blob/master/src/main/java/app/model/Model.java [Event-file]: https://gitlab.stud.idi.ntnu.no/oysteikt/h20-tdt4100-project/-/blob/master/src/main/java/app/events/Event.java