Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: add tests by integrating mws + vitest #143

Closed
wants to merge 36 commits into from
Closed

Conversation

alessbell
Copy link
Member

Closes #53.

Only ~30 min into this work :) but opening the PR for visibility. I'm using this process to dogfood the msw + Apollo Client testing experience as I explore AC devtools capabilities for generating mocks.

@codesandbox
Copy link

codesandbox bot commented Aug 28, 2023

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@netlify
Copy link

netlify bot commented Aug 28, 2023

Deploy Preview for comforting-syrniki-15960f failed.

Name Link
🔨 Latest commit 5bd6764
🔍 Latest deploy log https://app.netlify.com/sites/comforting-syrniki-15960f/deploys/651c57df9e920500074231ea

michael-watson and others added 27 commits August 29, 2023 15:41
Add Persisted Queries to router and actions
Fix FeaturedPlaylistConnection resolvers - return array
Enable Subscriptions for latest router version
Bump Apollo Server to be same package
This PR adds a couple of QoL tools to the router via rhai.
It automatically logs subgraph request headers and bodies, as well as subgraph responses headers and bodies.

It also adds authorization header forwarding for subscription payloads.
…rs_forwarding

Add a couple of QoL tools to debug the router.
@alessbell
Copy link
Member Author

I've started iterating on a testing utility elsewhere, and while this was a great branch to begin figuring out the high level direction, it's not quite ready yet. Closing for now.

@alessbell alessbell closed this Nov 7, 2023
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.

Add some tests for the app
5 participants