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

Unit tesztek #11

Merged
merged 23 commits into from
May 21, 2024
Merged

Unit tesztek #11

merged 23 commits into from
May 21, 2024

Conversation

leventebajak
Copy link
Contributor

@leventebajak leventebajak commented May 16, 2024

Egységtesztek készítése + tesztek kódlefedettségének mérése

Egységtesztek készítése

Az egységtesztek készítéséhez a JUnit és Mockito package-eket használtuk.

Nem minden osztály lett tesztelve, de azok, amelyeket teszteltünk, azok részletesen le lettek fedve.

Az egységtesztek a /src/test/java mappában találhatóak.

A UI osztályok tesztelése nem volt célunk, mivel ezt a Manuális tesztek megtervezése feladat magában foglalja.

Kódlefedettség mérése

A kódlefedettséget a JaCoCo segítségével mértük. A build konfigurációban be lett állítva, hogy a tesztek futtatása
után a JaCoCo generáljon egy jelentést.

Az elkészült jelentés a /target/site/jacoco/index.html fájl megnyitásával tekinthető meg.

A jelentések elemzése segített a le nem fedett ágak azonosításában.

Több fájl lefedettsége is eléri a 100%-ot.

Copy link
Contributor

@KarsaiArtur KarsaiArtur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

esetleg egy játék csapat pontszám teszt lehetett volna, de nagyon szép munka. szép alapos teszteket csináltatok. esetleg még annyi, hogy egy kódlefedettség képet tudtok csatolni?

src/test/java/TankTest.java Outdated Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

csak annyi, hogy esetleg a slippery effektjét mozgásnál is le lehetett volna tesztelni, de ezek is jótesztek


@Test
void setDirection() {
// TODO: Remove the SetDirection method because it does nothing, just like the super class's method
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jogos felvetés

src/test/java/PumpTest.java Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hát igen, a forráson sok mindent nem lehetett tesztelni

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ezek is jónak tűnnek

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

szép munka. szép alapos teszteket csináltatok

Copy link
Contributor

@tomzso tomzso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Szép részletesek a tesztek

Copy link
Contributor

@szilva07 szilva07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A kép alapján a fontosabb és tesztelhetőbb osztályok jól le vannak fedve.

doc/Coverage.png Show resolved Hide resolved
@tomzso tomzso self-requested a review May 17, 2024 20:27
Copy link
Contributor

@KarsaiArtur KarsaiArtur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Köszi a képet. Akkor a tesztelhetőbb osztályok jól le vannak fedve.

doc/Coverage.png Show resolved Hide resolved
@leventebajak leventebajak merged commit 613d270 into main May 21, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Egységtesztek készítése + tesztek kódlefedettségének mérése
5 participants