graphql/graphql-js

Expand test suite for resolve data

Open

#351 ouverte le 9 avr. 2016

Voir sur GitHub
 (0 commentaires) (2 réactions) (0 assignés)TypeScript (2 223 forks)batch import
help wanted

Métriques du dépôt

Stars
 (19 825 stars)
Métriques de merge PR
 (Merge moyen 12j 6h) (42 PRs mergées en 30 j)

Description

Currently there is only one test case for "gets execution info in resolver" which does not test the breadth of execution info. It would be useful to expand the test cases around what information the resolve functions can expect to receive. Currently, only rootValue is tested from the GraphQLResolveInfo parameter. The suite should have the same kind of test for the other fields, especially fieldASTs, returnType, and parentType in nested and nonNull conditions. It should also test all three functions that receive GraphQLResolveInfo.

This can help make it easier to change the execute and resolve process without worrying about regressions.

https://github.com/graphql/graphql-js/blob/master/src/execution/__tests__/union-interface-test.js#L351

Extracting this from #304 as a separate issue.

Guide contributeur

Direction de recherche
Identifiez les cas de test manquants pour les champs GraphQLResolveInfo et ajoutez les dans le fichier de test existant. Concentrez vous sur fieldASTs, returnType et parentType dans des conditions imbriquées et non nulles. Assurez vous que les trois fonctions de résolveur (resolve, resolveType et isTypeOf) soient testées.
Stack technique
typescriptjavascript
Domaine
api
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
GraphQLJest
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75