Skip to content
New issue

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

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

Already on GitHub? # to your account

tests: Update pluginOptionsSchema tests #27904

Merged

Conversation

MichaelDeBoey
Copy link
Contributor

Bring tests in line with the docs

@MichaelDeBoey MichaelDeBoey requested review from a team as code owners November 9, 2020 01:48
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Nov 9, 2020
@laurieontech laurieontech added topic: plugins Related to plugin system, themes & catch-all for plugins that don't have a label and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Nov 9, 2020
@laurieontech
Copy link
Contributor

Thanks for the PR! I certainly agree with the docs update, I'm less convinced that we need to look at both isValid and errors in all scenarios. Though we certainly can. cc: @mxstbr for his take.

mxstbr
mxstbr previously approved these changes Nov 13, 2020
Copy link
Contributor

@mxstbr mxstbr left a comment

Choose a reason for hiding this comment

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

This seems like a good idea to me, as it showcases the full API of testPluginOptionsSchema for people who are looking at our usage of it! Thank you @MichaelDeBoey!! 👍

@MichaelDeBoey
Copy link
Contributor Author

@laurieontech @mxstbr I think this one can be merged, no? 🤔

@ascorbic
Copy link
Contributor

@MichaelDeBoey Max and Laurie no longer work at Gatsby, but I can look at this. The tests are failing now. Are you able to see why?

@MichaelDeBoey
Copy link
Contributor Author

@ascorbic Tests should be fine now

@LekoArts LekoArts merged commit 2d967cb into gatsbyjs:master Nov 23, 2022
@MichaelDeBoey MichaelDeBoey deleted the update-plugin-validation-tests branch November 23, 2022 18:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
topic: plugins Related to plugin system, themes & catch-all for plugins that don't have a label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants