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

Embedded validator spec improvements #155

Merged
merged 3 commits into from
Feb 27, 2022
Merged

Embedded validator spec improvements #155

merged 3 commits into from
Feb 27, 2022

Conversation

Blacksmoke16
Copy link
Member

@Blacksmoke16 Blacksmoke16 commented Feb 26, 2022

Improves specs for constraint validators who leverage the validator provided off the context.

  • Remove reliance on ivar validator within ISIN
  • Make MockValidator more robust
  • Enable some additional AtLeastOneOf specs

Resolves #142

@Blacksmoke16 Blacksmoke16 added component:validator kind:enhancement New functionality to an existing feature labels Feb 26, 2022
Make `MockValidator` more robust
Enable some additional `AtLeastOneOf` specs
Some cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:validator kind:enhancement New functionality to an existing feature
Development

Successfully merging this pull request may close these issues.

AVD::Spec::ConstraintValidatorTestCase cannot handle constraints that require a validator from context
1 participant