graphql/graphql-js

Expand test suite for resolve data

Open

#351 aperta il 9 apr 2016

Vedi su GitHub
 (0 commenti) (2 reazioni) (0 assegnatari)TypeScript (2223 fork)batch import
help wanted

Metriche repository

Star
 (19.825 star)
Metriche merge PR
 (Merge medio 12g 6h) (42 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Identifica i casi di test mancanti per i campi di GraphQLResolveInfo e aggiungili nel file di test esistente. Concentrati su fieldASTs, returnType e parentType in condizioni annidate e non nulle. Assicurati che tutte e tre le funzioni del resolver (resolve, resolveType e isTypeOf) siano testate.
Tech stack
typescriptjavascript
Dominio
api
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GraphQLJest
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75