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

chore(docs): add useStaticQuery to docs for mocking Gatsby #12526

Merged
merged 1 commit into from
Mar 12, 2019
Merged

chore(docs): add useStaticQuery to docs for mocking Gatsby #12526

merged 1 commit into from
Mar 12, 2019

Conversation

entorenee
Copy link
Contributor

Description

Adds useStaticQuery to the docs for mocking out Gatsby related to unit testing.

@entorenee entorenee requested a review from a team March 12, 2019 21:02
@DSchau DSchau changed the title Add useStaticQuery to docs for mocking Gatsby chore(docs): add useStaticQuery to docs for mocking Gatsby Mar 12, 2019
Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

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

This is great--thanks!

Thinking it's getting pretty high-time to make this a package, similar to what we do with cypress-gatsby.

It'd be a suite of utils and mocks that make working with core Gatsby APIs even easier from a unit testing perspective.

I'll open an issue on this tomorrow with some of my thoughts!

@DSchau DSchau merged commit a5dbe7e into gatsbyjs:master Mar 12, 2019
@entorenee
Copy link
Contributor Author

That sounds really intriguing and curious to see the discussion on it. Experimenting with some higher level integration tests which may have multiple sub components that are using StaticQuery. I also wonder if that may be a place of either increased documentation and/or related to this package idea.

@entorenee entorenee deleted the docs/add-use-static-query branch March 12, 2019 21:05
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