Skip to content

test(integration): GraphQL query tests (port legacy 19)#731

Merged
kriszyp merged 2 commits into
mainfrom
claude/api-tests-graphql
May 23, 2026
Merged

test(integration): GraphQL query tests (port legacy 19)#731
kriszyp merged 2 commits into
mainfrom
claude/api-tests-graphql

Conversation

@kriszyp
Copy link
Copy Markdown
Member

@kriszyp kriszyp commented May 22, 2026

Summary

  • Ports apiTests/tests/19_graphQlTests.mjs to the new self-contained framework
  • Installs the full appGraphQL component schema (Related, SubObject, ManyToMany, FourProp, etc.) with installAppComponent and graphql: true config
  • Seeds 5 Related rows and 6 SubObject rows (id 0–5, including one null any)
  • Covers all 20 query patterns from the legacy file: shorthand, named, operationName, multi-resource, primary key args, nullable/non-null variables with and without defaults, nested attribute filters (single and double depth), variable injection at sub-level and top-level, and all fragment forms (top-level, nested, multi-resource, inline, per-field)

Test plan

  • CI passes on node shards (linux + macOS)
  • Suite is skipped on Windows (depends on restart_service http_workers)
  • All 20 GraphQL query tests pass

🤖 Generated with Claude Code

— Claude

Self-contained suite that installs the appGraphQL component (Related +
SubObject with @relationship), seeds 5 Related + 6 SubObject rows, then
exercises shorthand, named, multi-resource, variable, nested-attribute,
and fragment-based GraphQL queries against /graphql.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kriszyp kriszyp requested a review from Ethan-Arrowood May 22, 2026 13:01
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented May 22, 2026

Reviewed; no blockers found.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kriszyp kriszyp merged commit 6ee08f2 into main May 23, 2026
37 checks passed
@kriszyp kriszyp deleted the claude/api-tests-graphql branch May 23, 2026 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants