Move validation classes to clamp models package 84/143384/2 master
authordanielhanrahan <daniel.hanrahan@est.tech>
Thu, 19 Feb 2026 17:30:45 +0000 (17:30 +0000)
committerdanielhanrahan <daniel.hanrahan@est.tech>
Tue, 24 Feb 2026 15:13:39 +0000 (15:13 +0000)
This commit moves classes such as PfAuthorative from policy-models
into models module, closer to where there are used. This will allow
easier refactoring in future.

Issue-ID: POLICY-5509
Change-Id: I8f0b00ea907855d9683f9ac0bbb7a1b2a98d0c3b
Signed-off-by: danielhanrahan <daniel.hanrahan@est.tech>
45 files changed:
models/src/main/java/org/onap/policy/clamp/models/acm/base/PfAuthorative.java [moved from policy-models/src/main/java/org/onap/policy/models/base/PfAuthorative.java with 97% similarity]
models/src/main/java/org/onap/policy/clamp/models/acm/base/validation/BeanValidationResult.java [moved from policy-common/src/main/java/org/onap/policy/common/parameters/BeanValidationResult.java with 98% similarity]
models/src/main/java/org/onap/policy/clamp/models/acm/base/validation/BeanValidator.java [moved from policy-common/src/main/java/org/onap/policy/common/parameters/BeanValidator.java with 97% similarity]
models/src/main/java/org/onap/policy/clamp/models/acm/base/validation/ObjectValidationResult.java [moved from policy-common/src/main/java/org/onap/policy/common/parameters/ObjectValidationResult.java with 97% similarity]
models/src/main/java/org/onap/policy/clamp/models/acm/base/validation/ValidationResult.java [moved from policy-common/src/main/java/org/onap/policy/common/parameters/ValidationResult.java with 97% similarity]
models/src/main/java/org/onap/policy/clamp/models/acm/base/validation/ValidationResultImpl.java [moved from policy-common/src/main/java/org/onap/policy/common/parameters/ValidationResultImpl.java with 98% similarity]
models/src/main/java/org/onap/policy/clamp/models/acm/base/validation/ValidationStatus.java [moved from policy-common/src/main/java/org/onap/policy/common/parameters/ValidationStatus.java with 96% similarity]
models/src/main/java/org/onap/policy/clamp/models/acm/base/validation/annotations/VerifyKey.java [moved from policy-models/src/main/java/org/onap/policy/models/base/validation/annotations/VerifyKey.java with 97% similarity]
models/src/main/java/org/onap/policy/clamp/models/acm/base/validation/annotations/VerifyKeyValidator.java [moved from policy-models/src/main/java/org/onap/policy/models/base/validation/annotations/VerifyKeyValidator.java with 97% similarity]
models/src/main/java/org/onap/policy/clamp/models/acm/document/base/ToscaServiceTemplateValidation.java
models/src/main/java/org/onap/policy/clamp/models/acm/document/concepts/DocToscaCapabilityType.java
models/src/main/java/org/onap/policy/clamp/models/acm/document/concepts/DocToscaConstraint.java
models/src/main/java/org/onap/policy/clamp/models/acm/document/concepts/DocToscaEntity.java
models/src/main/java/org/onap/policy/clamp/models/acm/document/concepts/DocToscaParameter.java
models/src/main/java/org/onap/policy/clamp/models/acm/document/concepts/DocToscaPolicy.java
models/src/main/java/org/onap/policy/clamp/models/acm/document/concepts/DocToscaProperty.java
models/src/main/java/org/onap/policy/clamp/models/acm/document/concepts/DocToscaSchemaDefinition.java
models/src/main/java/org/onap/policy/clamp/models/acm/document/concepts/DocToscaTopologyTemplate.java
models/src/main/java/org/onap/policy/clamp/models/acm/messages/kafka/participant/PropertiesUpdate.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaAutomationComposition.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaAutomationCompositionDefinition.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaAutomationCompositionElement.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaAutomationCompositionRollback.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaMessage.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaNodeTemplateState.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaParticipant.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaParticipantReplica.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaParticipantSupportedElementType.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/provider/AcDefinitionProvider.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/provider/AutomationCompositionProvider.java
models/src/main/java/org/onap/policy/clamp/models/acm/persistence/provider/ProviderUtils.java
models/src/main/java/org/onap/policy/clamp/models/acm/utils/AcmUtils.java
models/src/test/java/org/onap/policy/clamp/models/acm/base/annotations/VerifyKeyValidatorTest.java [moved from policy-models/src/test/java/org/onap/policy/models/base/validation/annotations/VerifyKeyValidatorTest.java with 96% similarity]
models/src/test/java/org/onap/policy/clamp/models/acm/base/validation/TestBeanValidationResult.java [moved from policy-common/src/test/java/org/onap/policy/common/parameters/TestBeanValidationResult.java with 98% similarity]
models/src/test/java/org/onap/policy/clamp/models/acm/base/validation/TestObjectValidationResult.java [moved from policy-common/src/test/java/org/onap/policy/common/parameters/TestObjectValidationResult.java with 97% similarity]
models/src/test/java/org/onap/policy/clamp/models/acm/base/validation/TestValidationResultImpl.java [moved from policy-common/src/test/java/org/onap/policy/common/parameters/TestValidationResultImpl.java with 98% similarity]
models/src/test/java/org/onap/policy/clamp/models/acm/document/base/ToscaServiceTemplateValidationTest.java
models/src/test/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaAutomationCompositionElementTest.java
models/src/test/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaAutomationCompositionTest.java
models/src/test/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaMessageJobTest.java
models/src/test/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaMessageTest.java
models/src/test/java/org/onap/policy/clamp/models/acm/persistence/concepts/JpaParticipantTest.java
participant/participant-impl/participant-impl-acelement/src/main/java/org/onap/policy/clamp/acm/element/service/ConfigService.java
policy-common/src/test/java/org/onap/policy/common/message/bus/event/TopicParameterGroupTest.java
runtime-acm/src/main/java/org/onap/policy/clamp/acm/runtime/instantiation/AutomationCompositionInstantiationProvider.java

@@ -18,7 +18,7 @@
  * ============LICENSE_END=========================================================
  */
 
-package org.onap.policy.models.base;
+package org.onap.policy.clamp.models.acm.base;
 
 /**
  * Interface that provides conversion to and from authorative concepts for other concepts.
@@ -19,7 +19,7 @@
  * ============LICENSE_END=========================================================
  */
 
-package org.onap.policy.common.parameters;
+package org.onap.policy.clamp.models.acm.base.validation;
 
 import jakarta.validation.Validation;
 import jakarta.validation.Validator;
@@ -19,7 +19,7 @@
  * ============LICENSE_END=========================================================
  */
 
-package org.onap.policy.common.parameters;
+package org.onap.policy.clamp.models.acm.base.validation;
 
 /**
  * This class holds the result of the validation of an object within a bean.
@@ -19,7 +19,7 @@
  * ============LICENSE_END=========================================================
  */
 
-package org.onap.policy.common.parameters;
+package org.onap.policy.clamp.models.acm.base.validation;
 
 /**
  * This interface defines the result of a parameter validation.
@@ -19,7 +19,7 @@
  * ============LICENSE_END=========================================================
  */
 
-package org.onap.policy.models.base.validation.annotations;
+package org.onap.policy.clamp.models.acm.base.validation.annotations;
 
 import static java.lang.annotation.ElementType.FIELD;
 import static java.lang.annotation.ElementType.METHOD;
@@ -18,7 +18,7 @@
  * ============LICENSE_END=========================================================
  */
 
-package org.onap.policy.models.base.validation.annotations;
+package org.onap.policy.clamp.models.acm.base.validation.annotations;
 
 import jakarta.validation.ConstraintValidator;
 import jakarta.validation.ConstraintValidatorContext;
index d620855..31bf830 100644 (file)
@@ -27,11 +27,11 @@ import java.util.Set;
 import lombok.AccessLevel;
 import lombok.NoArgsConstructor;
 import lombok.NonNull;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidationResult;
+import org.onap.policy.clamp.models.acm.base.validation.ValidationStatus;
 import org.onap.policy.clamp.models.acm.document.concepts.DocToscaEntity;
 import org.onap.policy.clamp.models.acm.document.concepts.DocToscaServiceTemplate;
 import org.onap.policy.clamp.models.acm.document.concepts.DocToscaTopologyTemplate;
-import org.onap.policy.common.parameters.BeanValidationResult;
-import org.onap.policy.common.parameters.ValidationStatus;
 import org.onap.policy.models.base.PfUtils;
 
 @NoArgsConstructor(access = AccessLevel.PRIVATE)
index 87f5735..d88e9b7 100644 (file)
@@ -24,8 +24,8 @@ import java.io.Serial;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
+import org.onap.policy.clamp.models.acm.base.validation.annotations.VerifyKey;
 import org.onap.policy.models.base.PfConceptKey;
-import org.onap.policy.models.base.validation.annotations.VerifyKey;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaCapabilityType;
 
 @Data
index 662b4c5..e09f6ca 100644 (file)
@@ -27,7 +27,7 @@ import java.util.ArrayList;
 import java.util.List;
 import lombok.Data;
 import lombok.NoArgsConstructor;
-import org.onap.policy.models.base.PfAuthorative;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaConstraint;
 
 @Data
index e38f011..9efa16e 100644 (file)
@@ -32,8 +32,8 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 import org.apache.commons.lang3.ObjectUtils;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.clamp.models.acm.document.base.DocConceptKey;
-import org.onap.policy.models.base.PfAuthorative;
 import org.onap.policy.models.base.PfConceptKey;
 import org.onap.policy.models.base.PfKey;
 import org.onap.policy.models.base.PfNameVersion;
index ae5037f..650e18b 100644 (file)
@@ -26,7 +26,7 @@ import java.io.Serial;
 import java.io.Serializable;
 import lombok.Data;
 import lombok.NoArgsConstructor;
-import org.onap.policy.models.base.PfAuthorative;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.models.base.PfKey;
 import org.onap.policy.models.base.PfUtils;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaParameter;
index c85db2f..1d00c9a 100644 (file)
@@ -27,8 +27,8 @@ import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 import lombok.NonNull;
 import lombok.ToString;
+import org.onap.policy.clamp.models.acm.base.validation.annotations.VerifyKey;
 import org.onap.policy.models.base.PfConceptKey;
-import org.onap.policy.models.base.validation.annotations.VerifyKey;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaPolicy;
 
 @Data
index 5a408ba..8cd63e2 100644 (file)
@@ -32,9 +32,9 @@ import java.util.Map;
 import lombok.Data;
 import lombok.NoArgsConstructor;
 import org.apache.commons.lang3.ObjectUtils;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.clamp.models.acm.document.base.DocConceptKey;
 import org.onap.policy.clamp.models.acm.document.base.DocUtil;
-import org.onap.policy.models.base.PfAuthorative;
 import org.onap.policy.models.base.PfUtils;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaProperty;
 
index 1caf272..7938119 100644 (file)
@@ -29,9 +29,9 @@ import java.util.List;
 import lombok.Data;
 import lombok.NoArgsConstructor;
 import org.apache.commons.lang3.ObjectUtils;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.clamp.models.acm.document.base.DocConceptKey;
 import org.onap.policy.clamp.models.acm.document.base.DocUtil;
-import org.onap.policy.models.base.PfAuthorative;
 import org.onap.policy.models.base.PfUtils;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaSchemaDefinition;
 
index bed773d..822157e 100644 (file)
@@ -30,8 +30,8 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 import org.apache.commons.lang3.ObjectUtils;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.clamp.models.acm.document.base.DocUtil;
-import org.onap.policy.models.base.PfAuthorative;
 import org.onap.policy.models.base.PfUtils;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaTopologyTemplate;
 
index e76f4aa..5251ce1 100644 (file)
@@ -28,7 +28,6 @@ import lombok.ToString;
 import org.onap.policy.clamp.models.acm.concepts.ParticipantDeploy;
 import org.onap.policy.models.base.PfUtils;
 
-
 /**
  * Class to represent the PROPERTIES_UPDATE message that the ACM runtime sends to a participant.
  * The ACM Runtime sends updated automation composition element property values to Participants.
index fa6be7e..7a71b9e 100644 (file)
@@ -44,13 +44,13 @@ import lombok.EqualsAndHashCode;
 import lombok.NoArgsConstructor;
 import lombok.NonNull;
 import org.apache.commons.lang3.ObjectUtils;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.clamp.models.acm.concepts.AutomationComposition;
 import org.onap.policy.clamp.models.acm.concepts.DeployState;
 import org.onap.policy.clamp.models.acm.concepts.LockState;
 import org.onap.policy.clamp.models.acm.concepts.StateChangeResult;
 import org.onap.policy.clamp.models.acm.concepts.SubState;
 import org.onap.policy.clamp.models.acm.utils.TimestampHelper;
-import org.onap.policy.models.base.PfAuthorative;
 import org.onap.policy.models.base.PfUtils;
 
 /**
index c21c498..e53bb52 100644 (file)
@@ -41,12 +41,12 @@ import java.util.Set;
 import java.util.UUID;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.clamp.models.acm.concepts.AcTypeState;
 import org.onap.policy.clamp.models.acm.concepts.AutomationCompositionDefinition;
 import org.onap.policy.clamp.models.acm.concepts.StateChangeResult;
 import org.onap.policy.clamp.models.acm.document.concepts.DocToscaServiceTemplate;
 import org.onap.policy.clamp.models.acm.utils.TimestampHelper;
-import org.onap.policy.models.base.PfAuthorative;
 import org.onap.policy.models.base.PfKey;
 
 /**
index 58668b0..1784865 100644 (file)
@@ -39,15 +39,15 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NonNull;
 import org.apache.commons.lang3.ObjectUtils;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
+import org.onap.policy.clamp.models.acm.base.validation.annotations.VerifyKey;
 import org.onap.policy.clamp.models.acm.concepts.AutomationCompositionElement;
 import org.onap.policy.clamp.models.acm.concepts.DeployState;
 import org.onap.policy.clamp.models.acm.concepts.LockState;
 import org.onap.policy.clamp.models.acm.concepts.MigrationState;
 import org.onap.policy.clamp.models.acm.concepts.SubState;
-import org.onap.policy.models.base.PfAuthorative;
 import org.onap.policy.models.base.PfConceptKey;
 import org.onap.policy.models.base.PfUtils;
-import org.onap.policy.models.base.validation.annotations.VerifyKey;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaConceptIdentifier;
 
 /**
index 229501c..d811a3f 100644 (file)
@@ -38,9 +38,9 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;\r
 import lombok.NonNull;\r
 import org.apache.commons.lang3.ObjectUtils;\r
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;\r
 import org.onap.policy.clamp.models.acm.concepts.AutomationCompositionElement;\r
 import org.onap.policy.clamp.models.acm.concepts.AutomationCompositionRollback;\r
-import org.onap.policy.models.base.PfAuthorative;\r
 import org.onap.policy.models.base.PfUtils;\r
 \r
 @Entity\r
index 2eed79f..b7e944f 100644 (file)
@@ -33,9 +33,9 @@ import java.sql.Timestamp;
 import java.util.UUID;
 import lombok.Data;
 import lombok.NonNull;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.clamp.models.acm.document.concepts.DocMessage;
 import org.onap.policy.clamp.models.acm.utils.TimestampHelper;
-import org.onap.policy.models.base.PfAuthorative;
 
 @Entity
 @Table(name = "Message", indexes = {@Index(name = "m_identificationId", columnList = "identificationId")})
index 01b7876..9a5c3e1 100644 (file)
@@ -35,12 +35,12 @@ import java.util.UUID;
 import java.util.function.UnaryOperator;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
+import org.onap.policy.clamp.models.acm.base.validation.annotations.VerifyKey;
 import org.onap.policy.clamp.models.acm.concepts.AcTypeState;
 import org.onap.policy.clamp.models.acm.concepts.NodeTemplateState;
-import org.onap.policy.models.base.PfAuthorative;
 import org.onap.policy.models.base.PfConceptKey;
 import org.onap.policy.models.base.PfUtils;
-import org.onap.policy.models.base.validation.annotations.VerifyKey;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaConceptIdentifier;
 
 @Entity
index 87afa17..07ba4de 100644 (file)
@@ -41,8 +41,8 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NonNull;
 import org.apache.commons.lang3.ObjectUtils;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.clamp.models.acm.concepts.Participant;
-import org.onap.policy.models.base.PfAuthorative;
 
 /**
  * Class to represent a participant in the database.
index e203799..e31c011 100644 (file)
@@ -32,10 +32,10 @@ import java.util.UUID;
 import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NonNull;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.clamp.models.acm.concepts.ParticipantReplica;
 import org.onap.policy.clamp.models.acm.concepts.ParticipantState;
 import org.onap.policy.clamp.models.acm.utils.TimestampHelper;
-import org.onap.policy.models.base.PfAuthorative;
 
 @Entity
 @Table(name = "ParticipantReplica")
index 75dc9bc..0588689 100644 (file)
@@ -32,8 +32,8 @@ import lombok.Data;
 import lombok.EqualsAndHashCode;
 import lombok.NonNull;
 import org.apache.commons.lang3.ObjectUtils;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
 import org.onap.policy.clamp.models.acm.concepts.ParticipantSupportedElementType;
-import org.onap.policy.models.base.PfAuthorative;
 
 @Entity
 @Table(name = "ParticipantSupportedAcElements")
index accae2e..be2ac57 100644 (file)
@@ -29,6 +29,7 @@ import java.util.UUID;
 import java.util.stream.Collectors;
 import lombok.NonNull;
 import lombok.RequiredArgsConstructor;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidationResult;
 import org.onap.policy.clamp.models.acm.concepts.AcTypeState;
 import org.onap.policy.clamp.models.acm.concepts.AutomationCompositionDefinition;
 import org.onap.policy.clamp.models.acm.document.base.ToscaServiceTemplateValidation;
@@ -37,7 +38,6 @@ import org.onap.policy.clamp.models.acm.persistence.concepts.JpaAutomationCompos
 import org.onap.policy.clamp.models.acm.persistence.repository.AutomationCompositionDefinitionRepository;
 import org.onap.policy.clamp.models.acm.utils.AcmUtils;
 import org.onap.policy.clamp.models.acm.utils.TimestampHelper;
-import org.onap.policy.common.parameters.BeanValidationResult;
 import org.onap.policy.models.base.PfModelRuntimeException;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaServiceTemplate;
 import org.springframework.data.domain.Example;
index b238c5e..a38545f 100644 (file)
@@ -31,6 +31,8 @@ import java.util.UUID;
 import java.util.stream.Collectors;
 import lombok.AllArgsConstructor;
 import lombok.NonNull;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidationResult;
+import org.onap.policy.clamp.models.acm.base.validation.ValidationStatus;
 import org.onap.policy.clamp.models.acm.concepts.AutomationComposition;
 import org.onap.policy.clamp.models.acm.concepts.AutomationCompositionElement;
 import org.onap.policy.clamp.models.acm.concepts.AutomationCompositionRollback;
@@ -44,8 +46,6 @@ import org.onap.policy.clamp.models.acm.persistence.repository.AutomationComposi
 import org.onap.policy.clamp.models.acm.persistence.repository.AutomationCompositionRepository;
 import org.onap.policy.clamp.models.acm.persistence.repository.AutomationCompositionRollbackRepository;
 import org.onap.policy.clamp.models.acm.utils.AcmStateUtils;
-import org.onap.policy.common.parameters.BeanValidationResult;
-import org.onap.policy.common.parameters.ValidationStatus;
 import org.onap.policy.models.base.PfConceptKey;
 import org.onap.policy.models.base.PfKey;
 import org.onap.policy.models.base.PfModelRuntimeException;
index d7c6061..c6a28e5 100644 (file)
@@ -26,9 +26,9 @@ import java.util.List;
 import java.util.function.Supplier;
 import lombok.AccessLevel;
 import lombok.NoArgsConstructor;
-import org.onap.policy.common.parameters.BeanValidationResult;
-import org.onap.policy.common.parameters.BeanValidator;
-import org.onap.policy.models.base.PfAuthorative;
+import org.onap.policy.clamp.models.acm.base.PfAuthorative;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidationResult;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidator;
 import org.onap.policy.models.base.PfModelRuntimeException;
 
 @NoArgsConstructor(access = AccessLevel.PRIVATE)
index 8e370ab..208e925 100644 (file)
@@ -38,6 +38,10 @@ import lombok.NoArgsConstructor;
 import org.apache.commons.collections4.CollectionUtils;
 import org.apache.commons.collections4.MapUtils;
 import org.apache.commons.lang3.tuple.Pair;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidationResult;
+import org.onap.policy.clamp.models.acm.base.validation.ObjectValidationResult;
+import org.onap.policy.clamp.models.acm.base.validation.ValidationResult;
+import org.onap.policy.clamp.models.acm.base.validation.ValidationStatus;
 import org.onap.policy.clamp.models.acm.concepts.AcElementDeploy;
 import org.onap.policy.clamp.models.acm.concepts.AcElementRestart;
 import org.onap.policy.clamp.models.acm.concepts.AcTypeState;
@@ -52,10 +56,6 @@ import org.onap.policy.clamp.models.acm.concepts.ParticipantDeploy;
 import org.onap.policy.clamp.models.acm.concepts.ParticipantRestartAc;
 import org.onap.policy.clamp.models.acm.messages.rest.instantiation.DeployOrder;
 import org.onap.policy.clamp.models.acm.persistence.concepts.StringToMapConverter;
-import org.onap.policy.common.parameters.BeanValidationResult;
-import org.onap.policy.common.parameters.ObjectValidationResult;
-import org.onap.policy.common.parameters.ValidationResult;
-import org.onap.policy.common.parameters.ValidationStatus;
 import org.onap.policy.models.base.PfModelRuntimeException;
 import org.onap.policy.models.base.PfUtils;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaConceptIdentifier;
@@ -18,7 +18,7 @@
  * ============LICENSE_END=========================================================
  */
 
-package org.onap.policy.models.base.validation.annotations;
+package org.onap.policy.clamp.models.acm.base.annotations;
 
 import static org.assertj.core.api.Assertions.assertThat;
 
@@ -26,7 +26,8 @@ import jakarta.validation.Valid;
 import jakarta.validation.constraints.NotNull;
 import lombok.Getter;
 import org.junit.jupiter.api.Test;
-import org.onap.policy.common.parameters.BeanValidator;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidator;
+import org.onap.policy.clamp.models.acm.base.validation.annotations.VerifyKey;
 import org.onap.policy.models.base.PfConceptKey;
 import org.onap.policy.models.base.PfKey;
 
@@ -19,7 +19,7 @@
  * ============LICENSE_END=========================================================
  */
 
-package org.onap.policy.common.parameters;
+package org.onap.policy.clamp.models.acm.base.validation;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.junit.jupiter.api.Assertions.assertEquals;
@@ -19,7 +19,7 @@
  * ============LICENSE_END=========================================================
  */
 
-package org.onap.policy.common.parameters;
+package org.onap.policy.clamp.models.acm.base.validation;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
@@ -19,7 +19,7 @@
  * ============LICENSE_END=========================================================
  */
 
-package org.onap.policy.common.parameters;
+package org.onap.policy.clamp.models.acm.base.validation;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
index 1568af1..9a6e5f1 100644 (file)
@@ -26,6 +26,7 @@ import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import java.util.List;
 import java.util.Map;
 import org.junit.jupiter.api.Test;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidationResult;
 import org.onap.policy.clamp.models.acm.document.concepts.DocToscaDataType;
 import org.onap.policy.clamp.models.acm.document.concepts.DocToscaNodeTemplate;
 import org.onap.policy.clamp.models.acm.document.concepts.DocToscaPolicy;
@@ -33,7 +34,6 @@ import org.onap.policy.clamp.models.acm.document.concepts.DocToscaProperty;
 import org.onap.policy.clamp.models.acm.document.concepts.DocToscaRequirement;
 import org.onap.policy.clamp.models.acm.document.concepts.DocToscaServiceTemplate;
 import org.onap.policy.clamp.models.acm.document.concepts.DocToscaTopologyTemplate;
-import org.onap.policy.common.parameters.BeanValidationResult;
 
 class ToscaServiceTemplateValidationTest {
 
index a9b69f8..3081070 100644 (file)
@@ -30,13 +30,13 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 import java.util.Map;
 import java.util.UUID;
 import org.junit.jupiter.api.Test;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidator;
 import org.onap.policy.clamp.models.acm.concepts.AutomationCompositionElement;
 import org.onap.policy.clamp.models.acm.concepts.DeployState;
 import org.onap.policy.clamp.models.acm.concepts.LockState;
 import org.onap.policy.clamp.models.acm.concepts.MigrationState;
 import org.onap.policy.clamp.models.acm.concepts.SubState;
 import org.onap.policy.clamp.models.acm.utils.CommonTestData;
-import org.onap.policy.common.parameters.BeanValidator;
 import org.onap.policy.models.base.PfConceptKey;
 import org.onap.policy.models.tosca.authorative.concepts.ToscaConceptIdentifier;
 
index 8a386c8..9f32217 100644 (file)
@@ -32,13 +32,13 @@ import java.util.ArrayList;
 import java.util.LinkedHashMap;
 import java.util.UUID;
 import org.junit.jupiter.api.Test;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidator;
 import org.onap.policy.clamp.models.acm.concepts.AutomationComposition;
 import org.onap.policy.clamp.models.acm.concepts.DeployState;
 import org.onap.policy.clamp.models.acm.concepts.LockState;
 import org.onap.policy.clamp.models.acm.concepts.StateChangeResult;
 import org.onap.policy.clamp.models.acm.concepts.SubState;
 import org.onap.policy.clamp.models.acm.utils.TimestampHelper;
-import org.onap.policy.common.parameters.BeanValidator;
 
 /**
  * Test the{@link JpaAutomationCompositionTest} class.
index e174075..1e8106f 100644 (file)
@@ -25,7 +25,7 @@ import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import org.junit.jupiter.api.Test;
-import org.onap.policy.common.parameters.BeanValidator;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidator;
 
 class JpaMessageJobTest {
 
index dfc76bc..34c66f2 100644 (file)
@@ -27,10 +27,10 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.util.UUID;
 import org.junit.jupiter.api.Test;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidator;
 import org.onap.policy.clamp.models.acm.concepts.DeployState;
 import org.onap.policy.clamp.models.acm.document.concepts.DocMessage;
 import org.onap.policy.clamp.models.acm.messages.kafka.participant.ParticipantMessageType;
-import org.onap.policy.common.parameters.BeanValidator;
 
 class JpaMessageTest {
 
index 7ef721e..d49018c 100644 (file)
@@ -31,8 +31,8 @@ import java.util.ArrayList;
 import java.util.LinkedHashMap;
 import java.util.UUID;
 import org.junit.jupiter.api.Test;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidator;
 import org.onap.policy.clamp.models.acm.concepts.Participant;
-import org.onap.policy.common.parameters.BeanValidator;
 
 /**
  * Test the {@link JpaParticipant} class.
index e7bacb0..676f3ff 100644 (file)
@@ -30,7 +30,7 @@ import org.onap.policy.clamp.acm.element.handler.MessageHandler;
 import org.onap.policy.clamp.acm.element.main.concepts.ElementConfig;
 import org.onap.policy.clamp.acm.element.main.parameters.ElementTopicParameters;
 import org.onap.policy.clamp.common.acm.exception.AutomationCompositionRuntimeException;
-import org.onap.policy.common.parameters.BeanValidator;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidator;
 import org.onap.policy.common.parameters.topic.TopicParameterGroup;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
index e50d78f..863b413 100644 (file)
 
 package org.onap.policy.common.message.bus.event;
 
+import static org.assertj.core.api.Assertions.assertThat;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
+import jakarta.validation.Validation;
+import jakarta.validation.Validator;
+import jakarta.validation.ValidatorFactory;
 import java.beans.PropertyDescriptor;
 import java.lang.reflect.Field;
 import java.lang.reflect.Modifier;
 import java.util.List;
 import org.apache.commons.lang3.StringUtils;
 import org.junit.jupiter.api.Test;
-import org.onap.policy.common.parameters.BeanValidator;
-import org.onap.policy.common.parameters.ValidationResult;
 import org.onap.policy.common.parameters.topic.BusTopicParams;
 import org.onap.policy.common.parameters.topic.TopicParameterGroup;
 import org.onap.policy.common.parameters.topic.TopicParameters;
@@ -48,13 +50,15 @@ import org.onap.policy.common.utils.coder.StandardCoder;
 class TopicParameterGroupTest {
     private static final CommonTestData testData = new CommonTestData();
     private static final Coder coder = new StandardCoder();
+    private static final ValidatorFactory factory = Validation.buildDefaultValidatorFactory();
+    private static final Validator validator = factory.getValidator();
     private final String packageDir = "src/test/resources/org/onap/policy/common/message/bus/parameters/";
 
     @Test
     void test() throws CoderException {
         final TopicParameterGroup topicParameterGroup =
                 testData.toObject(testData.getTopicParameterGroupMap(false));
-        assertTrue(BeanValidator.isValid(topicParameterGroup));
+        assertTrue(isValid(topicParameterGroup));
         assertEquals(CommonTestData.TOPIC_PARAMS, topicParameterGroup.getTopicSinks());
         assertEquals(CommonTestData.TOPIC_PARAMS, topicParameterGroup.getTopicSources());
 
@@ -70,7 +74,7 @@ class TopicParameterGroupTest {
     void testValidate() {
         final TopicParameterGroup topicParameterGroup =
             testData.toObject(testData.getTopicParameterGroupMap(false));
-        assertTrue(BeanValidator.isValid(topicParameterGroup));
+        assertTrue(isValid(topicParameterGroup));
     }
 
     @Test
@@ -78,7 +82,7 @@ class TopicParameterGroupTest {
         String json = testData.getParameterGroupAsString(
             packageDir + "TopicParameters_valid.json");
         TopicParameterGroup topicParameterGroup = coder.decode(json, TopicParameterGroup.class);
-        assertTrue(BeanValidator.isValid(topicParameterGroup));
+        assertTrue(isValid(topicParameterGroup));
     }
 
     @Test
@@ -86,7 +90,7 @@ class TopicParameterGroupTest {
         String json = testData.getParameterGroupAsString(
             packageDir + "TopicParameters_invalid.json");
         TopicParameterGroup topicParameterGroup = coder.decode(json, TopicParameterGroup.class);
-        assertFalse(BeanValidator.isValid(topicParameterGroup));
+        assertFalse(isValid(topicParameterGroup));
     }
 
     @Test
@@ -94,9 +98,9 @@ class TopicParameterGroupTest {
         String json = testData.getParameterGroupAsString(
             packageDir + "TopicParameters_missing_mandatory.json");
         TopicParameterGroup topicParameterGroup = coder.decode(json, TopicParameterGroup.class);
-        final ValidationResult result = BeanValidator.validate("", topicParameterGroup);
-        assertTrue(result.getResult().contains("\"topicSources[0].servers\" value \"[]\" INVALID, must not be empty"));
-        assertFalse(result.isValid());
+        final var violations = validator.validate(topicParameterGroup);
+        assertFalse(violations.isEmpty());
+        assertThat(violations.toString()).contains("topicSources[0].servers").contains("must not be empty");
     }
 
     @Test
@@ -104,7 +108,7 @@ class TopicParameterGroupTest {
         String json = testData.getParameterGroupAsString(
             packageDir + "TopicParameters_all_params.json");
         TopicParameterGroup topicParameterGroup = coder.decode(json, TopicParameterGroup.class);
-        assertTrue(BeanValidator.isValid(topicParameterGroup));
+        assertTrue(isValid(topicParameterGroup));
         assertTrue(checkIfAllParamsNotEmpty(topicParameterGroup.getTopicSinks()));
         assertTrue(checkIfAllParamsNotEmpty(topicParameterGroup.getTopicSources()));
     }
@@ -135,4 +139,9 @@ class TopicParameterGroupTest {
         }
         return true;
     }
+
+    private boolean isValid(TopicParameterGroup group) {
+        var violations = validator.validate(group);
+        return violations.isEmpty();
+    }
 }
index 60fa172..e98943e 100644 (file)
@@ -33,6 +33,7 @@ import lombok.RequiredArgsConstructor;
 import org.onap.policy.clamp.acm.runtime.main.parameters.AcRuntimeParameterGroup;
 import org.onap.policy.clamp.acm.runtime.main.utils.EncryptionUtils;
 import org.onap.policy.clamp.acm.runtime.supervision.SupervisionAcHandler;
+import org.onap.policy.clamp.models.acm.base.validation.BeanValidationResult;
 import org.onap.policy.clamp.models.acm.concepts.AutomationComposition;
 import org.onap.policy.clamp.models.acm.concepts.AutomationCompositionDefinition;
 import org.onap.policy.clamp.models.acm.concepts.AutomationCompositionElement;
@@ -54,7 +55,6 @@ import org.onap.policy.clamp.models.acm.persistence.provider.ParticipantProvider
 import org.onap.policy.clamp.models.acm.utils.AcmStageUtils;
 import org.onap.policy.clamp.models.acm.utils.AcmStateUtils;
 import org.onap.policy.clamp.models.acm.utils.AcmUtils;
-import org.onap.policy.common.parameters.BeanValidationResult;
 import org.onap.policy.models.base.PfModelRuntimeException;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;