-
Notifications
You must be signed in to change notification settings - Fork 1
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
Unit tesztek #11
Conversation
100% lefedettség
There was a problem hiding this 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/SaboteurTest.java
Outdated
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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/FountainTest.java
Outdated
There was a problem hiding this comment.
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
src/test/java/FixerTest.java
Outdated
There was a problem hiding this comment.
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
src/test/java/BlankFieldTest.java
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
src/test/java/ActionHandlerTest.java
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
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
There was a problem hiding this 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
There was a problem hiding this 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.
There was a problem hiding this 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.
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.