Jest: Incohérence dans les docs et les exemples

Créé le 9 avr. 2017  ·  3Commentaires  ·  Source: facebook/jest

PR #3172 a mis à jour la documentation pour inclure les nouveaux mots-clés .resolves et .rejects pour l'exécution d'assertions sur Promises.

J'ai remarqué que ce PR introduisait des incohérences dans la section de gestion des erreurs mise à jour de https://github.com/facebook/jest/blob/master/docs/TutorialAsync.md. Il y a une capture d'écran de cette section ci-dessous :

screen shot 2017-04-09 at 09 11 46

  1. Les commentaires en haut du test font toujours référence à try-catch , ce qui est incohérent avec le code contenu dans les extraits de code.
  2. Le PR ne montre qu'un exemple avec .rejects qui ne sera disponible que dans Jest 20+. Cela devrait être mentionné dans la doc pour éviter toute confusion.
  3. Les personnes utilisant des versions plus anciennes de Jest ne pourront pas utiliser cette documentation. Je pense que nous devrions rajouter l'explication sur l'utilisation de try-catch pour attraper le rejet de Promise (qui a été supprimé dans PR #3172) pour les personnes utilisant Jest 19 et ci-dessous. Cela peut cohabiter avec les documents actuels en utilisant .rejects .
  4. L'exemple contenu dans le lien examples/async renvoie à des exemples de test comme celui-ci . Ces exemples utilisent toujours le mécanisme try-catch , ce qui est incohérent avec les exemples de code actuellement contenus dans la documentation. C'est une autre raison pour laquelle nous devrions rajouter l'explication de l'utilisation de try-catch à cette section de la documentation.

Commentaire le plus utile

Je ne sais pas quand jest 20 sortira mais j'espère bientôt :)

Tous les 3 commentaires

Est-il possible d'essayer jest 20.0.0+ pour ces nouveaux mots clés ?

Je ne sais pas quand jest 20 sortira mais j'espère bientôt :)

Je pense que cela est maintenant résolu avec #3201. Si vous pensez que nous pouvons faire plus ici, veuillez envoyer un PR :)

Cette page vous a été utile?
0 / 5 - 0 notes