avajs/ava

Make t.fail() accept an object

Open

#2615 aperta il 25 nov 2020

Vedi su GitHub
 (5 commenti) (1 reazione) (0 assegnatari)JavaScript (1408 fork)batch import
enhancementhelp wantedscope:assertions

Metriche repository

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

Descrizione

It seems like the only problem with #1094 is providing detailed error output when a custom assertion fails. Everything else is straightforward. What if t.fail() could accept an object to use for creating an AssertionError? The fields could be message, actual, expected, and format that will specifying which formatting function to use for setting the values field on the error object. Could also add an optional assertion field for a custom assertion name.

Would this be compatible with #2435?

If there's interest, I can submit a PR for this.

Guida contributor

Direzione di ricerca
Esamina l'implementazione corrente di t.fail() e AssertionError in AVA. Rivedi il problema #2435 per potenziali conflitti. Comprendi come funziona la formattazione dell'oggetto errore.
Tech stack
javascriptnodejs
Dominio
testing
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
78