Skip to content

Add support for GraphQL 2.4#281

Merged
mcelicalderon merged 1 commit intomasterfrom
support-gql-2-4
Nov 29, 2024
Merged

Add support for GraphQL 2.4#281
mcelicalderon merged 1 commit intomasterfrom
support-gql-2-4

Conversation

@mcelicalderon
Copy link
Copy Markdown
Member

No description provided.

@mcelicalderon mcelicalderon added the enhancement New feature or request label Nov 29, 2024
it 'skips the register mutation' do
expect(json_response[:errors]).to contain_exactly(
hash_including(message: "Field 'adminRegister' doesn't exist on type 'Mutation'")
hash_including(message: a_string_including("Field 'adminRegister' doesn't exist on type 'Mutation'"))
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

graphql 2.4 adds additional information to the error message. But we only care in the spec about the field not being defined in the API

@mcelicalderon
Copy link
Copy Markdown
Member Author

@00dav00 could you please review this one? No code changes were required for this upgrade 🙌

@mcelicalderon mcelicalderon merged commit 1bf828c into master Nov 29, 2024
@mcelicalderon mcelicalderon deleted the support-gql-2-4 branch November 29, 2024 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants