codenotary/immudb

Tests improvements: Better check for errors

Aperta

#1435 aperta il 26 ott 2022

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Go (371 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (8999 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

What would you like to be added or enhanced

In many tests we're using require.Error(t, err) which only checks if the error happened without checking if the error is actually expected.

We should add more detailed error check, either by using require.ErrorIs or require.ErrorContains assertions.

Why is this needed

When expecting an error it is necessary to ensure the returned error is what was expected. It may turn out that in some cases the error checked by the test is not what it should be.

Guida contributor

Direzione di ricerca
Cerca tutte le chiamate require.Error(t, err) nei file di test e sostituiscile con asserzioni più specifiche come require.ErrorIs o require.ErrorContains, in base al tipo o messaggio di errore previsto.
Tech stack
go
Dominio
databasebackend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
Gotesting
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90