Skip to content

Add unit tests for custom GraphQL directives - #4165

Merged
josephsnyder merged 2 commits into
Kitware:masterfrom
williamjallen:custom-graphql-directive-unit-tests
Oct 28, 2026
Merged

josephsnyder merged 2 commits into
Kitware:masterfrom
williamjallen:custom-graphql-directive-unit-tests

Conversation

@williamjallen

Copy link
Copy Markdown
Member

Our custom GraphQL directives are currently tested by the tests/Feature/GraphQL/FilterTest.php feature test and test cases in other GraphQL type tests. Those tests test basic behavior against a live database, but don't make assertions about the schema changes these directives cause. This PR adds unit tests which more precisely test what these directives actually do.

@josephsnyder
josephsnyder added this pull request to the merge queue Oct 28, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 28, 2026
@williamjallen
williamjallen added this pull request to the merge queue Oct 28, 2026
@williamjallen
williamjallen removed this pull request from the merge queue due to a manual request Oct 28, 2026
Our custom GraphQL directives are currently tested by the `tests/Feature/GraphQL/FilterTest.php` feature test and test cases in other GraphQL type tests.  Those tests test basic behavior against a live database, but don't make assertions about the schema changes these directives cause.  This PR adds unit tests which more precisely test what these directives actually do.
@williamjallen

Copy link
Copy Markdown
Member Author

Needs test updates due to a logical conflict the merge queue caught

@williamjallen
williamjallen force-pushed the custom-graphql-directive-unit-tests branch from 60fca77 to 25b39e0 Compare October 28, 2026 14:37
@josephsnyder
josephsnyder added this pull request to the merge queue Oct 28, 2026
Merged via the queue into Kitware:master with commit 0103fb8 Oct 28, 2026
8 checks passed
@williamjallen
williamjallen deleted the custom-graphql-directive-unit-tests branch October 28, 2026 19:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants