codenotary/immudb

Tests improvements: Better check for errors

Ouverte

#1 435 ouverte le 26 oct. 2022

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (0 personne assignée)Go (371 forks)auto 404
enhancementgood first issue

Métriques du dépôt

Stars
 (8 999 étoiles)
Métriques de merge PR
 (Merge moyen 5j 15h) (18 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Recherchez tous les appels require.Error(t, err) dans les fichiers de test et remplacez les par des assertions plus specifiques, telles que require.ErrorIs ou require.ErrorContains, en fonction du type ou du message d'erreur attendu.
Stack technique
go
Domaine
databasebackend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Gotesting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90