fix: improve error messages #1536
Conversation
|
|
| validating /openapi.yaml... | ||
| Something went wrong when processing /Users/andrewtatomyr/Tests/redocly-cli-backup/__tests__/lint/assertions-property-key-not-allowed/openapi.yaml: | ||
|
|
||
| - The 'disallowed' assertion can't be used on properties. Please remove the 'property' key. |
There was a problem hiding this comment.
Currently the error is:
- disallowed can't be used on a single property. Please use 'property'..
| validating /openapi.yaml... | ||
| Something went wrong when processing /Users/andrewtatomyr/Tests/redocly-cli-backup/__tests__/lint/assertions-property-key-required/openapi.yaml: | ||
|
|
||
| - The 'nonEmpty' assertion can't be used on all keys. Please provide a single property. |
There was a problem hiding this comment.
The current error is:
- nonEmpty can't be used on all keys. Please provide a single property.
Coverage report
Test suite run success734 tests passing in 102 suites. Report generated by 🧪jest coverage report action from ed380df |
tatomyr
force-pushed
the
fix/assertion-error-messages
branch
from
16f1e46
to
ed380df
Compare
What/Why/How?
When an assertion is supposed to run only on object keys and the assertion subject contains the
propertykey, the error is- <the assertion name> can't be used on a single property. Please use 'property'..which should be the other way around (thanks @adamaltman for pointing on that).This PR fixes that, and also cleans up the error messages by removing needless periods and using better formatting.
Reference
Testing
Screenshots (optional)
Check yourself
Security