Skip to content

fix: pass optionLoc when validating plugin object#10402

Merged
nicolo-ribaudo merged 1 commit intobabel:masterfrom
JLHwung:fix-9877
Sep 11, 2019
Merged

fix: pass optionLoc when validating plugin object#10402
nicolo-ribaudo merged 1 commit intobabel:masterfrom
JLHwung:fix-9877

Conversation

@JLHwung
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung commented Sep 6, 2019

Q                       A
Fixed Issues? Fixes #9877
Patch: Bug Fix? Yes
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

Construct option loc and pass it to the validator so that it can print helpful messages on validation error.

@JLHwung JLHwung added pkg: core PR: Bug Fix 🐛 A type of pull request used for our changelog categories labels Sep 6, 2019
};

expect(() => loadConfig(opts)).toThrow(
/\.inhertis must be a function, or undefined/,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What is the current message?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Quotes from #9877

"Assertion failure: Unknown type undefined" error instead of the proper error; "inherits must be a function, or undefined"

@babel-bot
Copy link
Copy Markdown
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/11498/

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue pkg: core PR: Bug Fix 🐛 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Invalid error message when validating plugin options

4 participants