summaryrefslogtreecommitdiffstats
path: root/catalog-model/src/main
diff options
context:
space:
mode:
authorvasraz <vasyl.razinkov@est.tech>2021-06-19 11:07:12 +0100
committerChristophe Closset <christophe.closset@intl.att.com>2021-07-05 09:20:23 +0000
commitf9f5f68caee795d97221071fb26ebe0f572bae0d (patch)
tree6cfa1c82db5e4bd9e93138d7bb06f0bc9961d92e /catalog-model/src/main
parentffce2fdab1c5f6efff795107ef7756d2e9fdcf57 (diff)
Remove usage of outdated library JMockit (catalog-model)
Signed-off-by: Vasyl Razinkov <vasyl.razinkov@est.tech> Change-Id: I3888fb6f9a4c89aefc3dcb124529abd726c400d9 Issue-ID: SDC-3379
Diffstat (limited to 'catalog-model/src/main')
-rw-r--r--catalog-model/src/main/java/org/openecomp/sdc/be/model/GroupInstance.java54
1 files changed, 26 insertions, 28 deletions
diff --git a/catalog-model/src/main/java/org/openecomp/sdc/be/model/GroupInstance.java b/catalog-model/src/main/java/org/openecomp/sdc/be/model/GroupInstance.java
index 386febdf86..331b407328 100644
--- a/catalog-model/src/main/java/org/openecomp/sdc/be/model/GroupInstance.java
+++ b/catalog-model/src/main/java/org/openecomp/sdc/be/model/GroupInstance.java
@@ -40,7 +40,7 @@ public class GroupInstance extends GroupInstanceDataDefinition {
super();
}
- public GroupInstance(GroupInstanceDataDefinition r) {
+ public GroupInstance(final GroupInstanceDataDefinition r) {
super(r);
}
@@ -50,12 +50,11 @@ public class GroupInstance extends GroupInstanceDataDefinition {
* @return
*/
public List<GroupInstanceProperty> convertToGroupInstancesProperties() {
- List<GroupInstanceProperty> groupInstancesProperties = null;
- List<PropertyDataDefinition> propertiesList = super.getProperties();
- if (propertiesList != null && !propertiesList.isEmpty()) {
- groupInstancesProperties = propertiesList.stream().map(GroupInstanceProperty::new).collect(Collectors.toList());
+ final List<PropertyDataDefinition> propertiesList = super.getProperties();
+ if (CollectionUtils.isNotEmpty(propertiesList)) {
+ return propertiesList.stream().map(GroupInstanceProperty::new).collect(Collectors.toList());
}
- return groupInstancesProperties;
+ return null;
}
/**
@@ -64,23 +63,22 @@ public class GroupInstance extends GroupInstanceDataDefinition {
*
* @param groupInstancesProperties
*/
- public void convertFromGroupInstancesProperties(List<GroupInstanceProperty> groupInstancesProperties) {
+ public void convertFromGroupInstancesProperties(final List<GroupInstanceProperty> groupInstancesProperties) {
if (groupInstancesProperties != null && !groupInstancesProperties.isEmpty()) {
- List<PropertyDataDefinition> propList = groupInstancesProperties.stream().map(PropertyDataDefinition::new).collect(Collectors.toList());
- super.setProperties(propList);
+ super.setProperties(groupInstancesProperties.stream().map(PropertyDataDefinition::new).collect(Collectors.toList()));
}
}
private void removeArtifactsDuplicates() {
- List<String> artifacts = getArtifacts();
- Set<String> artifactsSet = new HashSet<>();
+ final List<String> artifacts = getArtifacts();
+ final Set<String> artifactsSet = new HashSet<>();
if (artifacts != null && !artifacts.isEmpty()) {
artifactsSet.addAll(artifacts);
artifacts.clear();
artifacts.addAll(artifactsSet);
}
- List<String> giArtifacts = getGroupInstanceArtifacts();
- Set<String> giArtifactsSet = new HashSet<>();
+ final List<String> giArtifacts = getGroupInstanceArtifacts();
+ final Set<String> giArtifactsSet = new HashSet<>();
if (giArtifacts != null && !giArtifacts.isEmpty()) {
giArtifactsSet.addAll(giArtifacts);
giArtifacts.clear();
@@ -89,16 +87,16 @@ public class GroupInstance extends GroupInstanceDataDefinition {
}
private void clearArtifactsUuid() {
- List<String> artifactsUuid = getArtifactsUuid();
+ final List<String> artifactsUuid = getArtifactsUuid();
if (CollectionUtils.isNotEmpty(artifactsUuid)) {
artifactsUuid.clear();
} else if (artifactsUuid == null) {
setArtifactsUuid(new ArrayList<>());
}
- List<String> giartifactsUuid = this.getGroupInstanceArtifactsUuid();
- if (CollectionUtils.isNotEmpty(giartifactsUuid)) {
- giartifactsUuid.clear();
- } else if (giartifactsUuid == null) {
+ final List<String> groupInstanceArtifactsUuid = this.getGroupInstanceArtifactsUuid();
+ if (CollectionUtils.isNotEmpty(groupInstanceArtifactsUuid)) {
+ groupInstanceArtifactsUuid.clear();
+ } else if (groupInstanceArtifactsUuid == null) {
setGroupInstanceArtifactsUuid(new ArrayList<>());
}
}
@@ -108,19 +106,19 @@ public class GroupInstance extends GroupInstanceDataDefinition {
*
* @param deploymentArtifacts
*/
- public void alignArtifactsUuid(Map<String, ArtifactDefinition> deploymentArtifacts) {
- List<String> artifactIds = getArtifacts();
+ public void alignArtifactsUuid(final Map<String, ArtifactDefinition> deploymentArtifacts) {
+ final List<String> artifactIds = getArtifacts();
log.debug("the artifacts Id's are: {}, and the deployment artifacts Id's are: {}", artifactIds, deploymentArtifacts);
if (CollectionUtils.isNotEmpty(artifactIds) && deploymentArtifacts != null) {
removeArtifactsDuplicates();
clearArtifactsUuid();
- List<String> artifactUuids = getArtifactsUuid();
- List<String> giArtifactUuids = getGroupInstanceArtifactsUuid();
- for (String artifactId : artifactIds) {
- String label = artifactId.substring(artifactId.lastIndexOf('.') + 1);
- ArtifactDefinition artifact = deploymentArtifacts.get(label);
+ final List<String> artifactUuids = getArtifactsUuid();
+ final List<String> giArtifactUuids = getGroupInstanceArtifactsUuid();
+ for (final String artifactId : artifactIds) {
+ final var label = artifactId.substring(artifactId.lastIndexOf('.') + 1);
+ final ArtifactDefinition artifact = deploymentArtifacts.get(label);
log.debug("current artifact id: {}, current artifact definition: {}", artifactId, artifact);
- ArtifactTypeEnum artifactType = ArtifactTypeEnum.parse(artifact.getArtifactType());
+ final var artifactType = ArtifactTypeEnum.parse(artifact.getArtifactType());
if (artifactType != ArtifactTypeEnum.HEAT_ENV) {
addArtifactsIdToCollection(artifactUuids, artifact);
} else {
@@ -130,8 +128,8 @@ public class GroupInstance extends GroupInstanceDataDefinition {
}
}
- private void addArtifactsIdToCollection(List<String> artifactUuids, ArtifactDefinition artifact) {
- if (!artifactUuids.contains(artifact.getArtifactUUID()) && StringUtils.isNotEmpty(artifact.getArtifactUUID())) {
+ private void addArtifactsIdToCollection(final List<String> artifactUuids, final ArtifactDefinition artifact) {
+ if (StringUtils.isNotEmpty(artifact.getArtifactUUID()) && !artifactUuids.contains(artifact.getArtifactUUID())) {
artifactUuids.add(artifact.getArtifactUUID());
}
}