Skip to content

Conversation

@anmolbhadoriya5849
Copy link

Removed valid test case for additionalItems schema validation.
Fixes #768

Removed valid test case for additionalItems schema validation.
@anmolbhadoriya5849 anmolbhadoriya5849 requested a review from a team as a code owner November 24, 2025 09:08
Copy link
Member

@jdesrosiers jdesrosiers left a comment

Choose a reason for hiding this comment

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

Thanks for picking this us. Please make the change for the other drafts as well.

Removed a valid test case for additionalItems schema validation.
Removed test case for additionalItems not looking in applicators.
Removed a valid test case for additionalItems schema validation.
Removed a valid test case for additionalItems schema validation.
Removed test case for additionalProperties not examining applicators.
Removed test case for additionalProperties not considering applicators.
@anmolbhadoriya5849
Copy link
Author

I’ve updated all drafts and removed the corresponding test cases as requested.
Please take another look when you have time.

Copy link
Member

@jdesrosiers jdesrosiers left a comment

Choose a reason for hiding this comment

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

You can undo the changes in 2020-12 and v1. Those are a different tests for additionalProperties, not additionalItems. additionalItems was removed in 2020-12, so this test doesn't appear after that.

@anmolbhadoriya5849
Copy link
Author

Reverted the incorrect removals.

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.

description of additionalItems test case is insufficient / misleading

2 participants