chucker
chucker copied to clipboard
Update transaction list to differentiate GraphQL requests
:camera: Screenshots
:page_facing_up: Context
Issue #116 talks about the shortcoming of the initial approach of supporting GraphQL on Chucker.
This PR is an attempt support GraphQL on Chucker using the guidelines mentioned in Issue #116.
:pencil: Changes
- Enable clients to set a
graphQLEndpoint
through theChuckerInterceptor.Builder
- Pass the
graphQLEndpoint
as an argument to theRequestProcessor.process(..)
method - Add
isGraphQLRequest
property in theHttpTransaction
entity &HttpTransactionTuple
- Expose
isGraphQLRequest
viaHttpTransactionTuple
- Tests
- Update Sample app to illustrate the effects of the change.
:paperclip: Related PR
PR#805 is also doing the same thing but does not handle GraphQL requests via HTTP GET methods. But this PR does not block any other PR.
:no_entry_sign: Breaking
-
ChuckerInterceptor
now contains an additional parameter to represent a GraphQL path or url -
RequestProcessor
'sprocess(..)
method would accept the Graphql path passed by Interceptor.
:hammer_and_wrench: How to test
Test cases for RequestProcessor
included in the PR.