Add Node Filter Business Logic Unit Tests

Issue-ID: SDC-3125
Signed-off-by: aribeiro <anderson.ribeiro@est.tech>
Change-Id: I80c755f4fffa813396ceb85c6aa7fde5b32590cd

1 files changed, 16 insertions, 16 deletions

diff --git a/catalog-be/src/main/java/org/openecomp/sdc/be/components/validation/NodeFilterValidator.java b/catalog-be/src/main/java/org/openecomp/sdc/be/components/validation/NodeFilterValidator.java
index 27daaf2aae..30632e9012 100644
--- a/catalog-be/src/main/java/org/openecomp/sdc/be/components/validation/NodeFilterValidator.java
+++ b/catalog-be/src/main/java/org/openecomp/sdc/be/components/validation/NodeFilterValidator.java
@@ -59,28 +59,28 @@ public class NodeFilterValidator {
     public static final Set<String> comparableConstraintsOperators =
             ImmutableSet.of(ConstraintConvertor.GREATER_THAN_OPERATOR, ConstraintConvertor.LESS_THAN_OPERATOR);
 
-    @Autowired
-    protected ToscaOperationFacade toscaOperationFacade;
-
-    @Autowired
-    protected ComponentsUtils componentsUtils;
+    protected final ToscaOperationFacade toscaOperationFacade;
+    protected final ComponentsUtils componentsUtils;
 
     private static final Logger LOGGER = LoggerFactory.getLogger(NodeFilterValidator.class);
 
-    public Either<Boolean, ResponseFormat> validateComponentInstanceExist(
-        final Component component,
-        final String componentInstanceId) {
+    @Autowired
+    public NodeFilterValidator(final ToscaOperationFacade toscaOperationFacade,
+                               final ComponentsUtils componentsUtils) {
+        this.toscaOperationFacade = toscaOperationFacade;
+        this.componentsUtils = componentsUtils;
+    }
+
+    public Either<Boolean, ResponseFormat> validateComponentInstanceExist(final Component component,
+                                                                          final String componentInstanceId) {
         if (component == null || StringUtils.isEmpty(componentInstanceId)) {
-            LOGGER.debug("Input data cannot be empty");
-            return getErrorResponse(ActionStatus.NODE_FILTER_NOT_FOUND);
-        }
-        if (CollectionUtils.isEmpty(component.getComponentInstances())) {
-            LOGGER.debug("Component Instance list is empty");
+            LOGGER.error("Input data cannot be empty");
             return getErrorResponse(ActionStatus.NODE_FILTER_NOT_FOUND);
         }
-        if (component.getComponentInstances().stream()
-            .noneMatch(ci -> ci.getUniqueId().equals(componentInstanceId))) {
-            LOGGER.debug("Component Instance list is empty");
+        if (CollectionUtils.isEmpty(component.getComponentInstances()) ||
+            component.getComponentInstances().stream()
+                .noneMatch(ci -> ci.getUniqueId().equals(componentInstanceId))) {
+            LOGGER.error("Component Instance list is empty");
             return getErrorResponse(ActionStatus.NODE_FILTER_NOT_FOUND);
         }
         return Either.left(Boolean.TRUE);