decentralized-identity/dwn-server

Remove direct use of default `config` to avoid config changes bleed/pollute between tests

Ouverte

#144 ouverte le 10 juil. 2024

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)TypeScript (43 forks)github user discovery
good first issuetest

Métriques du dépôt

Stars
 (41 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Remove direct use of default config to avoid config changes bleed/pollute between tests, should clone before modify.

See http-api.spec.ts as a prime example of misuse. As a results, tests like dynamic-plugin-loading.spec.ts need to override the config explicitly to ensure reliability.

Guide contributeur

Direction de recherche
Identifiez les tests qui utilisent directement l'objet de configuration par défaut (comme http api.spec.ts) et modifiez les pour cloner la configuration avant d'apporter des modifications. Assurez vous que la substitution dans dynamic plugin loading.spec.ts n'est plus nécessaire.
Stack technique
typescript
Domaine
testingbackend
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.
1-3 heures
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
TypeScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65