martinmacko47 / chcemvediet

FOIA requests for the lazy
4 stars 4 forks source link

Doplnit testy pre nove features #281

Open martinmacko47 opened 4 years ago

martinmacko47 commented 4 years ago

Depends on #280

Prejdi si vsetky svoje PR, co si doteraz spravil, a ku kazdemu PR sa zamysli, ci netreba doplnit nejake testy. Ak ano, dopln ich.

Testy by mali byt napisane tak, aby presli vsetkymi vetvami kodu. Tj. vo vseobecnosti ak je niekde if X, tak by mal existovat test, kde X bolo true, aj test, kde X bolo false. Testy by mali pokryvat pozitivne pripady aj negativne pripady. Tj. ze sa da, co sa ma dat, a ze sa neda, co sa nema dat. Testy by mali overovat hlavne extremne pripady. Napriklad ak sa nieco da od 0 do 100 a neda od 101, tak by mali byt testy pre -1, 0, 100 a 101.

Okrem toho by testy mali overovat funkcnost pouzitych libiek resp. inych externych nastrojov. Hlavne ak sa niekde v kode spoliehas sa nieco, co nie je uplne presne zdokumentovane v dokumentacii danej libky/toolu. Aby ked danu libku/tool upgradneme, a nastala v nej nekompatibilna zmena, aby sme si to na zaklade padajuceho testu vsimli.

Ku kazdemu PR, co si doteraz spravil, sprav novy PR s testami pren. Do popisu PR s testami daj linku na povodny PR, ku ktoremu testy patria.

Napriklad vid: https://github.com/martinmacko47/chcemvediet/pull/283#issuecomment-633587892

martinmacko47 commented 3 years ago

*label blocked rusim, stav issue nahradza stlpec v boarde