avajs/ava

Diffs for arrays of strings can be pretty confusing

Open

#1521 aperta il 16 set 2017

Vedi su GitHub
 (3 commenti) (1 reazione) (0 assegnatari)JavaScript (1408 fork)batch import
bughelp wantedscope:assertionsscope:reporters

Metriche repository

Star
 (20.600 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

If you do t.deepEqual (and I think also t.is) on two arrays containing strings. If there are a large amount of strings and they only differ slightly the output is very confusing. It appears that as soon as a new entry in an array has shifted the items after it across, AVA text diffs every single entry against the same index in the original array.

e.g: An array with 219 strings that has had 6 additions and 1 deletion: AVA diff: https://travis-ci.org/lukechilds/window/jobs/266859856 GitHub diff: https://github.com/lukechilds/window/pull/7/commits/6ecbade39561cf705ddc2833cbc3b93e2abfb223

Guida contributor

Direzione di ricerca
Esamina la logica di differenziazione di uguaglianza profonda nel codice sorgente di AVA, specificamente per array di stringhe, e migliora l'output per renderlo più leggibile quando vengono aggiunti o rimossi elementi.
Tech stack
javascriptnodejs
Dominio
testing
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65