diggsweden / DCAT-AP-SE-Processor

GNU General Public License v3.0
9 stars 8 forks source link

Koden för enhetstester i ManagerTest är kommenterad men testerna fungerar #20

Open jonasseglare opened 1 year ago

jonasseglare commented 1 year ago

Contact Details

jonas.ostlund@arbetsformedlingen.se

What happened?

Jag har börjat titta på koden för klassen Manager för att eventuellt utöka den. Jag noterade att enhetstesterna i ManagerTest är utkommenterade. Jag är nyfiken på varför de är det.

Testerna kommenterades i denna commit: https://github.com/diggsweden/DCAT-AP-SE-Processor/commit/eb72ea6055fc00377e31ea43596d29f3425339a6

Är det någon som känner till bakgrunden till detta?

När jag tar bort kommentaren runt testerna provkör testen med mvn test och nuvarande commit fungerar de bra.

Steps To Reproduce

För att förstå varför testerna kommenterades checkade jag ut commiten där det gjordes: git checkout eb72ea6055fc00377e31ea43596d29f3425339a6. I det här tillståndet är alltså testerna kommenterade. Sedan skrev jag mvn test men lyckades inte bygga projektet pg.a. Fatal error compiling: java.lang.IllegalAccessError: class lombok.javac.apt.LombokProcessor.

Sedan provade jag att checka ut aktuell commit:

git checkout 76c0cdd78140f8cd06335566e2bdd7274b2ae7a0
mvn clean
mvn test

och får rapporten Tests run: 74, Failures: 0, Errors: 0, Skipped: 0. Slutligen tog jag bort kommentaren runt testerna i ManagerTest och körde igen

mvn clean
mvn test

och får då rapporten Tests run: 79, Failures: 0, Errors: 0, Skipped: 0, d.v.s. ytterligare 5 tester kördes utan problem.

What did you expect?

Jag hade förväntat mig att testerna i ManagerTest skulle misslyckas när jag tog bort kommentaren runt dem.

Version

Detta är aktuell commit: https://github.com/diggsweden/DCAT-AP-SE-Processor/commit/76c0cdd78140f8cd06335566e2bdd7274b2ae7a0

Optional Screenshots

No response

Relevant log output

Se `Steps to Reproduce`.