Ogólnie wpadłem w projekt, w…

Categories Programowanie

Ogólnie wpadłem w projekt, w którym jakość kodu jest porównywalna tylko z Comarchem za moich czasów (8 lat temu) z tych, które widziałem
Siedzę sobie w pracy dostaję PR do akceptacji. Nawet jako tako wygląda. No, ale jedna rzecz jest zapala żółtą lampkę. Zmiana z asercji z true na false. Patrzę na nazwę testu no jasno mówi, że powinno być true. No to piszę, że tutaj mamy nieścisłość. Nazwa mówi, że ma być true, a asercja sprawdza czy jest false. Kolega właśnie napisał, że fixed.

pokaż spoiler zmienił nazwę testu

kurtyna

pokaż spoiler tak dla jasności, wg mnie albo nazwa w ogóle nie powinna wspominać czy ma być true/false albo jeśli robi to jakąś różnicę to powinny być sprawdzone oba przypadki, a nie zmienianie nazwy, bo coś tam zmieniłem i mi test failuje. Po to są testy, żeby właśnie wykrywać błędy, a nie arbitralnie zmieniać sobie test, bo coś tam wyjebało się.

#programowanie