test fladgejt

[janick0]

testovanie metod z fladgejt webui tried pomocou novej triedy v subore test_fladgejt.py

---

This change is 

[TomiBelan]

Reviewable status: 0 of 2 files reviewed, 5 unresolved discussions (waiting on @janick0 and @TomiBelan)

---

fladgejt/webui/test_fladgejt.py, line 13 at r1 (raw file):

class WebuiTestFladgejtMixin:
    

Formátovacie drobnosti:
Tvoj editor necháva medzery na konci riadku. Nastav si ho aby ich automaticky mazal, a existujúce zmaž.
Mnohé riadky majú 12 medzier aj keby mali mať 8.
Namiesto WebuiNejakyMixin.meno_funkcie(self, ...) stačí self.meno_funkcie(...).

---

fladgejt/webui/test_fladgejt.py, line 14 at r1 (raw file):

class WebuiTestFladgejtMixin:
    
    # je dobry napad definovat si takto globalne premenne?

Je lepšie sa im vyhnúť.
Niektoré z týchto premenných sa nikdy nemenia alebo sa používajú len v jednej funkcii, a môžu sa z nich stať lokálne premenné.
Na posielanie dát medzi funkciami pôjde použiť return. Napríklad _test_predmety môže skončiť s return studium_key, zapisny_list_key a test ich dá ostatným v argumentoch.

---

fladgejt/webui/test_fladgejt.py, line 127 at r1 (raw file):

je v pohode skusat robit veci ako vytvaranie / mazanie zapisnych listov a pod.? to sa bude takto robit z nejakeho bezneho aisovskeho uctu ?

Bude to robiť bežný užívateľ (náhodný študent). Ale bude to na inštancii ais2-beta.uniba.sk kde sú testovacie dáta, takže aj write operácie sú teoreticky OK. Ale iba teoreticky, lebo zápis predmetov beží iba časť roka, a prihlasovanie na skúšky tiež, takže inokedy by test vždy zlyhal. Sú dve možné riešenia: jedno je nejako kontrolovať či je práve zápis zapnutý (resp. či práve existuje nejaká skúška kde by sme sa mohli prihlásiť), a robiť to iba ak áno. Druhé "riešenie" je netestovať tieto write veci nikdy a sústrediť sa iba na read-only operácie (čo pokryje menej Votr kódu ale stále dosť veľa).

neni to take moc staticke, da sa to vobec spravit lepsie?

Napríklad so skratkou predmetu: zadrátovať jeden konkrétny predmet, na ktorom sa testuje get_studenti_zapisani_na_predmet apod, je úplne OK. Horšie je predpokladať, že ten konkrétny predmet má tento užívateľ zapísaný (to je len príklad, neviem či to ten kód robí). Časom môžeme skúsiť otestovať, či to funguje viacerým rôznym prihláseným ľuďom.

---

fladgejt/webui/test_fladgejt.py, line 155 at r1 (raw file):

            
        
    def test(self):

Skúšal si ten test spustiť? Funguje ti?
Radšej na beta databáze, kde sa pripojíš takto:

• Prihlás sa na ais2-beta.uniba.sk
• Otvor developer tools a v Application -> Cookies nájdi hodnotu JSESSIONID
• V notebooku: client = create_client(ais_beta, dict(type='cosigncookie', ais_cookie='tá hodnota'))

---

fladgejt/webui/test_fladgejt.py, line 162 at r1 (raw file):

        print("Test predmety: " + self._test_predmety())
        print("Test hodnotenia: " + self._test_hodnotenia())
        print("Test termin: " + self._test_termin())

Z webu print nebude fungovať.
Jednotlivé podmetódy nemusia vracať "OK", a iba na konci celého testu stačí return "OK".

[Adman]

@janick0 ahoj, budes mat cas na updatnutie kodu podla pripomienok?