martinmacko47
commented
4 years ago Pri opravovani existujucich testov bude asi lahsie zacat od testov v adresari /poleno, kedze tam su vseobecnejsie utility.
Edit: :heavy_check_mark:
Open martinmacko47 opened 4 years ago
martinmacko47
commented
4 years ago Pri opravovani existujucich testov bude asi lahsie zacat od testov v adresari /poleno, kedze tam su vseobecnejsie utility.
Edit: :heavy_check_mark:
martinmacko47
commented
4 years ago @viliambalaz Doplnil som do tasku zoznam testov, kde mame skip. Nech mame prehlad, kolko z toho je uz fixnute.
Btw, preco sme premenovali poleno/utils/test.py na poleno/utils/skip_test.py? Ved to nie su testy, ale utility k testom. Potom jasne, ze ti nefungovali importy v testoch, ked si premenoval importovany modul. Pls najprv, predtym nez budes fixovat ostatne testy, premenuj ten modul naspat a odznac skip_ prefixom moduly, ktorym uz importy potom budu fungovat.
Edit: :heavy_check_mark:
viliambalaz
commented
4 years ago Pouzit sample HTML vsade, kde je sample HTML dokument. <html><body>HTML alternative</body></html>.
Odlozene z: https://github.com/martinmacko47/chcemvediet/pull/303#pullrequestreview-478190307
PR: https://github.com/martinmacko47/chcemvediet/pull/319 :heavy_check_mark:
viliambalaz
commented
4 years ago Vo viacerych suborov treba zmenit import na from poleno.utils.urls import reverse.
Odlozene z: https://github.com/martinmacko47/chcemvediet/pull/304#pullrequestreview-485071627
PR: https://github.com/martinmacko47/chcemvediet/pull/320 :heavy_check_mark:
viliambalaz
commented
4 years ago Upravit testy pre tag change_lang definovany v poleno/utils/.
Odlozene z: https://github.com/martinmacko47/chcemvediet/pull/304#pullrequestreview-499589044
PR: https://github.com/martinmacko47/chcemvediet/pull/321 :heavy_check_mark:
viliambalaz
commented
3 years ago filter(name=value).order_by_pk().first() prepisat na .get(name=value)
Odlozene z: https://github.com/martinmacko47/chcemvediet/pull/308#pullrequestreview-506445401
martinmacko47
commented
3 years ago *label working rusim, stav issue nahradza stlpec v boarde
[x] V prvom PR si spust lokalne vsetky testy a padajuce testy s dekoratorom @skip. Testy sa pustaju pomocou
env/bin/python manage.py test[x] V druhom PR rozbehaj CI na githube. Aby sa po kazdom pushnutom commite automaticky na githube spustili testy. Ak si nebudes vediet rady, googli. Na githube som CI este nerozbehaval. Ale v principe by malo stacit vytvorit subor
.github/workflows/main.ymls popisom CI. Ciel je, aby v kazdom PR bol check, ci testy zbehli a nedovolito to PR mergnut ak testy nezbehli.[ ] Potom dalej v samostatnych PR si zober jednu skupinu testov a padajuce testy fixni. V kazdom PR si zober napr. jeden adresar z
poleno/, alebo jednu app zchcemvediet/apps/. Pripadne len jeden subor s testami, ak je tam vela parajucich testov.Pri niektorych testoch bude jasne, co sa v kode zmenilo a ako testy treba upravit. Pri niektorych testoch to nemusi byt uplne jasne. Kde to nie je jasne, mozeme si k tomu zavolat a zamysliet sa nad tym.