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

Tests #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Tests #1

wants to merge 2 commits into from

Conversation

patrikKaras123
Copy link

Po pozreti kodu som dosiel na to ze sa nespravne vymazavaju tabulky a pada to pri vytvarani ucitela alebo studenta. Pada to na if check ze najde AisId (z predosleho testu) a tym padom vzdy vrati null. Updatol som testy a dal tam beforeEach vymazavanie udajov z tabulky a prejde vacsina testov. Vopred dakujem

@tuplle
Copy link
Member

tuplle commented Apr 17, 2023

Nie je vhodné na merge nakoľko sú upravené aj iné zdrojové kódy projektu ako testy. Navrhnuté riešenie vymazania databázy nie je aplikovateľné nakoľko sa viaže na presné pomenovanie tabuliek, ktoré nemusí byť v iných projektoch študentov. Vymazanie databázy som testoval na rôznych nastaveniach a vždy fungovalo spoľahlivo. Odporúčam si dôkladne pozrieť metódu TestUtils.clearDB(db)

@tuplle tuplle self-requested a review April 17, 2023 23:39
@tuplle tuplle added the bug Something isn't working label Apr 17, 2023
@patrikKaras123
Copy link
Author

Dobry den, ano viem ze to clearovanie db nieje vhodne pre vsetky testy a iba pre tie ktore sa maju classes nazvane presne tak isto ale clearDb funkcia nefunguje spravne podla mna lebo ostavaju hodnoty v db. Jedine testy ktore mi prejdu su tie ktore sú prve a ptm pri creatovani dalsieho teachera alebo studenta ostavajú v db hodnoty. Cize creatovanie je zavisle na ostatnych testoch a tym padom neprejdu skoro vsetky testy (prejdu iba tie ktore su prve). Tento problem nemam len ja ale aj ini ludia, ze to pada na aisId. Iba som si vsimol tuto vec. Vopred dakujem.

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 this pull request may close these issues.

3 participants