diff options
author | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2022-01-18 17:22:21 +0100 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@verizonmedia.com> | 2022-01-18 17:50:40 +0100 |
commit | 3a19facd01e844a4dc87d720165c28548bce7fcf (patch) | |
tree | 345a1bd23f42e933e6a00456649a66fc71c718ac /config-model/src/test/java | |
parent | 8ca62417e9c3b1bb2887226054069fb4c346c835 (diff) |
Support additional model validators
Inject additional model validators through VespaModelFactory.
Useful for adding validators that are only relevant for hosted and cannot be open-sourced.
Diffstat (limited to 'config-model/src/test/java')
5 files changed, 7 insertions, 7 deletions
diff --git a/config-model/src/test/java/com/yahoo/vespa/model/application/validation/ComplexAttributeFieldsValidatorTestCase.java b/config-model/src/test/java/com/yahoo/vespa/model/application/validation/ComplexAttributeFieldsValidatorTestCase.java index 689f1c70b6e..85050aa0cf9 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/application/validation/ComplexAttributeFieldsValidatorTestCase.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/application/validation/ComplexAttributeFieldsValidatorTestCase.java @@ -105,7 +105,7 @@ public class ComplexAttributeFieldsValidatorTestCase { DeployState deployState = createDeployState(servicesXml(), schema); VespaModel model = new VespaModel(new NullConfigModelRegistry(), deployState); ValidationParameters validationParameters = new ValidationParameters(CheckRouting.FALSE); - Validation.validate(model, validationParameters, deployState); + new Validation().validate(model, validationParameters, deployState); } private static DeployState createDeployState(String servicesXml, String schema) { diff --git a/config-model/src/test/java/com/yahoo/vespa/model/application/validation/ValidationOverridesValidatorTest.java b/config-model/src/test/java/com/yahoo/vespa/model/application/validation/ValidationOverridesValidatorTest.java index 16bff341c0b..93d42e07dd8 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/application/validation/ValidationOverridesValidatorTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/application/validation/ValidationOverridesValidatorTest.java @@ -37,7 +37,7 @@ public class ValidationOverridesValidatorTest { var deployState = createDeployState(validationOverridesXml); VespaModel model = new VespaModel(new NullConfigModelRegistry(), deployState); - Validation.validate(model, new ValidationParameters(), deployState); + new Validation().validate(model, new ValidationParameters(), deployState); } @Test @@ -58,7 +58,7 @@ public class ValidationOverridesValidatorTest { try { var deployState = createDeployState(validationOverridesXml); VespaModel model = new VespaModel(new NullConfigModelRegistry(), deployState); - Validation.validate(model, new ValidationParameters(), deployState); + new Validation().validate(model, new ValidationParameters(), deployState); fail("Did not get expected exception"); } catch (IllegalArgumentException e) { assertEquals(message, e.getMessage()); diff --git a/config-model/src/test/java/com/yahoo/vespa/model/test/VespaModelTestCase.java b/config-model/src/test/java/com/yahoo/vespa/model/test/VespaModelTestCase.java index bf4d066e454..7f8bca825d2 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/test/VespaModelTestCase.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/test/VespaModelTestCase.java @@ -242,7 +242,7 @@ public class VespaModelTestCase { .build(); DeployState deployState = builder.deployLogger(logger).applicationPackage(app).build(); VespaModel model = new VespaModel(new NullConfigModelRegistry(), deployState); - Validation.validate(model, new ValidationParameters(ValidationParameters.IgnoreValidationErrors.TRUE), deployState); + new Validation().validate(model, new ValidationParameters(ValidationParameters.IgnoreValidationErrors.TRUE), deployState); assertFalse(logger.msgs.isEmpty()); } @@ -312,7 +312,7 @@ public class VespaModelTestCase { .deployLogger(logger) .build(); VespaModel model = new VespaModel(new NullConfigModelRegistry(), deployState); - Validation.validate(model, new ValidationParameters(), deployState); + new Validation().validate(model, new ValidationParameters(), deployState); assertContainsWarning(logger.msgs, "Directory searchdefinitions/ should not be used for schemas, use schemas/ instead"); } diff --git a/config-model/src/test/java/com/yahoo/vespa/model/test/utils/VespaModelCreatorWithFilePkg.java b/config-model/src/test/java/com/yahoo/vespa/model/test/utils/VespaModelCreatorWithFilePkg.java index 351b080eb5b..565e4c7c076 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/test/utils/VespaModelCreatorWithFilePkg.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/test/utils/VespaModelCreatorWithFilePkg.java @@ -66,7 +66,7 @@ public class VespaModelCreatorWithFilePkg { // is constructed in a special way and cannot always be validated in // this step for unit tests) ValidationParameters validationParameters = new ValidationParameters(IgnoreValidationErrors.TRUE, FailOnIncompatibleChange.TRUE, CheckRouting.FALSE); - Validation.validate(model, validationParameters, deployState); + new Validation().validate(model, validationParameters, deployState); return model; } catch (Exception e) { throw e instanceof RuntimeException ? (RuntimeException) e : new RuntimeException(e); diff --git a/config-model/src/test/java/com/yahoo/vespa/model/test/utils/VespaModelCreatorWithMockPkg.java b/config-model/src/test/java/com/yahoo/vespa/model/test/utils/VespaModelCreatorWithMockPkg.java index 66dc63bbb02..d777fefbbe5 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/test/utils/VespaModelCreatorWithMockPkg.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/test/utils/VespaModelCreatorWithMockPkg.java @@ -75,7 +75,7 @@ public class VespaModelCreatorWithMockPkg { // is constructed in a special way and cannot always be validated in // this step for unit tests) ValidationParameters validationParameters = new ValidationParameters(CheckRouting.FALSE); - configChangeActions = Validation.validate(model, validationParameters, deployState); + configChangeActions = new Validation().validate(model, validationParameters, deployState); } return model; } catch (Exception e) { |