Allow simpler tests for error.code
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- javascript
- Domaine
- testing-qa
Piste de recherche
Commencez par localiser les points d’entrée des assertions throw() et reject() ainsi que le comportement existant de contain(). Ajoutez une couverture de tests pour un message facultatif et l’objet de propriétés d’erreur proposé, y compris l’exemple ERR_INVALID_ARG_TYPE, et vérifiez que les deux assertions conservent leur comportement actuel.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Support plan
- is this issue currently blocking your project? (yes/no): no
- is this issue affecting a production system? (yes/no): no
Context
- node version: any
- module version: 8.0.3
- environment (e.g. node, browser, native): node
- used with (e.g. hapi application, another framework, standalone, ...):
- any other relevant information:
What problem are you trying to solve?
To test for the .code property attached to node errors (instead of .message) that are not fixed).
Currently I have to do this manually, eg. for throw():
const Code = require('code');
const expect = Code.expect;
const err = expect(() => process.chdir(42)).to.throw(TypeError);
expect(err).to.contain({ code: 'ERR_INVALID_ARG_TYPE' });
// or as one-liner
expect(expect(() => process.chdir(42)).to.throw(TypeError)).to.contain({ code: 'ERR_INVALID_ARG_TYPE' });
Do you have a new or modified API suggestion to solve the problem?
Add a third has parameter to throw() and reject(), which takes an object which is passed to a contain() test. Also make the message optional (works since it uses a string and type must be instanceof Error). Eg.
expect(() => process.chdir(42)).to.throw(TypeError, { code: 'ERR_INVALID_ARG_TYPE' });
- Langage dominant
- JavaScript
- Étoiles
- 231
- Forks
- 75
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de hapijs/code
-
Incorrect prototype checkOuvertesupport
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
-
pass parameter to throwOuvertefeature
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
Toutes les issues de hapijs/code
Issues similaires
-
Remove: Fox Deportes SDOuvertecheck:passed feeds:remove
Difficulté 1/5 Moins d'une heure Accessibilité débutants 65/100
iptv-org/database#37176 · 1 commentaire · 1 réaction ·
Les mainteneurs répondent en général sous 9 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
hawk-digital-environments/HAWKI#443 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour
-
feedback simulation workshop
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
githubnext/gh-aw-workshop#4370 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
AltimateAI/vscode-dbt-power-user#2089 ·