Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
jonasseglare opened this issue May 12, 2023 · 0 comments · May be fixed by #21
Open

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

jonasseglare opened this issue May 12, 2023 · 0 comments · May be fixed by #21
Labels
bug Something isn't working

Comments

@jonasseglare
Copy link

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: eb72ea6

Ä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: 76c0cdd

Optional Screenshots

No response

Relevant log output

Se `Steps to Reproduce`.
@jonasseglare jonasseglare added the bug Something isn't working label May 12, 2023
@jonasseglare jonasseglare linked a pull request May 12, 2023 that will close this issue
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant