handshake-org/hsd

Improve test coverage

Open

#684 ouverte le 11 janv. 2022

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (1 814 stars) (286 forks)batch import
Good first issuetests

Description

I noticed in the coverage report that a lot of the wallet make___() errors are not covered (eg "Auction is not yet open" and "Name has not reached bidding phase", etc). This could be another "good first issue" that would be easy to do and get a few more lines covered.

Originally posted by @pinheadmz in https://github.com/handshake-org/hsd/issues/683#issuecomment-1009248506

Guide contributeur

Stack technique
javascript
Domaine
testing
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Node.jsbasic testing knowledge
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85
Direction de recherche
The issue requests adding test coverage for wallet `make ()` error messages. Review the referenced comment in issue #683 for context. Locate the wallet module in the codebase, identify all error strings (e.g., 'Auction is not yet open'), and write unit tests that verify these errors are thrown correctly under relevant conditions.