Viděno až po ac6c41cc88, LGTM. (TreeNody mi tu klidně zatím nech, domlátím během schůzky asi…)
Mám za to, že Pythoní logování drží hierarchii podle teček, takže personalni.prihlaska.problem.edit se chová by default stejně jako personalni.prihlaska.problem. Ale nevěřím tomu…
Hmm, ale zrovna authtoken je součástí django-rest-framework, který v requirements je. Tak nevím…
Očividně jsou na testu starší verze knihoven. (
Asi závislost na verzi Pythonu, protože pip install --upgrade nefunguje…Reinstall envu pomohl (což vlastně mohlo aktualizovat verzi…
(A já se jdu pokusit to někam mergenout, ze cviku…)
Nějak to mergenout šlo (konflikt byl myslím jen v treenode/models.py a bylo potřeba předělat pár views), ukázkový merge…
8283b530e9
Merge remote-tracking branch 'gitea/podezrele-seminare' into bez_treenodu_i_bez_podezrelych_seminaru
Asi OK (až na to, co píšu výš). Věci kolem TreeNodů spíš prosím někam poznamenat, chápu, že se nikomu nechce je implementovat teď hned…
Heh, tohle nikde není deklarované, i když se to od pohledu má chovat (zvlášť co se týče ochrany dat) stejně jako personalni.prihlaska.problem. Myslím, že by mělo stačit tady (jinde to stejně není) přejmenovat na personalni.prihlaska.problem.edit (ale teď nekontroluji, jestli si pamatuji chování pythoního logování správně…)
V principu myslím, že tady je lepší řešení spíš import treenode.models as tn, ale až někdo někdy bude pokračovat ve psaní těchto testů, tak si to tam nejspíš dopíše, tak je to teď asi jedno…
Tyhle komentáře způsobí tiché spadnutí. Byl by to NameError, nejspíš i při validním volání, teď je to return None, díky čemuž se bude chyba špatně lovit. NameError je za mě fajn, ale klidně to jde vyřešit nějakým globálním raise ValueError('tenhle typ TreeNode neexistuje').