aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPeyton Puckett <peyton.puckett@att.com>2019-10-02 08:29:51 -0500
committerPeyton Puckett <peyton.puckett@att.com>2019-10-02 08:49:30 -0500
commitb3819015a98f619b7cfbfc13ac1709f3e6dd3334 (patch)
treea9dafcd80c676204566e2894eedbfe3835d5916c
parent68b17b2a1ee2b6d1920cfc72f08dae25565bbe6a (diff)
Remove windows carriage return
Remove windows CR from code in feature-controller-logging/ Remove windows CR from code in feature-mdc-filters/ Issue-ID: POLICY-2032 Change-Id: I9779f6907fb5a50ab6129969081c2228ce91d3a7 Signed-off-by: Peyton Puckett <peyton.puckett@att.com>
-rwxr-xr-xfeature-controller-logging/src/main/java/org/onap/policy/drools/controller/logging/ControllerLoggingFeature.java198
-rwxr-xr-xfeature-controller-logging/src/test/java/org/onap/policy/drools/controller/logging/ControllerLoggingTest.java466
-rwxr-xr-xfeature-mdc-filters/src/main/java/org/onap/policy/drools/mdc/filters/MdcFilterFeature.java402
-rwxr-xr-xfeature-mdc-filters/src/main/java/org/onap/policy/drools/mdc/filters/MdcTopicFilter.java672
-rwxr-xr-xfeature-mdc-filters/src/test/java/org/onap/policy/drools/mdc/filters/MdcFilterFeatureTest.java338
-rwxr-xr-xfeature-mdc-filters/src/test/java/org/onap/policy/drools/mdc/filters/MdcTopicFilterTest.java1322
6 files changed, 1699 insertions, 1699 deletions
diff --git a/feature-controller-logging/src/main/java/org/onap/policy/drools/controller/logging/ControllerLoggingFeature.java b/feature-controller-logging/src/main/java/org/onap/policy/drools/controller/logging/ControllerLoggingFeature.java
index 37857d39..5e90025f 100755
--- a/feature-controller-logging/src/main/java/org/onap/policy/drools/controller/logging/ControllerLoggingFeature.java
+++ b/feature-controller-logging/src/main/java/org/onap/policy/drools/controller/logging/ControllerLoggingFeature.java
@@ -1,99 +1,99 @@
-/*
- * ============LICENSE_START=======================================================
- * feature-controller-logging
- * ================================================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
- * ================================================================================
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * ============LICENSE_END=========================================================
- */
-
-package org.onap.policy.drools.controller.logging;
-
-import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure;
-import org.onap.policy.common.endpoints.event.comm.TopicSink;
-import org.onap.policy.drools.controller.DroolsController;
-import org.onap.policy.drools.features.DroolsControllerFeatureApi;
-import org.onap.policy.drools.features.PolicyControllerFeatureApi;
-import org.onap.policy.drools.features.PolicyEngineFeatureApi;
-import org.onap.policy.drools.protocol.configuration.ControllerConfiguration;
-import org.onap.policy.drools.protocol.configuration.PdpdConfiguration;
-import org.onap.policy.drools.system.PolicyController;
-import org.onap.policy.drools.system.PolicyControllerConstants;
-import org.onap.policy.drools.system.PolicyEngine;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
-/**
- * This class hooks the network logging implementation into DroolsPDP. It will disable the
- * default network logger where all topic traffic is logged and segregates the topic
- * traffic by controller for each supported control loop use case.
- */
-
-/*
- * PolicyControllerFeatureAPI - the 'beforeStart' hook is used to shut off the default
- * network logger and the 'beforeOffer' hook is used to log incoming topic messages
- *
- * DroolsControllerFeatureAPI - the 'afterDeliver' hook is where the outgoing topic
- * messages are logged
- *
- */
-public class ControllerLoggingFeature
- implements PolicyEngineFeatureApi, DroolsControllerFeatureApi, PolicyControllerFeatureApi {
-
- @Override
- public int getSequenceNumber() {
- return 1000;
- }
-
- /**
- * The 'beforeOffer' hook will intercept an incoming topic message and append it to
- * the log file that is configured for the controller logger.
- */
- @Override
- public boolean beforeOffer(PolicyController controller, CommInfrastructure protocol, String topic, String event) {
- Logger controllerLogger = LoggerFactory.getLogger(controller.getName());
- controllerLogger.info("[IN|{}|{}]{}{}", protocol, topic, System.lineSeparator(), event);
- return false;
- }
-
- /**
- * The 'afterDeliver' hook will intercept an outgoing topic message and append it to
- * the log file that is configured for the controller logger.
- */
- @Override
- public boolean afterDeliver(DroolsController controller, TopicSink sink, Object fact, String json,
- boolean success) {
- if (success) {
- Logger controllerLogger = LoggerFactory
- .getLogger(PolicyControllerConstants.getFactory().get(controller).getName());
- controllerLogger.info("[OUT|{}|{}]{}{}", sink.getTopicCommInfrastructure(), sink.getTopic(),
- System.lineSeparator(), json);
- }
- return false;
- }
-
- /**
- * The 'afterOnTopicEvent' hook will determine which controllers were updated and log
- * the event to the appropriate controller logs.
- */
- @Override
- public boolean afterOnTopicEvent(PolicyEngine engine, PdpdConfiguration configuration, CommInfrastructure commType,
- String topic, String event) {
- for (ControllerConfiguration controller : configuration.getControllers()) {
- Logger controllerLogger = LoggerFactory.getLogger(controller.getName());
- controllerLogger.info("[IN|{}|{}]{}{}", commType, topic, System.lineSeparator(), event);
- }
- return false;
- }
-}
+/*
+ * ============LICENSE_START=======================================================
+ * feature-controller-logging
+ * ================================================================================
+ * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ */
+
+package org.onap.policy.drools.controller.logging;
+
+import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure;
+import org.onap.policy.common.endpoints.event.comm.TopicSink;
+import org.onap.policy.drools.controller.DroolsController;
+import org.onap.policy.drools.features.DroolsControllerFeatureApi;
+import org.onap.policy.drools.features.PolicyControllerFeatureApi;
+import org.onap.policy.drools.features.PolicyEngineFeatureApi;
+import org.onap.policy.drools.protocol.configuration.ControllerConfiguration;
+import org.onap.policy.drools.protocol.configuration.PdpdConfiguration;
+import org.onap.policy.drools.system.PolicyController;
+import org.onap.policy.drools.system.PolicyControllerConstants;
+import org.onap.policy.drools.system.PolicyEngine;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+/**
+ * This class hooks the network logging implementation into DroolsPDP. It will disable the
+ * default network logger where all topic traffic is logged and segregates the topic
+ * traffic by controller for each supported control loop use case.
+ */
+
+/*
+ * PolicyControllerFeatureAPI - the 'beforeStart' hook is used to shut off the default
+ * network logger and the 'beforeOffer' hook is used to log incoming topic messages
+ *
+ * DroolsControllerFeatureAPI - the 'afterDeliver' hook is where the outgoing topic
+ * messages are logged
+ *
+ */
+public class ControllerLoggingFeature
+ implements PolicyEngineFeatureApi, DroolsControllerFeatureApi, PolicyControllerFeatureApi {
+
+ @Override
+ public int getSequenceNumber() {
+ return 1000;
+ }
+
+ /**
+ * The 'beforeOffer' hook will intercept an incoming topic message and append it to
+ * the log file that is configured for the controller logger.
+ */
+ @Override
+ public boolean beforeOffer(PolicyController controller, CommInfrastructure protocol, String topic, String event) {
+ Logger controllerLogger = LoggerFactory.getLogger(controller.getName());
+ controllerLogger.info("[IN|{}|{}]{}{}", protocol, topic, System.lineSeparator(), event);
+ return false;
+ }
+
+ /**
+ * The 'afterDeliver' hook will intercept an outgoing topic message and append it to
+ * the log file that is configured for the controller logger.
+ */
+ @Override
+ public boolean afterDeliver(DroolsController controller, TopicSink sink, Object fact, String json,
+ boolean success) {
+ if (success) {
+ Logger controllerLogger = LoggerFactory
+ .getLogger(PolicyControllerConstants.getFactory().get(controller).getName());
+ controllerLogger.info("[OUT|{}|{}]{}{}", sink.getTopicCommInfrastructure(), sink.getTopic(),
+ System.lineSeparator(), json);
+ }
+ return false;
+ }
+
+ /**
+ * The 'afterOnTopicEvent' hook will determine which controllers were updated and log
+ * the event to the appropriate controller logs.
+ */
+ @Override
+ public boolean afterOnTopicEvent(PolicyEngine engine, PdpdConfiguration configuration, CommInfrastructure commType,
+ String topic, String event) {
+ for (ControllerConfiguration controller : configuration.getControllers()) {
+ Logger controllerLogger = LoggerFactory.getLogger(controller.getName());
+ controllerLogger.info("[IN|{}|{}]{}{}", commType, topic, System.lineSeparator(), event);
+ }
+ return false;
+ }
+}
diff --git a/feature-controller-logging/src/test/java/org/onap/policy/drools/controller/logging/ControllerLoggingTest.java b/feature-controller-logging/src/test/java/org/onap/policy/drools/controller/logging/ControllerLoggingTest.java
index 213ac493..82cfbd91 100755
--- a/feature-controller-logging/src/test/java/org/onap/policy/drools/controller/logging/ControllerLoggingTest.java
+++ b/feature-controller-logging/src/test/java/org/onap/policy/drools/controller/logging/ControllerLoggingTest.java
@@ -1,233 +1,233 @@
-/*
- * ============LICENSE_START=======================================================
- * feature-controller-logging
- * ================================================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
- * ================================================================================
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * ============LICENSE_END=========================================================
- */
-
-package org.onap.policy.drools.controller.logging;
-
-import static org.junit.Assert.assertEquals;
-
-import ch.qos.logback.classic.spi.LoggingEvent;
-import ch.qos.logback.core.AppenderBase;
-import com.google.gson.Gson;
-import com.google.gson.GsonBuilder;
-import java.io.IOException;
-import java.nio.file.Paths;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.List;
-import java.util.Properties;
-import org.junit.After;
-import org.junit.BeforeClass;
-import org.junit.Test;
-import org.onap.policy.common.endpoints.event.comm.Topic;
-import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure;
-import org.onap.policy.common.endpoints.event.comm.bus.NoopTopicSink;
-import org.onap.policy.drools.controller.DroolsController;
-import org.onap.policy.drools.controller.DroolsControllerConstants;
-import org.onap.policy.drools.properties.DroolsPropertyConstants;
-import org.onap.policy.drools.protocol.configuration.ControllerConfiguration;
-import org.onap.policy.drools.protocol.configuration.PdpdConfiguration;
-import org.onap.policy.drools.system.PolicyController;
-import org.onap.policy.drools.system.PolicyEngineConstants;
-import org.onap.policy.drools.util.KieUtils;
-
-/**
- * Controller Logger Tests.
- */
-public class ControllerLoggingTest {
-
- /**
- * These properties are for installing a test artifact that the drools controller can
- * fetch while testing.
- */
- private static final String JUNIT_KMODULE_DRL_PATH = "src/test/resources/test.drl";
- private static final String JUNIT_KMODULE_POM_PATH = "src/test/resources/test.pom";
- private static final String JUNIT_KMODULE_PATH = "src/test/resources/kmodule.xml";
- private static final String JUNIT_KJAR_DRL_PATH = "src/main/resources/org/onap/policy/drools/test/test.drl";
-
- /**
- * These properties are used for the Policy Controller to point to the test artifact.
- */
- private static final String TEST_CONTROLLER_NAME = "test-controller";
- private static final String TEST_GROUP_ID = "org.onap.policy.drools.test";
- private static final String TEST_ARTIFACT_ID = "test";
- private static final String TEST_VERSION = "1.0.0";
-
- /**
- * A test topic used for delivery and network logging.
- */
- private static final String TEST_TOPIC = "test-topic";
- private static final String TEST_SERVER = "http://test.com";
-
- /**
- * These are used for sending PDPD configuration notifications to a policy controller.
- */
- private static Properties controllerProps = null;
- private static String message = null;
- private static PdpdConfiguration pdpdNotification = null;
- private static PolicyController policyController = null;
-
- /**
- * This is a list of events that are appended to the controller-test logger.
- */
- private static List<LoggingEvent> events = new ArrayList<>();
-
- /**
- * A custom appender used to intercept events and add them to a list of events that
- * the junits can use to determine logging was successful.
- */
- public static class NetworkAppender extends AppenderBase<LoggingEvent> {
-
- @Override
- protected void append(LoggingEvent event) {
- events.add(event);
- }
-
- }
-
- /**
- * Runs before all the test cases to install the drools artifact, create a policy
- * controller, and create a PDPD configuration notification.
- */
- @BeforeClass
- public static void setUp() throws IOException {
- KieUtils.installArtifact(Paths.get(JUNIT_KMODULE_PATH).toFile(), Paths.get(JUNIT_KMODULE_POM_PATH).toFile(),
- JUNIT_KJAR_DRL_PATH, Paths.get(JUNIT_KMODULE_DRL_PATH).toFile());
-
- controllerProps = new Properties();
- controllerProps.put(DroolsPropertyConstants.PROPERTY_CONTROLLER_NAME, TEST_CONTROLLER_NAME);
- controllerProps.put(DroolsPropertyConstants.RULES_GROUPID, TEST_GROUP_ID);
- controllerProps.put(DroolsPropertyConstants.RULES_ARTIFACTID, TEST_ARTIFACT_ID);
- controllerProps.put(DroolsPropertyConstants.RULES_VERSION, TEST_VERSION);
-
- policyController = PolicyEngineConstants.getManager().createPolicyController(TEST_CONTROLLER_NAME,
- controllerProps);
-
- message = "{\"requestID\":\"38adde30-cc22-11e8-a8d5-f2801f1b9fd1\",\"entity\":\"controller\",\"controllers\":"
- + "[{\"name\":\"test-controller\",\"drools\":{\"groupId\":\"org.onap.policy.drools.test\","
- + "\"artifactId\":\"test\",\"version\":\"0.0.1\"},\"operation\":\"update\"}]}";
-
- Gson decoder = new GsonBuilder().disableHtmlEscaping().create();
- pdpdNotification = decoder.fromJson(message, PdpdConfiguration.class);
- }
-
- /**
- * Runs after every test case to clean up the events added to the event list during
- * unit test.
- */
- @After
- public void cleanUpLogs() {
- events.clear();
- }
-
- /**
- * Obtains the sequence number of the controller logging feature. This should return
- * 1000.
- */
- @Test
- public void getSequenceNumberTest() {
- ControllerLoggingFeature nlf = new ControllerLoggingFeature();
- assertEquals(1000, nlf.getSequenceNumber());
- }
-
- /**
- * Asserts that the controller-test logger appends the incoming message to the event
- * list.
- */
- @Test
- public void beforeOffer() {
- ControllerLoggingFeature nlf = new ControllerLoggingFeature();
-
- nlf.beforeOffer(policyController, Topic.CommInfrastructure.UEB, TEST_TOPIC, "{\"test\":\"test\"}");
-
- assertEquals(1, events.size());
- }
-
- /**
- * Asserts that the controller-test logger appends the outgoing message to the event
- * list.
- */
- @Test
- public void afterDeliverSuccess() {
-
- final ControllerLoggingFeature nlf = new ControllerLoggingFeature();
-
- DroolsController droolsController =
- DroolsControllerConstants.getFactory().get(TEST_GROUP_ID, TEST_ARTIFACT_ID, TEST_VERSION);
-
- NoopTopicSink sinkTopic = new NoopTopicSink(Arrays.asList(TEST_SERVER), TEST_TOPIC);
-
- nlf.afterDeliver(droolsController, sinkTopic, null, "{\"test\":\"test\"}", true);
-
- assertEquals(1, events.size());
-
- }
-
- /**
- * Asserts that the controller-test logger does not append the outgoing message to the
- * event list if there was a failure.
- */
- @Test
- public void afterDeliverFailure() {
-
- final ControllerLoggingFeature nlf = new ControllerLoggingFeature();
-
- DroolsController droolsController =
- DroolsControllerConstants.getFactory().get(TEST_GROUP_ID, TEST_ARTIFACT_ID, TEST_VERSION);
-
- NoopTopicSink sinkTopic = new NoopTopicSink(Arrays.asList(TEST_SERVER), TEST_TOPIC);
-
- nlf.afterDeliver(droolsController, sinkTopic, null, "{\"test\":\"test\"}", false);
-
- assertEquals(0, events.size());
- }
-
- /**
- * Asserts that the controller logging feature can log the messages to the proper
- * controller based on the message containing the controller name.
- */
- @Test
- public void afterOnTopicEventSuccess() {
- final ControllerLoggingFeature nlf = new ControllerLoggingFeature();
-
- nlf.afterOnTopicEvent(PolicyEngineConstants.getManager(), pdpdNotification, CommInfrastructure.UEB, TEST_TOPIC,
- message);
-
- assertEquals(1, events.size());
- }
-
- /**
- * Asserts that the controller logging feature can skip logging messages that don't
- * contain the controller names in it.
- */
- @Test
- public void afterOnTopicEventFailure() {
- final ControllerLoggingFeature nlf = new ControllerLoggingFeature();
-
- PdpdConfiguration notification = new PdpdConfiguration();
- ControllerConfiguration config = new ControllerConfiguration();
- config.setName("test-controller-2");
- notification.setControllers(Arrays.asList(config));
-
- nlf.afterOnTopicEvent(PolicyEngineConstants.getManager(), notification, CommInfrastructure.UEB, TEST_TOPIC,
- message);
-
- assertEquals(0, events.size());
- }
-}
+/*
+ * ============LICENSE_START=======================================================
+ * feature-controller-logging
+ * ================================================================================
+ * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ */
+
+package org.onap.policy.drools.controller.logging;
+
+import static org.junit.Assert.assertEquals;
+
+import ch.qos.logback.classic.spi.LoggingEvent;
+import ch.qos.logback.core.AppenderBase;
+import com.google.gson.Gson;
+import com.google.gson.GsonBuilder;
+import java.io.IOException;
+import java.nio.file.Paths;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.List;
+import java.util.Properties;
+import org.junit.After;
+import org.junit.BeforeClass;
+import org.junit.Test;
+import org.onap.policy.common.endpoints.event.comm.Topic;
+import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure;
+import org.onap.policy.common.endpoints.event.comm.bus.NoopTopicSink;
+import org.onap.policy.drools.controller.DroolsController;
+import org.onap.policy.drools.controller.DroolsControllerConstants;
+import org.onap.policy.drools.properties.DroolsPropertyConstants;
+import org.onap.policy.drools.protocol.configuration.ControllerConfiguration;
+import org.onap.policy.drools.protocol.configuration.PdpdConfiguration;
+import org.onap.policy.drools.system.PolicyController;
+import org.onap.policy.drools.system.PolicyEngineConstants;
+import org.onap.policy.drools.util.KieUtils;
+
+/**
+ * Controller Logger Tests.
+ */
+public class ControllerLoggingTest {
+
+ /**
+ * These properties are for installing a test artifact that the drools controller can
+ * fetch while testing.
+ */
+ private static final String JUNIT_KMODULE_DRL_PATH = "src/test/resources/test.drl";
+ private static final String JUNIT_KMODULE_POM_PATH = "src/test/resources/test.pom";
+ private static final String JUNIT_KMODULE_PATH = "src/test/resources/kmodule.xml";
+ private static final String JUNIT_KJAR_DRL_PATH = "src/main/resources/org/onap/policy/drools/test/test.drl";
+
+ /**
+ * These properties are used for the Policy Controller to point to the test artifact.
+ */
+ private static final String TEST_CONTROLLER_NAME = "test-controller";
+ private static final String TEST_GROUP_ID = "org.onap.policy.drools.test";
+ private static final String TEST_ARTIFACT_ID = "test";
+ private static final String TEST_VERSION = "1.0.0";
+
+ /**
+ * A test topic used for delivery and network logging.
+ */
+ private static final String TEST_TOPIC = "test-topic";
+ private static final String TEST_SERVER = "http://test.com";
+
+ /**
+ * These are used for sending PDPD configuration notifications to a policy controller.
+ */
+ private static Properties controllerProps = null;
+ private static String message = null;
+ private static PdpdConfiguration pdpdNotification = null;
+ private static PolicyController policyController = null;
+
+ /**
+ * This is a list of events that are appended to the controller-test logger.
+ */
+ private static List<LoggingEvent> events = new ArrayList<>();
+
+ /**
+ * A custom appender used to intercept events and add them to a list of events that
+ * the junits can use to determine logging was successful.
+ */
+ public static class NetworkAppender extends AppenderBase<LoggingEvent> {
+
+ @Override
+ protected void append(LoggingEvent event) {
+ events.add(event);
+ }
+
+ }
+
+ /**
+ * Runs before all the test cases to install the drools artifact, create a policy
+ * controller, and create a PDPD configuration notification.
+ */
+ @BeforeClass
+ public static void setUp() throws IOException {
+ KieUtils.installArtifact(Paths.get(JUNIT_KMODULE_PATH).toFile(), Paths.get(JUNIT_KMODULE_POM_PATH).toFile(),
+ JUNIT_KJAR_DRL_PATH, Paths.get(JUNIT_KMODULE_DRL_PATH).toFile());
+
+ controllerProps = new Properties();
+ controllerProps.put(DroolsPropertyConstants.PROPERTY_CONTROLLER_NAME, TEST_CONTROLLER_NAME);
+ controllerProps.put(DroolsPropertyConstants.RULES_GROUPID, TEST_GROUP_ID);
+ controllerProps.put(DroolsPropertyConstants.RULES_ARTIFACTID, TEST_ARTIFACT_ID);
+ controllerProps.put(DroolsPropertyConstants.RULES_VERSION, TEST_VERSION);
+
+ policyController = PolicyEngineConstants.getManager().createPolicyController(TEST_CONTROLLER_NAME,
+ controllerProps);
+
+ message = "{\"requestID\":\"38adde30-cc22-11e8-a8d5-f2801f1b9fd1\",\"entity\":\"controller\",\"controllers\":"
+ + "[{\"name\":\"test-controller\",\"drools\":{\"groupId\":\"org.onap.policy.drools.test\","
+ + "\"artifactId\":\"test\",\"version\":\"0.0.1\"},\"operation\":\"update\"}]}";
+
+ Gson decoder = new GsonBuilder().disableHtmlEscaping().create();
+ pdpdNotification = decoder.fromJson(message, PdpdConfiguration.class);
+ }
+
+ /**
+ * Runs after every test case to clean up the events added to the event list during
+ * unit test.
+ */
+ @After
+ public void cleanUpLogs() {
+ events.clear();
+ }
+
+ /**
+ * Obtains the sequence number of the controller logging feature. This should return
+ * 1000.
+ */
+ @Test
+ public void getSequenceNumberTest() {
+ ControllerLoggingFeature nlf = new ControllerLoggingFeature();
+ assertEquals(1000, nlf.getSequenceNumber());
+ }
+
+ /**
+ * Asserts that the controller-test logger appends the incoming message to the event
+ * list.
+ */
+ @Test
+ public void beforeOffer() {
+ ControllerLoggingFeature nlf = new ControllerLoggingFeature();
+
+ nlf.beforeOffer(policyController, Topic.CommInfrastructure.UEB, TEST_TOPIC, "{\"test\":\"test\"}");
+
+ assertEquals(1, events.size());
+ }
+
+ /**
+ * Asserts that the controller-test logger appends the outgoing message to the event
+ * list.
+ */
+ @Test
+ public void afterDeliverSuccess() {
+
+ final ControllerLoggingFeature nlf = new ControllerLoggingFeature();
+
+ DroolsController droolsController =
+ DroolsControllerConstants.getFactory().get(TEST_GROUP_ID, TEST_ARTIFACT_ID, TEST_VERSION);
+
+ NoopTopicSink sinkTopic = new NoopTopicSink(Arrays.asList(TEST_SERVER), TEST_TOPIC);
+
+ nlf.afterDeliver(droolsController, sinkTopic, null, "{\"test\":\"test\"}", true);
+
+ assertEquals(1, events.size());
+
+ }
+
+ /**
+ * Asserts that the controller-test logger does not append the outgoing message to the
+ * event list if there was a failure.
+ */
+ @Test
+ public void afterDeliverFailure() {
+
+ final ControllerLoggingFeature nlf = new ControllerLoggingFeature();
+
+ DroolsController droolsController =
+ DroolsControllerConstants.getFactory().get(TEST_GROUP_ID, TEST_ARTIFACT_ID, TEST_VERSION);
+
+ NoopTopicSink sinkTopic = new NoopTopicSink(Arrays.asList(TEST_SERVER), TEST_TOPIC);
+
+ nlf.afterDeliver(droolsController, sinkTopic, null, "{\"test\":\"test\"}", false);
+
+ assertEquals(0, events.size());
+ }
+
+ /**
+ * Asserts that the controller logging feature can log the messages to the proper
+ * controller based on the message containing the controller name.
+ */
+ @Test
+ public void afterOnTopicEventSuccess() {
+ final ControllerLoggingFeature nlf = new ControllerLoggingFeature();
+
+ nlf.afterOnTopicEvent(PolicyEngineConstants.getManager(), pdpdNotification, CommInfrastructure.UEB, TEST_TOPIC,
+ message);
+
+ assertEquals(1, events.size());
+ }
+
+ /**
+ * Asserts that the controller logging feature can skip logging messages that don't
+ * contain the controller names in it.
+ */
+ @Test
+ public void afterOnTopicEventFailure() {
+ final ControllerLoggingFeature nlf = new ControllerLoggingFeature();
+
+ PdpdConfiguration notification = new PdpdConfiguration();
+ ControllerConfiguration config = new ControllerConfiguration();
+ config.setName("test-controller-2");
+ notification.setControllers(Arrays.asList(config));
+
+ nlf.afterOnTopicEvent(PolicyEngineConstants.getManager(), notification, CommInfrastructure.UEB, TEST_TOPIC,
+ message);
+
+ assertEquals(0, events.size());
+ }
+}
diff --git a/feature-mdc-filters/src/main/java/org/onap/policy/drools/mdc/filters/MdcFilterFeature.java b/feature-mdc-filters/src/main/java/org/onap/policy/drools/mdc/filters/MdcFilterFeature.java
index 0f6c3d72..015aac21 100755
--- a/feature-mdc-filters/src/main/java/org/onap/policy/drools/mdc/filters/MdcFilterFeature.java
+++ b/feature-mdc-filters/src/main/java/org/onap/policy/drools/mdc/filters/MdcFilterFeature.java
@@ -1,201 +1,201 @@
-/*
- * ============LICENSE_START=======================================================
- * feature-mdc-filters
- * ================================================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
- * ================================================================================
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * ============LICENSE_END=========================================================
- */
-
-package org.onap.policy.drools.mdc.filters;
-
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-import java.util.Properties;
-import org.onap.policy.common.endpoints.event.comm.Topic;
-import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure;
-import org.onap.policy.common.endpoints.features.NetLoggerFeatureApi;
-import org.onap.policy.common.endpoints.utils.NetLoggerUtil.EventType;
-import org.onap.policy.drools.features.PolicyControllerFeatureApi;
-import org.onap.policy.drools.persistence.SystemPersistenceConstants;
-import org.onap.policy.drools.system.PolicyController;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import org.slf4j.MDC;
-
-public class MdcFilterFeature implements NetLoggerFeatureApi, PolicyControllerFeatureApi {
-
- /**
- * Logger.
- */
- private static final Logger logger = LoggerFactory.getLogger(MdcFilterFeature.class);
-
- /**
- * Feature properties.
- */
- public static final String FEATURE_NAME = "feature-mdc-filters";
- public static final String SOURCE = "source";
- public static final String SINK = "sink";
- public static final String MDC_FILTERS = ".mdcFilters";
-
- /**
- * Mapping of 'protocol:type:topic' key to a 'MdcTopicFilter' object.
- */
- private Map<String, MdcTopicFilter> topicFilters = new HashMap<>();
-
- /**
- * Feature properties map obtained from the feature properties file.
- */
- private Properties featureProps = null;
-
- /**
- * Constructor.
- */
- public MdcFilterFeature() {
- super();
- featureProps = getFeatureProps();
- }
-
- /**
- * Gets the feature properties.
- *
- * @return the properties for this feature.
- */
- protected Properties getFeatureProps() {
- return SystemPersistenceConstants.getManager().getProperties(FEATURE_NAME);
- }
-
- /**
- * Sequence number to be used for order of feature implementer execution.
- */
- @Override
- public int getSequenceNumber() {
- return 1;
- }
-
- /**
- * Loops through all source and sink topics to find which topics have mdc filters and
- * inserts an MdcTopicFilter in to the topicFilters map.
- */
- @Override
- public boolean afterCreate(PolicyController controller) {
- createSourceTopicFilters(controller);
- createSinkTopicFilters(controller);
- return false;
- }
-
- /**
- * Extracts the fields in a JSON string that are to be logged in an abbreviated
- * message. The event delivery infrastructure details are put in the MDC as well using
- * the keys networkEventType (IN/OUT), networkProtocol (UEB/DMAAP/NOOP/REST), and
- * networkTopic.
- */
- @Override
- public boolean beforeLog(Logger eventLogger, EventType type, CommInfrastructure protocol, String topic,
- String message) {
-
- String filterKey = null;
- if (type == EventType.IN) {
- filterKey = getTopicKey(protocol.name().toLowerCase(), SOURCE, topic);
- } else {
- filterKey = getTopicKey(protocol.name().toLowerCase(), SINK, topic);
- }
-
- MDC.put("networkEventType", type.name());
- MDC.put("networkProtocol", protocol.name());
- MDC.put("networkTopic", topic);
-
- MdcTopicFilter filter = topicFilters.get(filterKey);
- if (filter != null) {
- for (Map.Entry<String, List<String>> entry : filter.find(message).entrySet()) {
- String mdcKey = entry.getKey();
- List<String> results = entry.getValue();
- if (results.isEmpty()) {
- logger.debug("No results found for key {}", mdcKey);
- } else if (results.size() > 1) {
- logger.debug("Multple results found for key {}, returning list as a string", mdcKey);
- MDC.put(mdcKey, results.toString());
- } else {
- MDC.put(mdcKey, results.get(0));
- }
- }
- } else {
- logger.debug("No mdc topic filters exist for key {}", filterKey);
- }
-
- return false;
- }
-
- /**
- * Clears the MDC mapping after a message is logged.
- */
- @Override
- public boolean afterLog(Logger eventLogger, EventType type, CommInfrastructure protocol, String topic,
- String message) {
- MDC.clear();
- return false;
- }
-
- /**
- * Creates a key using the protocol, type, and topic name.
- *
- * @param protocol defined as ueb, dmaap, noop
- * @param type defined as source or sink
- * @param topic name of the topic
- * @return a key that is the concatenation of the protocol, type, and topic name
- */
- private String getTopicKey(String protocol, String type, String topic) {
- return protocol + ":" + type + ":" + topic;
- }
-
- /**
- * Creates MdcTopicFilters for a source/sink topic based on the type.
- *
- * @param topic the topic name
- * @param type 'source' or 'sink'
- */
- private void createTopicFilter(Topic topic, String type) {
- String protocol = topic.getTopicCommInfrastructure().name().toLowerCase();
- String topicName = topic.getTopic();
-
- String propertyKey = protocol + "." + type + ".topics." + topicName + MDC_FILTERS;
- String propertyValue = featureProps.getProperty(propertyKey);
- if (propertyValue != null) {
- String topicKey = getTopicKey(protocol, type, topicName);
- if (!topicFilters.containsKey(topicKey)) {
- logger.debug("MdcTopicFilter created for {} {} topic {}", protocol, type, topicName);
- topicFilters.put(topicKey, new MdcTopicFilter(propertyValue));
- } else {
- logger.debug("An MdcTopicFilter already exists for key {}", topicKey);
- }
- } else {
- logger.debug("No MDC filters defined for {} {} topic {}", protocol, type, topicName);
- }
- }
-
- /**
- * Creates MdcTopicFilters for the controller's source topics.
- */
- private void createSourceTopicFilters(PolicyController controller) {
- controller.getTopicSources().forEach(sourceTopic -> createTopicFilter(sourceTopic, SOURCE));
- }
-
- /**
- * Creates MdcTopicFilters for the controller's sink topics.
- */
- private void createSinkTopicFilters(PolicyController controller) {
- controller.getTopicSinks().forEach(sinkTopic -> createTopicFilter(sinkTopic, SINK));
- }
-}
+/*
+ * ============LICENSE_START=======================================================
+ * feature-mdc-filters
+ * ================================================================================
+ * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ */
+
+package org.onap.policy.drools.mdc.filters;
+
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+import java.util.Properties;
+import org.onap.policy.common.endpoints.event.comm.Topic;
+import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure;
+import org.onap.policy.common.endpoints.features.NetLoggerFeatureApi;
+import org.onap.policy.common.endpoints.utils.NetLoggerUtil.EventType;
+import org.onap.policy.drools.features.PolicyControllerFeatureApi;
+import org.onap.policy.drools.persistence.SystemPersistenceConstants;
+import org.onap.policy.drools.system.PolicyController;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.slf4j.MDC;
+
+public class MdcFilterFeature implements NetLoggerFeatureApi, PolicyControllerFeatureApi {
+
+ /**
+ * Logger.
+ */
+ private static final Logger logger = LoggerFactory.getLogger(MdcFilterFeature.class);
+
+ /**
+ * Feature properties.
+ */
+ public static final String FEATURE_NAME = "feature-mdc-filters";
+ public static final String SOURCE = "source";
+ public static final String SINK = "sink";
+ public static final String MDC_FILTERS = ".mdcFilters";
+
+ /**
+ * Mapping of 'protocol:type:topic' key to a 'MdcTopicFilter' object.
+ */
+ private Map<String, MdcTopicFilter> topicFilters = new HashMap<>();
+
+ /**
+ * Feature properties map obtained from the feature properties file.
+ */
+ private Properties featureProps = null;
+
+ /**
+ * Constructor.
+ */
+ public MdcFilterFeature() {
+ super();
+ featureProps = getFeatureProps();
+ }
+
+ /**
+ * Gets the feature properties.
+ *
+ * @return the properties for this feature.
+ */
+ protected Properties getFeatureProps() {
+ return SystemPersistenceConstants.getManager().getProperties(FEATURE_NAME);
+ }
+
+ /**
+ * Sequence number to be used for order of feature implementer execution.
+ */
+ @Override
+ public int getSequenceNumber() {
+ return 1;
+ }
+
+ /**
+ * Loops through all source and sink topics to find which topics have mdc filters and
+ * inserts an MdcTopicFilter in to the topicFilters map.
+ */
+ @Override
+ public boolean afterCreate(PolicyController controller) {
+ createSourceTopicFilters(controller);
+ createSinkTopicFilters(controller);
+ return false;
+ }
+
+ /**
+ * Extracts the fields in a JSON string that are to be logged in an abbreviated
+ * message. The event delivery infrastructure details are put in the MDC as well using
+ * the keys networkEventType (IN/OUT), networkProtocol (UEB/DMAAP/NOOP/REST), and
+ * networkTopic.
+ */
+ @Override
+ public boolean beforeLog(Logger eventLogger, EventType type, CommInfrastructure protocol, String topic,
+ String message) {
+
+ String filterKey = null;
+ if (type == EventType.IN) {
+ filterKey = getTopicKey(protocol.name().toLowerCase(), SOURCE, topic);
+ } else {
+ filterKey = getTopicKey(protocol.name().toLowerCase(), SINK, topic);
+ }
+
+ MDC.put("networkEventType", type.name());
+ MDC.put("networkProtocol", protocol.name());
+ MDC.put("networkTopic", topic);
+
+ MdcTopicFilter filter = topicFilters.get(filterKey);
+ if (filter != null) {
+ for (Map.Entry<String, List<String>> entry : filter.find(message).entrySet()) {
+ String mdcKey = entry.getKey();
+ List<String> results = entry.getValue();
+ if (results.isEmpty()) {
+ logger.debug("No results found for key {}", mdcKey);
+ } else if (results.size() > 1) {
+ logger.debug("Multple results found for key {}, returning list as a string", mdcKey);
+ MDC.put(mdcKey, results.toString());
+ } else {
+ MDC.put(mdcKey, results.get(0));
+ }
+ }
+ } else {
+ logger.debug("No mdc topic filters exist for key {}", filterKey);
+ }
+
+ return false;
+ }
+
+ /**
+ * Clears the MDC mapping after a message is logged.
+ */
+ @Override
+ public boolean afterLog(Logger eventLogger, EventType type, CommInfrastructure protocol, String topic,
+ String message) {
+ MDC.clear();
+ return false;
+ }
+
+ /**
+ * Creates a key using the protocol, type, and topic name.
+ *
+ * @param protocol defined as ueb, dmaap, noop
+ * @param type defined as source or sink
+ * @param topic name of the topic
+ * @return a key that is the concatenation of the protocol, type, and topic name
+ */
+ private String getTopicKey(String protocol, String type, String topic) {
+ return protocol + ":" + type + ":" + topic;
+ }
+
+ /**
+ * Creates MdcTopicFilters for a source/sink topic based on the type.
+ *
+ * @param topic the topic name
+ * @param type 'source' or 'sink'
+ */
+ private void createTopicFilter(Topic topic, String type) {
+ String protocol = topic.getTopicCommInfrastructure().name().toLowerCase();
+ String topicName = topic.getTopic();
+
+ String propertyKey = protocol + "." + type + ".topics." + topicName + MDC_FILTERS;
+ String propertyValue = featureProps.getProperty(propertyKey);
+ if (propertyValue != null) {
+ String topicKey = getTopicKey(protocol, type, topicName);
+ if (!topicFilters.containsKey(topicKey)) {
+ logger.debug("MdcTopicFilter created for {} {} topic {}", protocol, type, topicName);
+ topicFilters.put(topicKey, new MdcTopicFilter(propertyValue));
+ } else {
+ logger.debug("An MdcTopicFilter already exists for key {}", topicKey);
+ }
+ } else {
+ logger.debug("No MDC filters defined for {} {} topic {}", protocol, type, topicName);
+ }
+ }
+
+ /**
+ * Creates MdcTopicFilters for the controller's source topics.
+ */
+ private void createSourceTopicFilters(PolicyController controller) {
+ controller.getTopicSources().forEach(sourceTopic -> createTopicFilter(sourceTopic, SOURCE));
+ }
+
+ /**
+ * Creates MdcTopicFilters for the controller's sink topics.
+ */
+ private void createSinkTopicFilters(PolicyController controller) {
+ controller.getTopicSinks().forEach(sinkTopic -> createTopicFilter(sinkTopic, SINK));
+ }
+}
diff --git a/feature-mdc-filters/src/main/java/org/onap/policy/drools/mdc/filters/MdcTopicFilter.java b/feature-mdc-filters/src/main/java/org/onap/policy/drools/mdc/filters/MdcTopicFilter.java
index 7aa0a920..b66c254a 100755
--- a/feature-mdc-filters/src/main/java/org/onap/policy/drools/mdc/filters/MdcTopicFilter.java
+++ b/feature-mdc-filters/src/main/java/org/onap/policy/drools/mdc/filters/MdcTopicFilter.java
@@ -1,336 +1,336 @@
-/*
- * ============LICENSE_START=======================================================
- * feature-mdc-filters
- * ================================================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
- * ================================================================================
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * ============LICENSE_END=========================================================
- */
-
-package org.onap.policy.drools.mdc.filters;
-
-import com.att.aft.dme2.internal.apache.commons.lang3.StringUtils;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
-import lombok.Getter;
-import org.onap.policy.drools.protocol.coders.JsonProtocolFilter;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
-public class MdcTopicFilter {
-
- private static final Logger logger = LoggerFactory.getLogger(MdcTopicFilter.class);
-
- public static final String MDC_KEY_ERROR = "mdcKey must be provided";
- public static final String JSON_PATH_ERROR = "json path(s) must be provided";
-
- private Map<String, FilterRule> rules = new HashMap<>();
-
- @Getter
- public static class FilterRule {
- private String mdcKey;
- private List<String> paths;
-
- public FilterRule(String mdcKey, String path) {
- this.mdcKey = mdcKey;
- this.paths = Arrays.asList(path);
- }
-
- /**
- * Constructor.
- *
- * @param mdcKey the key to the filter rule
- * @param paths the list of potential paths to the key
- */
- public FilterRule(String mdcKey, List<String> paths) {
- this.mdcKey = mdcKey;
- this.paths = paths;
- }
-
- protected void setMdcKey(String mdcKey) {
- if (StringUtils.isBlank(mdcKey)) {
- throw new IllegalArgumentException(MDC_KEY_ERROR);
- }
- this.mdcKey = mdcKey;
- }
-
- protected void setPaths(List<String> paths) {
- if (nullOrEmpty(paths)) {
- throw new IllegalArgumentException(JSON_PATH_ERROR);
- }
- this.paths = paths;
- }
-
- protected void addPaths(List<String> paths) {
- if (nullOrEmpty(paths)) {
- throw new IllegalArgumentException(JSON_PATH_ERROR);
- }
- this.paths.addAll(paths);
- }
-
- protected void addPath(String path) {
- if (StringUtils.isBlank(path)) {
- throw new IllegalArgumentException(JSON_PATH_ERROR);
- }
- this.paths.add(path);
- }
- }
-
- protected MdcTopicFilter(String rawFilters) {
- for (String filter : rawFilters.split("\\s*,\\s*")) {
- FilterRule rule = createFilterRule(filter);
- rules.put(rule.mdcKey, rule);
- }
- }
-
- private FilterRule createFilterRule(String filter) {
- String[] filterKeyPaths = filter.split("\\s*=\\s*");
- if (filterKeyPaths.length != 2) {
- throw new IllegalArgumentException("could not parse filter rule");
- }
-
- String filterKey = filterKeyPaths[0];
- String paths = filterKeyPaths[1];
- List<String> filterPaths = new ArrayList<>(Arrays.asList(paths.split("(?<!\\|)\\|(?!\\|)")));
- return new FilterRule(filterKey, filterPaths);
- }
-
- /**
- * Gets all the filter rules for the topic.
- *
- * @return an array list of the rules for the topic
- */
- protected List<FilterRule> getFilterRule() {
- return new ArrayList<>(rules.values());
- }
-
- /**
- * Gets the filter rule for the specified key.
- *
- * @param mdcKey the key to the filter rule
- * @return the filter rule associated with the key
- */
- protected FilterRule getFilterRule(String mdcKey) {
- if (StringUtils.isBlank(mdcKey)) {
- throw new IllegalArgumentException(MDC_KEY_ERROR);
- }
- return rules.get(mdcKey);
- }
-
- /**
- * Adds a filter rule for the specified key and path.
- *
- * @param mdcKey the key to the filter rule
- * @param path the json path to the key
- * @return the filter rule that was added for the topic
- */
- protected FilterRule addFilterRule(String mdcKey, String path) {
- if (StringUtils.isBlank(path)) {
- throw new IllegalArgumentException(JSON_PATH_ERROR);
- }
- return addFilterRule(mdcKey, Arrays.asList(path));
- }
-
- /**
- * Adds a filter rule for the specified key and paths.
- *
- * @param mdcKey the key to the filter rule
- * @param paths the list of potential paths to the key
- * @return the filter rule that was added for the topic
- */
- protected FilterRule addFilterRule(String mdcKey, List<String> paths) {
- if (StringUtils.isBlank(mdcKey)) {
- throw new IllegalArgumentException(MDC_KEY_ERROR);
- }
-
- if (nullOrEmpty(paths)) {
- throw new IllegalArgumentException(JSON_PATH_ERROR);
- }
-
- if (rules.containsKey(mdcKey)) {
- throw new IllegalArgumentException("a filter rule already exists for key: " + mdcKey);
- }
-
- FilterRule rule = new FilterRule(mdcKey, paths);
- rules.put(mdcKey, rule);
- return rule;
- }
-
- private static boolean nullOrEmpty(List<String> paths) {
- return paths == null || paths.isEmpty();
- }
-
- /**
- * Modifies an existing filter rule by adding the specified path.
- *
- * @param mdcKey the key to the filter rule
- * @param path the path to the key
- * @return the filter rule that was modified
- */
- protected FilterRule modifyFilterRule(String mdcKey, String path) {
- if (StringUtils.isBlank(path)) {
- throw new IllegalArgumentException(JSON_PATH_ERROR);
- }
- return modifyFilterRule(mdcKey, Arrays.asList(path));
- }
-
- /**
- * Modifies an existing filter rule by adding the specified paths.
- *
- * @param mdcKey the key to the filter rule
- * @param paths the list of potential paths to the key
- * @return the filter rule that was modified
- */
- protected FilterRule modifyFilterRule(String mdcKey, List<String> paths) {
- if (StringUtils.isBlank(mdcKey)) {
- throw new IllegalArgumentException(MDC_KEY_ERROR);
- }
-
- if (nullOrEmpty(paths)) {
- throw new IllegalArgumentException(JSON_PATH_ERROR);
- }
-
- if (!rules.containsKey(mdcKey)) {
- throw new IllegalArgumentException("a filter rule doesn't exist for key: " + mdcKey);
- }
-
- FilterRule rule = rules.get(mdcKey);
- rule.addPaths(paths);
- return rule;
- }
-
- /**
- * Modifies an existing filter rule's key and replaces the paths with the specified
- * paths.
- *
- * @param oldMdcKey the old key to the filter rule
- * @param newMdcKey the new key to the filter rule
- * @param paths the list of potential paths to the key
- * @return the filter rule that was modified
- */
- protected FilterRule modifyFilterRule(String oldMdcKey, String newMdcKey, List<String> paths) {
- if (StringUtils.isBlank(oldMdcKey)) {
- throw new IllegalArgumentException("current mdcKey must be provided");
- }
-
- if (StringUtils.isBlank(newMdcKey)) {
- throw new IllegalArgumentException("new mdcKey must be provided");
- }
-
- if (oldMdcKey.equals(newMdcKey)) {
- throw new IllegalArgumentException("the old and new mdcKey are equivalent");
- }
- if (nullOrEmpty(paths)) {
- throw new IllegalArgumentException(JSON_PATH_ERROR);
- }
-
- if (rules.containsKey(newMdcKey)) {
- throw new IllegalArgumentException("a filter rule already exists for key: " + newMdcKey);
- }
-
- FilterRule rule = rules.remove(oldMdcKey);
- if (rule == null) {
- throw new IllegalArgumentException("a filter rule doesn't exist for key: " + oldMdcKey);
- }
-
- rule.setMdcKey(newMdcKey);
- rule.setPaths(paths);
- rules.put(newMdcKey, rule);
- return rule;
- }
-
- /**
- * Deletes all filter rules for the topic filter.
- */
- protected void deleteFilterRule() {
- rules.clear();
- }
-
- /**
- * Deletes an existing filter rule.
- *
- * @param mdcKey the key to the filter rule
- * @return the filter rule that was deleted
- */
- protected FilterRule deleteFilterRule(String mdcKey) {
- if (StringUtils.isBlank(mdcKey)) {
- throw new IllegalArgumentException(MDC_KEY_ERROR);
- }
- return rules.remove(mdcKey);
- }
-
- /**
- * Finds all fields for each topic filter rule. The results are stored in a map that
- * is indexed by the MDC key. Each MDC key has a list of results as multiple
- * occurrences of a key can be found in a JSON document.
- *
- * @param json the json string to be parsed
- * @return a map of mdc keys and list of results for each key
- */
- protected Map<String, List<String>> find(String json) {
- Map<String, List<String>> results = new HashMap<>();
- for (FilterRule rule : rules.values()) {
- List<String> matches = new ArrayList<>();
- for (String path : rule.getPaths()) {
-
- try {
- matches = JsonProtocolFilter.filter(json, path);
- } catch (Exception e) {
- logger.debug("Could not filter on path {} because of {}", path, e.getMessage(), e);
- }
-
- if (!matches.isEmpty()) {
- break;
- } else {
- logger.error("Could not find path {} in json {}", path, json);
- }
-
- }
- results.put(rule.getMdcKey(), matches);
- }
- return results;
- }
-
- /**
- * Finds all occurrences of a field in a JSON document based on the filter rule paths.
- *
- * @param json the json string to be parsed
- * @return a list of matches from the JSON document
- */
- protected List<String> find(String json, String mdcKey) {
- List<String> matches = new ArrayList<>();
- for (String path : rules.get(mdcKey).getPaths()) {
-
- try {
- matches = JsonProtocolFilter.filter(json, path);
- } catch (Exception e) {
- logger.debug("Could not filter on path {} because of {}", path, e.getMessage(), e);
- }
-
- if (!matches.isEmpty()) {
- break;
- }
-
- }
-
- if (matches.isEmpty()) {
- logger.error("Could not find any matches for key {} in json {}", mdcKey, json);
- }
-
- return matches;
- }
-}
+/*
+ * ============LICENSE_START=======================================================
+ * feature-mdc-filters
+ * ================================================================================
+ * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ */
+
+package org.onap.policy.drools.mdc.filters;
+
+import com.att.aft.dme2.internal.apache.commons.lang3.StringUtils;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+import lombok.Getter;
+import org.onap.policy.drools.protocol.coders.JsonProtocolFilter;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+public class MdcTopicFilter {
+
+ private static final Logger logger = LoggerFactory.getLogger(MdcTopicFilter.class);
+
+ public static final String MDC_KEY_ERROR = "mdcKey must be provided";
+ public static final String JSON_PATH_ERROR = "json path(s) must be provided";
+
+ private Map<String, FilterRule> rules = new HashMap<>();
+
+ @Getter
+ public static class FilterRule {
+ private String mdcKey;
+ private List<String> paths;
+
+ public FilterRule(String mdcKey, String path) {
+ this.mdcKey = mdcKey;
+ this.paths = Arrays.asList(path);
+ }
+
+ /**
+ * Constructor.
+ *
+ * @param mdcKey the key to the filter rule
+ * @param paths the list of potential paths to the key
+ */
+ public FilterRule(String mdcKey, List<String> paths) {
+ this.mdcKey = mdcKey;
+ this.paths = paths;
+ }
+
+ protected void setMdcKey(String mdcKey) {
+ if (StringUtils.isBlank(mdcKey)) {
+ throw new IllegalArgumentException(MDC_KEY_ERROR);
+ }
+ this.mdcKey = mdcKey;
+ }
+
+ protected void setPaths(List<String> paths) {
+ if (nullOrEmpty(paths)) {
+ throw new IllegalArgumentException(JSON_PATH_ERROR);
+ }
+ this.paths = paths;
+ }
+
+ protected void addPaths(List<String> paths) {
+ if (nullOrEmpty(paths)) {
+ throw new IllegalArgumentException(JSON_PATH_ERROR);
+ }
+ this.paths.addAll(paths);
+ }
+
+ protected void addPath(String path) {
+ if (StringUtils.isBlank(path)) {
+ throw new IllegalArgumentException(JSON_PATH_ERROR);
+ }
+ this.paths.add(path);
+ }
+ }
+
+ protected MdcTopicFilter(String rawFilters) {
+ for (String filter : rawFilters.split("\\s*,\\s*")) {
+ FilterRule rule = createFilterRule(filter);
+ rules.put(rule.mdcKey, rule);
+ }
+ }
+
+ private FilterRule createFilterRule(String filter) {
+ String[] filterKeyPaths = filter.split("\\s*=\\s*");
+ if (filterKeyPaths.length != 2) {
+ throw new IllegalArgumentException("could not parse filter rule");
+ }
+
+ String filterKey = filterKeyPaths[0];
+ String paths = filterKeyPaths[1];
+ List<String> filterPaths = new ArrayList<>(Arrays.asList(paths.split("(?<!\\|)\\|(?!\\|)")));
+ return new FilterRule(filterKey, filterPaths);
+ }
+
+ /**
+ * Gets all the filter rules for the topic.
+ *
+ * @return an array list of the rules for the topic
+ */
+ protected List<FilterRule> getFilterRule() {
+ return new ArrayList<>(rules.values());
+ }
+
+ /**
+ * Gets the filter rule for the specified key.
+ *
+ * @param mdcKey the key to the filter rule
+ * @return the filter rule associated with the key
+ */
+ protected FilterRule getFilterRule(String mdcKey) {
+ if (StringUtils.isBlank(mdcKey)) {
+ throw new IllegalArgumentException(MDC_KEY_ERROR);
+ }
+ return rules.get(mdcKey);
+ }
+
+ /**
+ * Adds a filter rule for the specified key and path.
+ *
+ * @param mdcKey the key to the filter rule
+ * @param path the json path to the key
+ * @return the filter rule that was added for the topic
+ */
+ protected FilterRule addFilterRule(String mdcKey, String path) {
+ if (StringUtils.isBlank(path)) {
+ throw new IllegalArgumentException(JSON_PATH_ERROR);
+ }
+ return addFilterRule(mdcKey, Arrays.asList(path));
+ }
+
+ /**
+ * Adds a filter rule for the specified key and paths.
+ *
+ * @param mdcKey the key to the filter rule
+ * @param paths the list of potential paths to the key
+ * @return the filter rule that was added for the topic
+ */
+ protected FilterRule addFilterRule(String mdcKey, List<String> paths) {
+ if (StringUtils.isBlank(mdcKey)) {
+ throw new IllegalArgumentException(MDC_KEY_ERROR);
+ }
+
+ if (nullOrEmpty(paths)) {
+ throw new IllegalArgumentException(JSON_PATH_ERROR);
+ }
+
+ if (rules.containsKey(mdcKey)) {
+ throw new IllegalArgumentException("a filter rule already exists for key: " + mdcKey);
+ }
+
+ FilterRule rule = new FilterRule(mdcKey, paths);
+ rules.put(mdcKey, rule);
+ return rule;
+ }
+
+ private static boolean nullOrEmpty(List<String> paths) {
+ return paths == null || paths.isEmpty();
+ }
+
+ /**
+ * Modifies an existing filter rule by adding the specified path.
+ *
+ * @param mdcKey the key to the filter rule
+ * @param path the path to the key
+ * @return the filter rule that was modified
+ */
+ protected FilterRule modifyFilterRule(String mdcKey, String path) {
+ if (StringUtils.isBlank(path)) {
+ throw new IllegalArgumentException(JSON_PATH_ERROR);
+ }
+ return modifyFilterRule(mdcKey, Arrays.asList(path));
+ }
+
+ /**
+ * Modifies an existing filter rule by adding the specified paths.
+ *
+ * @param mdcKey the key to the filter rule
+ * @param paths the list of potential paths to the key
+ * @return the filter rule that was modified
+ */
+ protected FilterRule modifyFilterRule(String mdcKey, List<String> paths) {
+ if (StringUtils.isBlank(mdcKey)) {
+ throw new IllegalArgumentException(MDC_KEY_ERROR);
+ }
+
+ if (nullOrEmpty(paths)) {
+ throw new IllegalArgumentException(JSON_PATH_ERROR);
+ }
+
+ if (!rules.containsKey(mdcKey)) {
+ throw new IllegalArgumentException("a filter rule doesn't exist for key: " + mdcKey);
+ }
+
+ FilterRule rule = rules.get(mdcKey);
+ rule.addPaths(paths);
+ return rule;
+ }
+
+ /**
+ * Modifies an existing filter rule's key and replaces the paths with the specified
+ * paths.
+ *
+ * @param oldMdcKey the old key to the filter rule
+ * @param newMdcKey the new key to the filter rule
+ * @param paths the list of potential paths to the key
+ * @return the filter rule that was modified
+ */
+ protected FilterRule modifyFilterRule(String oldMdcKey, String newMdcKey, List<String> paths) {
+ if (StringUtils.isBlank(oldMdcKey)) {
+ throw new IllegalArgumentException("current mdcKey must be provided");
+ }
+
+ if (StringUtils.isBlank(newMdcKey)) {
+ throw new IllegalArgumentException("new mdcKey must be provided");
+ }
+
+ if (oldMdcKey.equals(newMdcKey)) {
+ throw new IllegalArgumentException("the old and new mdcKey are equivalent");
+ }
+ if (nullOrEmpty(paths)) {
+ throw new IllegalArgumentException(JSON_PATH_ERROR);
+ }
+
+ if (rules.containsKey(newMdcKey)) {
+ throw new IllegalArgumentException("a filter rule already exists for key: " + newMdcKey);
+ }
+
+ FilterRule rule = rules.remove(oldMdcKey);
+ if (rule == null) {
+ throw new IllegalArgumentException("a filter rule doesn't exist for key: " + oldMdcKey);
+ }
+
+ rule.setMdcKey(newMdcKey);
+ rule.setPaths(paths);
+ rules.put(newMdcKey, rule);
+ return rule;
+ }
+
+ /**
+ * Deletes all filter rules for the topic filter.
+ */
+ protected void deleteFilterRule() {
+ rules.clear();
+ }
+
+ /**
+ * Deletes an existing filter rule.
+ *
+ * @param mdcKey the key to the filter rule
+ * @return the filter rule that was deleted
+ */
+ protected FilterRule deleteFilterRule(String mdcKey) {
+ if (StringUtils.isBlank(mdcKey)) {
+ throw new IllegalArgumentException(MDC_KEY_ERROR);
+ }
+ return rules.remove(mdcKey);
+ }
+
+ /**
+ * Finds all fields for each topic filter rule. The results are stored in a map that
+ * is indexed by the MDC key. Each MDC key has a list of results as multiple
+ * occurrences of a key can be found in a JSON document.
+ *
+ * @param json the json string to be parsed
+ * @return a map of mdc keys and list of results for each key
+ */
+ protected Map<String, List<String>> find(String json) {
+ Map<String, List<String>> results = new HashMap<>();
+ for (FilterRule rule : rules.values()) {
+ List<String> matches = new ArrayList<>();
+ for (String path : rule.getPaths()) {
+
+ try {
+ matches = JsonProtocolFilter.filter(json, path);
+ } catch (Exception e) {
+ logger.debug("Could not filter on path {} because of {}", path, e.getMessage(), e);
+ }
+
+ if (!matches.isEmpty()) {
+ break;
+ } else {
+ logger.error("Could not find path {} in json {}", path, json);
+ }
+
+ }
+ results.put(rule.getMdcKey(), matches);
+ }
+ return results;
+ }
+
+ /**
+ * Finds all occurrences of a field in a JSON document based on the filter rule paths.
+ *
+ * @param json the json string to be parsed
+ * @return a list of matches from the JSON document
+ */
+ protected List<String> find(String json, String mdcKey) {
+ List<String> matches = new ArrayList<>();
+ for (String path : rules.get(mdcKey).getPaths()) {
+
+ try {
+ matches = JsonProtocolFilter.filter(json, path);
+ } catch (Exception e) {
+ logger.debug("Could not filter on path {} because of {}", path, e.getMessage(), e);
+ }
+
+ if (!matches.isEmpty()) {
+ break;
+ }
+
+ }
+
+ if (matches.isEmpty()) {
+ logger.error("Could not find any matches for key {} in json {}", mdcKey, json);
+ }
+
+ return matches;
+ }
+}
diff --git a/feature-mdc-filters/src/test/java/org/onap/policy/drools/mdc/filters/MdcFilterFeatureTest.java b/feature-mdc-filters/src/test/java/org/onap/policy/drools/mdc/filters/MdcFilterFeatureTest.java
index 8efa96d6..51ff3e92 100755
--- a/feature-mdc-filters/src/test/java/org/onap/policy/drools/mdc/filters/MdcFilterFeatureTest.java
+++ b/feature-mdc-filters/src/test/java/org/onap/policy/drools/mdc/filters/MdcFilterFeatureTest.java
@@ -1,169 +1,169 @@
-/*
- * ============LICENSE_START=======================================================
- * feature-mdc-filters
- * ================================================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
- * ================================================================================
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * ============LICENSE_END=========================================================
- */
-
-package org.onap.policy.drools.mdc.filters;
-
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
-import static org.mockito.Mockito.doReturn;
-import static org.mockito.Mockito.mock;
-
-import java.io.IOException;
-import java.nio.file.Files;
-import java.nio.file.Paths;
-import java.util.List;
-import java.util.Properties;
-import org.junit.Before;
-import org.junit.Test;
-import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure;
-import org.onap.policy.common.endpoints.event.comm.TopicEndpointManager;
-import org.onap.policy.common.endpoints.event.comm.TopicSink;
-import org.onap.policy.common.endpoints.event.comm.TopicSource;
-import org.onap.policy.common.endpoints.utils.NetLoggerUtil.EventType;
-import org.onap.policy.drools.system.PolicyController;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import org.slf4j.MDC;
-
-public class MdcFilterFeatureTest {
-
- /**
- * Logger.
- */
- private static final Logger logger = LoggerFactory.getLogger(MdcFilterFeatureTest.class);
-
- /**
- * Test topic names for mdc topic filters.
- */
- private static final String TEST_TOPIC_A = "org.onap.policy.test-topic-a";
- private static final String TEST_TOPIC_B = "org.onap.policy.test-topic-b";
-
- /**
- * The mock PolicyController to be used for the junits.
- */
- private PolicyController controller;
-
- /**
- * The mock properties to be used for the junits.
- */
- private Properties props;
-
- /**
- * An instance of the MdcFilterFeature.
- */
- private MdcFilterFeature mdcFilterFeature;
-
- /**
- * Sample json string to be logged.
- */
- private String message;
-
- /**
- * Setup.
- * @throws IOException thrown if onset.json file could not be read
- */
- @Before
- public void setUp() throws IOException {
- message = new String(Files.readAllBytes(Paths.get("src/test/resources/onset.json")));
-
- props = mockFeatureProperties();
- controller = mock(PolicyController.class);
-
- props.setProperty("dmaap.source.topics", TEST_TOPIC_A);
- props.setProperty("dmaap.source.topics." + TEST_TOPIC_A + ".servers", "http://testing123.com/");
- props.setProperty("noop.sink.topics", TEST_TOPIC_B);
-
- List<TopicSource> topicSources = TopicEndpointManager.getManager().addTopicSources(props);
- doReturn(topicSources).when(controller).getTopicSources();
-
- List<TopicSink> topicSinks = TopicEndpointManager.getManager().addTopicSinks(props);
- doReturn(topicSinks).when(controller).getTopicSinks();
-
- mdcFilterFeature = new MdcFilterFeatureImpl();
- mdcFilterFeature.afterCreate(controller);
- }
-
- /**
- * Tests extracting fields from a JSON message and place them in the MDC and
- * then clearing the MDC.
- */
- @Test
- public void mdcLogTest() {
- mdcFilterFeature.beforeLog(logger, EventType.IN,
- CommInfrastructure.DMAAP, TEST_TOPIC_A, message);
-
- assertEquals("8c1b8bd8-06f7-493f-8ed7-daaa4cc481bc", MDC.get("requestID"));
- assertEquals("CL-TEST", MDC.get("closedLoopControlName"));
-
- assertNotNull(MDC.getCopyOfContextMap());
-
- mdcFilterFeature.afterLog(logger, EventType.IN,
- CommInfrastructure.DMAAP, TEST_TOPIC_A, message);
-
- assertNull(MDC.getCopyOfContextMap());
- }
-
- /**
- * Tests that the feature does not search for fields in a JSON message
- * if there is not an MdcTopicFilter object for the generated key.
- */
- @Test
- public void noTopicFilterTest() {
- mdcFilterFeature.beforeLog(logger, EventType.OUT,
- CommInfrastructure.NOOP, "no-topic", message);
-
- assertEquals("OUT", MDC.get("networkEventType"));
- assertEquals("NOOP", MDC.get("networkProtocol"));
- assertEquals("no-topic", MDC.get("networkTopic"));
- assertNull(MDC.get("requestID"));
- }
-
- /**
- * Creates a simple properties map containing an mdc filter for a test
- * topic.
- *
- * @return a properties map with mdc filter properties.
- */
- private Properties mockFeatureProperties() {
- Properties props = new Properties();
-
- String key = "dmaap.source.topics." + TEST_TOPIC_A + ".mdcFilters";
- String value = "requestID=$.requestID,closedLoopControlName=$.closedLoopControlName";
- props.setProperty(key, value);
-
- return props;
- }
-
- /**
- * Subclass of MdcFilterFeature for junit usage.
- */
- private class MdcFilterFeatureImpl extends MdcFilterFeature {
-
- public MdcFilterFeatureImpl() {
- super();
- }
-
- @Override
- protected Properties getFeatureProps() {
- return props;
- }
- }
-}
+/*
+ * ============LICENSE_START=======================================================
+ * feature-mdc-filters
+ * ================================================================================
+ * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ */
+
+package org.onap.policy.drools.mdc.filters;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
+import static org.mockito.Mockito.doReturn;
+import static org.mockito.Mockito.mock;
+
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.Paths;
+import java.util.List;
+import java.util.Properties;
+import org.junit.Before;
+import org.junit.Test;
+import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure;
+import org.onap.policy.common.endpoints.event.comm.TopicEndpointManager;
+import org.onap.policy.common.endpoints.event.comm.TopicSink;
+import org.onap.policy.common.endpoints.event.comm.TopicSource;
+import org.onap.policy.common.endpoints.utils.NetLoggerUtil.EventType;
+import org.onap.policy.drools.system.PolicyController;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import org.slf4j.MDC;
+
+public class MdcFilterFeatureTest {
+
+ /**
+ * Logger.
+ */
+ private static final Logger logger = LoggerFactory.getLogger(MdcFilterFeatureTest.class);
+
+ /**
+ * Test topic names for mdc topic filters.
+ */
+ private static final String TEST_TOPIC_A = "org.onap.policy.test-topic-a";
+ private static final String TEST_TOPIC_B = "org.onap.policy.test-topic-b";
+
+ /**
+ * The mock PolicyController to be used for the junits.
+ */
+ private PolicyController controller;
+
+ /**
+ * The mock properties to be used for the junits.
+ */
+ private Properties props;
+
+ /**
+ * An instance of the MdcFilterFeature.
+ */
+ private MdcFilterFeature mdcFilterFeature;
+
+ /**
+ * Sample json string to be logged.
+ */
+ private String message;
+
+ /**
+ * Setup.
+ * @throws IOException thrown if onset.json file could not be read
+ */
+ @Before
+ public void setUp() throws IOException {
+ message = new String(Files.readAllBytes(Paths.get("src/test/resources/onset.json")));
+
+ props = mockFeatureProperties();
+ controller = mock(PolicyController.class);
+
+ props.setProperty("dmaap.source.topics", TEST_TOPIC_A);
+ props.setProperty("dmaap.source.topics." + TEST_TOPIC_A + ".servers", "http://testing123.com/");
+ props.setProperty("noop.sink.topics", TEST_TOPIC_B);
+
+ List<TopicSource> topicSources = TopicEndpointManager.getManager().addTopicSources(props);
+ doReturn(topicSources).when(controller).getTopicSources();
+
+ List<TopicSink> topicSinks = TopicEndpointManager.getManager().addTopicSinks(props);
+ doReturn(topicSinks).when(controller).getTopicSinks();
+
+ mdcFilterFeature = new MdcFilterFeatureImpl();
+ mdcFilterFeature.afterCreate(controller);
+ }
+
+ /**
+ * Tests extracting fields from a JSON message and place them in the MDC and
+ * then clearing the MDC.
+ */
+ @Test
+ public void mdcLogTest() {
+ mdcFilterFeature.beforeLog(logger, EventType.IN,
+ CommInfrastructure.DMAAP, TEST_TOPIC_A, message);
+
+ assertEquals("8c1b8bd8-06f7-493f-8ed7-daaa4cc481bc", MDC.get("requestID"));
+ assertEquals("CL-TEST", MDC.get("closedLoopControlName"));
+
+ assertNotNull(MDC.getCopyOfContextMap());
+
+ mdcFilterFeature.afterLog(logger, EventType.IN,
+ CommInfrastructure.DMAAP, TEST_TOPIC_A, message);
+
+ assertNull(MDC.getCopyOfContextMap());
+ }
+
+ /**
+ * Tests that the feature does not search for fields in a JSON message
+ * if there is not an MdcTopicFilter object for the generated key.
+ */
+ @Test
+ public void noTopicFilterTest() {
+ mdcFilterFeature.beforeLog(logger, EventType.OUT,
+ CommInfrastructure.NOOP, "no-topic", message);
+
+ assertEquals("OUT", MDC.get("networkEventType"));
+ assertEquals("NOOP", MDC.get("networkProtocol"));
+ assertEquals("no-topic", MDC.get("networkTopic"));
+ assertNull(MDC.get("requestID"));
+ }
+
+ /**
+ * Creates a simple properties map containing an mdc filter for a test
+ * topic.
+ *
+ * @return a properties map with mdc filter properties.
+ */
+ private Properties mockFeatureProperties() {
+ Properties props = new Properties();
+
+ String key = "dmaap.source.topics." + TEST_TOPIC_A + ".mdcFilters";
+ String value = "requestID=$.requestID,closedLoopControlName=$.closedLoopControlName";
+ props.setProperty(key, value);
+
+ return props;
+ }
+
+ /**
+ * Subclass of MdcFilterFeature for junit usage.
+ */
+ private class MdcFilterFeatureImpl extends MdcFilterFeature {
+
+ public MdcFilterFeatureImpl() {
+ super();
+ }
+
+ @Override
+ protected Properties getFeatureProps() {
+ return props;
+ }
+ }
+}
diff --git a/feature-mdc-filters/src/test/java/org/onap/policy/drools/mdc/filters/MdcTopicFilterTest.java b/feature-mdc-filters/src/test/java/org/onap/policy/drools/mdc/filters/MdcTopicFilterTest.java
index 568a316b..7fd65a29 100755
--- a/feature-mdc-filters/src/test/java/org/onap/policy/drools/mdc/filters/MdcTopicFilterTest.java
+++ b/feature-mdc-filters/src/test/java/org/onap/policy/drools/mdc/filters/MdcTopicFilterTest.java
@@ -1,661 +1,661 @@
-/*
- * ============LICENSE_START=======================================================
- * feature-mdc-filters
- * ================================================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
- * ================================================================================
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * ============LICENSE_END=========================================================
- */
-
-package org.onap.policy.drools.mdc.filters;
-
-import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertTrue;
-
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collections;
-import java.util.List;
-import java.util.Map;
-import org.junit.Test;
-import org.onap.policy.drools.mdc.filters.MdcTopicFilter.FilterRule;
-
-public class MdcTopicFilterTest {
-
- /**
- * Test the simple case of having one filter rule for a key.
- */
- @Test
- public void singleFilterOnePathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- FilterRule rule = topicFilter.getFilterRule("requestID");
- assertEquals("requestID", rule.getMdcKey());
- assertEquals("[$.requestID]", rule.getPaths().toString());
- }
-
- /**
- * Tests having one filter rule with a set of potential paths to the key.
- */
- @Test
- public void singleFilterMultiPathTest() {
- String topicFilterProp = "requestID=$.requestID|$.request-id";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- FilterRule rule = topicFilter.getFilterRule("requestID");
- assertEquals("requestID", rule.getMdcKey());
- assertEquals(2, rule.getPaths().size());
- assertEquals("[$.requestID, $.request-id]", rule.getPaths().toString());
- }
-
- /**
- * Tests having two filter rules that each have one key/path pair.
- */
- @Test
- public void multiFilterSinglePathTest() {
- String topicFilterProp = "requestID=$.requestID,closedLoopControlName=$.closedLoopControlName";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- FilterRule rule = topicFilter.getFilterRule("requestID");
- assertEquals("requestID", rule.getMdcKey());
- assertEquals(1, rule.getPaths().size());
- assertEquals("[$.requestID]", rule.getPaths().toString());
-
- FilterRule rule2 = topicFilter.getFilterRule("closedLoopControlName");
- assertEquals("closedLoopControlName", rule2.getMdcKey());
- assertEquals(1, rule2.getPaths().size());
- assertEquals("[$.closedLoopControlName]", rule2.getPaths().toString());
- }
-
- /**
- * Tests having two filter rules that each have two key/path pairs.
- */
- @Test
- public void multiFilterMultiPathTest() {
- String topicFilterProp = "requestID=$.requestID|$.body.request-id,"
- + "closedLoopControlName=$.closedLoopControlName"
- + "|$.body.closedLoopControlName";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- FilterRule rule = topicFilter.getFilterRule("requestID");
- assertEquals("requestID", rule.getMdcKey());
- assertEquals(2, rule.getPaths().size());
- assertEquals("[$.requestID, $.body.request-id]", rule.getPaths().toString());
-
- FilterRule rule2 = topicFilter.getFilterRule("closedLoopControlName");
- assertEquals("closedLoopControlName", rule2.getMdcKey());
- assertEquals(2, rule2.getPaths().size());
- assertEquals("[$.closedLoopControlName, $.body.closedLoopControlName]", rule2.getPaths().toString());
- }
-
- /**
- * Tests that the regex split logic for '|' in the feature code doesn't
- * break parsing when "||" is used as a predicate in a JsonPath query.
- */
- @Test
- public void addOrPredicateFilterTest() {
- String topicFilterProp = "requestID=$.requestID||$.body.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- assertEquals(1, topicFilter.getFilterRule().size());
- assertEquals("requestID", topicFilter.getFilterRule("requestID").getMdcKey());
- assertEquals(Arrays.asList("$.requestID||$.body.requestID"), topicFilter
- .getFilterRule("requestID").getPaths());
- }
-
- /**
- * Tests getting all filter rules for a given topic.
- */
- @Test
- public void getAllFilterRulesTest() {
- String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
- + "closedLoopControlName=$.closedLoopControlName";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- assertEquals(3, topicFilter.getFilterRule().size());
- }
-
- /**
- * Tests getting a filter rule by its key.
- */
- @Test
- public void getFilterRuleTest() {
- String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
- + "closedLoopControlName=$.closedLoopControlName";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- FilterRule rule = topicFilter.getFilterRule("requestID");
- assertNotNull(rule);
- }
-
- /**
- * Tests throwing an exception for passing in a null key.
- */
- @Test(expected = IllegalArgumentException.class)
- public void getFilterRuleNullKeyTest() {
- String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.getFilterRule(null);
- }
-
- /**
- * Tests throwing an exception for passing in an empty key.
- */
- @Test(expected = IllegalArgumentException.class)
- public void getFilterRuleEmptyKeyTest() {
- String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.getFilterRule("");
- }
-
- /**
- * Tests adding a filter rule with a single path.
- */
- @Test
- public void addFilterRuleSinglePathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- String key = "subRequestID";
- String path = "$.subRequestID";
- FilterRule rule = topicFilter.addFilterRule(key, path);
- assertEquals(topicFilter.getFilterRule(key), rule);
- }
-
- /**
- * Tests adding a filter rule with multiple paths.
- */
- @Test
- public void addFilterRuleMultiPathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- String key = "subRequestID";
- List<String> paths = Arrays.asList("$.subRequestID", "$.sub-request-id");
- FilterRule rule = topicFilter.addFilterRule(key, paths);
- assertEquals(topicFilter.getFilterRule(key), rule);
- }
-
- /**
- * Tests throwing an exception for passing a null key and a
- * single path.
- */
- @Test(expected = IllegalArgumentException.class)
- public void addFilterRuleNullKeyStringPathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.addFilterRule(null, "$.subRequestID");
- }
-
- /**
- * Tests throwing an exception for passing a null key and a list
- * of paths.
- */
- @Test(expected = IllegalArgumentException.class)
- public void addFilterRuleNullKeyPathListTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.addFilterRule(null, Arrays.asList("$.subRequestID"));
- }
-
- /**
- * Tests throwing an exception for passing an empty key and
- * a single path.
- */
- @Test(expected = IllegalArgumentException.class)
- public void addFilterRuleEmptyKeyStringPathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.addFilterRule("", "$.subRequestID");
- }
-
- /**
- * Tests throwing an exception for passing an empty key and
- * a list of paths.
- */
- @Test(expected = IllegalArgumentException.class)
- public void addFilterRuleEmptyKeyPathListTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.addFilterRule("", Arrays.asList("$.subRequestID"));
- }
-
- /**
- * Tests throwing an exception for passing an empty path string.
- */
- @Test(expected = IllegalArgumentException.class)
- public void addFilterRuleEmptyPathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.addFilterRule("subRequestID", "");
- }
-
- /**
- * Tests throwing an exception for passing an empty paths list.
- */
- @Test(expected = IllegalArgumentException.class)
- public void addFilterRuleEmptyPathsTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.addFilterRule("subRequestID", Arrays.asList());
- }
-
- /**
- * Tests throwing an exception for trying to add a filter with a key that
- * already exists with a single filter.
- */
- @Test(expected = IllegalArgumentException.class)
- public void addExistingFilterRuleStringTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.addFilterRule("requestID", "$.test");
- }
-
- /**
- * Tests throwing an exception for trying to add a filter with a key that
- * already exists with a list of filters.
- */
- @Test(expected = IllegalArgumentException.class)
- public void addExistingFilterRuleListTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.addFilterRule("requestID", Arrays.asList("$.test"));
- }
-
- @Test
- public void createFilterRuleExceptionTest() {
- assertThatIllegalArgumentException().isThrownBy(() -> new MdcTopicFilter("invalid filter"))
- .withMessage("could not parse filter rule");
- }
-
- /**
- * Tests modifying a filter rule to add a new path.
- */
- @Test
- public void modifyFilterRuleSinglePathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- FilterRule rule = topicFilter.modifyFilterRule("requestID", "$.request-id");
- assertEquals(topicFilter.getFilterRule("requestID"), rule);
- assertEquals(Arrays.asList("$.requestID", "$.request-id"), rule.getPaths());
- }
-
- /**
- * Tests modifying a filter rule to add a list of new paths.
- */
- @Test
- public void modifyFilterRuleMultiPathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- FilterRule rule = topicFilter.modifyFilterRule("requestID",
- Arrays.asList("$.request-id", "$.requestId"));
- assertEquals(topicFilter.getFilterRule("requestID"), rule);
- assertEquals(
- Arrays.asList("$.requestID", "$.request-id", "$.requestId"),
- rule.getPaths());
- }
-
- @Test
- public void modifyFilterRuleMultiPathExceptionTest() {
- MdcTopicFilter filter = new MdcTopicFilter("abc=$a.value");
- assertThatIllegalArgumentException()
- .isThrownBy(() -> filter.modifyFilterRule("def", "abc", Arrays.asList("$.b", "$.c")))
- .withMessage("a filter rule already exists for key: abc");
- }
-
- /**
- * Tests modifying a filter rule key.
- */
- @Test
- public void modifyFilterRuleKeyTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- FilterRule rule = topicFilter.modifyFilterRule("requestID",
- "request-id", Arrays.asList("$.request-id"));
- assertEquals(topicFilter.getFilterRule("request-id"), rule);
- assertEquals("[$.request-id]", rule.getPaths().toString());
- }
-
- /**
- * Tests throwing an exception when passing a null key and
- * a single path.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleNullKeyStringPathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule(null, "$.request-id");
- }
-
- /**
- * Tests throwing an exception when passing a null key and
- * a list of multiple paths.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleNullKeyPathListTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule(null, Arrays.asList("$.request-id"));
- }
-
- /**
- * Tests throwing an exception when passing an empty key and
- * a single path.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleEmptyKeyStringPathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("", "$.request-id");
- }
-
- /**
- * Tests throwing an exception when passing an empty key and
- * a list of multiple paths.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleEmptyKeyPathListTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("", Arrays.asList("$.request-id"));
- }
-
- /**
- * Tests throwing an exception when passing an empty string path.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleEmptyPathStringTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("requestID", "");
- }
-
- /**
- * Tests throwing an exception when passing an empty list of paths.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleEmptyPathListTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("requestID", Arrays.asList());
- }
-
- /**
- * Tests throwing an exception when passing a key that is
- * not in the filter rules map and a string path.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleMissingKeyStringPathTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("request-id", "$.request-id");
- }
-
- /**
- * Tests throwing an exception when passing a key that is
- * not in the filter rules map and a list of paths.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleMissingKeyPathListTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("request-id", Arrays.asList("$.request-id"));
- }
-
-
- /**
- * Tests throwing an exception when passing a null oldKey.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleNullOldKeyTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule(null, "request-id", Arrays.asList("$.request-id"));
- }
-
- /**
- * Tests throwing an exception when passing an empty oldKey.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleEmptyOldKeyTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("", "request-id", Arrays.asList("$.request-id"));
- }
-
- /**
- * Tests throwing an exception when passing a null newKey.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleNullNewKeyTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("requestID", null, Arrays.asList("$.request-id"));
- }
-
- /**
- * Tests throwing an exception when passing an empty newKey.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleEmptyNewKeyTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("requestID", "", Arrays.asList("$.request-id"));
- }
-
- /**
- * Tests throwing an exception when the old and new key are the same.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleSameKeyTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("requestID", "requestID",
- Arrays.asList("$.request-id"));
- }
-
- /**
- * Tests throwing an exception when passing an empty paths list.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleEmptyPathsTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("requestID", "request-id", Arrays.asList());
- }
-
- /**
- * Tests throwing an exception when the old key doesn't exist
- * in the rules map.
- */
- @Test(expected = IllegalArgumentException.class)
- public void modifyFilterRuleNonExistingOldKeyTest() {
- String topicFilterProp = "requestID=$.requestID";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.modifyFilterRule("request-id", "id", Arrays.asList("$.request-id"));
- }
-
- /**
- * Tests deleting all filter rules in the rules map.
- */
- @Test
- public void deleteAllFilterRulesTest() {
- String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
- + "closedLoopControlName=$.closedLoopControlName";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- assertEquals(3, topicFilter.getFilterRule().size());
- topicFilter.deleteFilterRule();
- assertEquals(0, topicFilter.getFilterRule().size());
- }
-
- /**
- * Tests deleting a single filter rule by its key from the rules map.
- */
- @Test
- public void deleteFilterRuleTest() {
- String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
- + "closedLoopControlName=$.closedLoopControlName";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- assertEquals(3, topicFilter.getFilterRule().size());
- topicFilter.deleteFilterRule("closedLoopControlName");
- assertEquals(2, topicFilter.getFilterRule().size());
- }
-
- /**
- * Tests throwing an exception if the key is null.
- */
- @Test(expected = IllegalArgumentException.class)
- public void deleteFilterRuleNullKeyTest() {
- String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
- + "closedLoopControlName=$.closedLoopControlName";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.deleteFilterRule(null);
- }
-
- /**
- * Tests throwing an exception if the key is empty.
- */
- @Test(expected = IllegalArgumentException.class)
- public void deleteFilterRuleEmptyKeyTest() {
- String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
- + "closedLoopControlName=$.closedLoopControlName";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
- topicFilter.deleteFilterRule("");
- }
-
- /**
- * Tests finding all results for each filter rule corresponding to a topic.
- */
- @Test
- public void findAllTest() {
- String message = "{\"requestID\":\"38adde30-cc22-11e8-a8d5-f2801f1b9fd1\",\"entity\":\"controller\","
- + "\"controllers\":[{\"name\":\"test-controller\","
- + "\"drools\":{\"groupId\":\"org.onap.policy.drools.test\","
- + "\"artifactId\":\"test\",\"version\":\"0.0.1\"},\"operation\":\"update\"}]}";
-
- String topicFilterProp = "requestID=$.requestID,controllerName=$.controllers[0].name,"
- + "operation=$.controllers[0].operation";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- Map<String, List<String>> results = topicFilter.find(message);
- assertEquals("38adde30-cc22-11e8-a8d5-f2801f1b9fd1",
- results.get("requestID").get(0));
- assertEquals("test-controller", results.get("controllerName").get(0));
- assertEquals("update", results.get("operation").get(0));
- }
-
- @Test
- public void findAllNotFoundTest() {
- String message = "{\"requestID\":\"38adde30-cc22-11e8-a8d5-f2801f1b9fd1\",\"entity\":\"controller\","
- + "\"controllers\":[{\"name\":\"test-controller\","
- + "\"drools\":{\"groupId\":\"org.onap.policy.drools.test\","
- + "\"artifactId\":\"test\",\"version\":\"0.0.1\"},\"operation\":\"update\"}]}";
-
- String topicFilterProp = "requestID=$.requestID[3]";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- assertTrue(topicFilter.find(message).get("requestID").isEmpty());
- }
-
- /**
- * Tests finding field matches for a filter rule corresponding to a topic.
- */
- @Test
- public void findTest() {
- String message = "{\"requestID\":\"38adde30-cc22-11e8-a8d5-f2801f1b9fd1\",\"entity\":\"controller\","
- + "\"controllers\":[{\"name\":\"test-controller\","
- + "\"drools\":{\"groupId\":\"org.onap.policy.drools.test\","
- + "\"artifactId\":\"test\",\"version\":\"0.0.1\"},\"operation\":\"update\"}]}";
-
- String topicFilterProp = "requestID=$.requestID,controllerName=$.controllers[0].name,"
- + "operation=$.controllers[0].operation";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- List<String> results = topicFilter.find(message, "requestID");
- assertEquals("38adde30-cc22-11e8-a8d5-f2801f1b9fd1", results.get(0));
- }
-
- @Test
- public void findNotFoundTest() {
- String message = "{\"requestID\":\"38adde30-cc22-11e8-a8d5-f2801f1b9fd1\",\"entity\":\"controller\","
- + "\"controllers\":[{\"name\":\"test-controller\","
- + "\"drools\":{\"groupId\":\"org.onap.policy.drools.test\","
- + "\"artifactId\":\"test\",\"version\":\"0.0.1\"},\"operation\":\"update\"}]}";
-
- String topicFilterProp = "requestID=$.requestID[3]";
- MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
-
- assertTrue(topicFilter.find(message, "requestID").isEmpty());
- }
-
- @Test
- public void testFilterRuleStringString() {
- FilterRule rule = new FilterRule("hello", "world");
-
- assertEquals("hello", rule.getMdcKey());
- assertEquals("[world]", rule.getPaths().toString());
- }
-
- @Test
- public void testFilterRuleMdcKey() {
- FilterRule rule = new FilterRule("abc", "def");
-
- // check error cases first
- assertThatIllegalArgumentException().isThrownBy(() -> rule.setMdcKey(null))
- .withMessage(MdcTopicFilter.MDC_KEY_ERROR);
- assertThatIllegalArgumentException().isThrownBy(() -> rule.setMdcKey(""))
- .withMessage(MdcTopicFilter.MDC_KEY_ERROR);
-
- // success cases
- rule.setMdcKey("my-mdc-key");
- assertEquals("my-mdc-key", rule.getMdcKey());
- }
-
- @Test
- public void testFilterRulePaths() {
- FilterRule rule = new FilterRule("abc", "def");
-
- // check error cases first
- assertThatIllegalArgumentException().isThrownBy(() -> rule.setPaths(null))
- .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
- assertThatIllegalArgumentException().isThrownBy(() -> rule.setPaths(Collections.emptyList()))
- .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
-
- assertThatIllegalArgumentException().isThrownBy(() -> rule.addPaths(null))
- .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
- assertThatIllegalArgumentException().isThrownBy(() -> rule.addPaths(Collections.emptyList()))
- .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
-
- assertThatIllegalArgumentException().isThrownBy(() -> rule.addPath(null))
- .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
- assertThatIllegalArgumentException().isThrownBy(() -> rule.addPath(""))
- .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
-
- // success cases
- rule.setPaths(new ArrayList<>(Arrays.asList("pathA", "pathB")));
- assertEquals("[pathA, pathB]", rule.getPaths().toString());
-
- rule.addPath("pathC");
- assertEquals("[pathA, pathB, pathC]", rule.getPaths().toString());
-
- rule.addPaths(Arrays.asList("pathD", "pathE"));
- assertEquals("[pathA, pathB, pathC, pathD, pathE]", rule.getPaths().toString());
- }
-}
+/*
+ * ============LICENSE_START=======================================================
+ * feature-mdc-filters
+ * ================================================================================
+ * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ */
+
+package org.onap.policy.drools.mdc.filters;
+
+import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertTrue;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.List;
+import java.util.Map;
+import org.junit.Test;
+import org.onap.policy.drools.mdc.filters.MdcTopicFilter.FilterRule;
+
+public class MdcTopicFilterTest {
+
+ /**
+ * Test the simple case of having one filter rule for a key.
+ */
+ @Test
+ public void singleFilterOnePathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ FilterRule rule = topicFilter.getFilterRule("requestID");
+ assertEquals("requestID", rule.getMdcKey());
+ assertEquals("[$.requestID]", rule.getPaths().toString());
+ }
+
+ /**
+ * Tests having one filter rule with a set of potential paths to the key.
+ */
+ @Test
+ public void singleFilterMultiPathTest() {
+ String topicFilterProp = "requestID=$.requestID|$.request-id";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ FilterRule rule = topicFilter.getFilterRule("requestID");
+ assertEquals("requestID", rule.getMdcKey());
+ assertEquals(2, rule.getPaths().size());
+ assertEquals("[$.requestID, $.request-id]", rule.getPaths().toString());
+ }
+
+ /**
+ * Tests having two filter rules that each have one key/path pair.
+ */
+ @Test
+ public void multiFilterSinglePathTest() {
+ String topicFilterProp = "requestID=$.requestID,closedLoopControlName=$.closedLoopControlName";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ FilterRule rule = topicFilter.getFilterRule("requestID");
+ assertEquals("requestID", rule.getMdcKey());
+ assertEquals(1, rule.getPaths().size());
+ assertEquals("[$.requestID]", rule.getPaths().toString());
+
+ FilterRule rule2 = topicFilter.getFilterRule("closedLoopControlName");
+ assertEquals("closedLoopControlName", rule2.getMdcKey());
+ assertEquals(1, rule2.getPaths().size());
+ assertEquals("[$.closedLoopControlName]", rule2.getPaths().toString());
+ }
+
+ /**
+ * Tests having two filter rules that each have two key/path pairs.
+ */
+ @Test
+ public void multiFilterMultiPathTest() {
+ String topicFilterProp = "requestID=$.requestID|$.body.request-id,"
+ + "closedLoopControlName=$.closedLoopControlName"
+ + "|$.body.closedLoopControlName";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ FilterRule rule = topicFilter.getFilterRule("requestID");
+ assertEquals("requestID", rule.getMdcKey());
+ assertEquals(2, rule.getPaths().size());
+ assertEquals("[$.requestID, $.body.request-id]", rule.getPaths().toString());
+
+ FilterRule rule2 = topicFilter.getFilterRule("closedLoopControlName");
+ assertEquals("closedLoopControlName", rule2.getMdcKey());
+ assertEquals(2, rule2.getPaths().size());
+ assertEquals("[$.closedLoopControlName, $.body.closedLoopControlName]", rule2.getPaths().toString());
+ }
+
+ /**
+ * Tests that the regex split logic for '|' in the feature code doesn't
+ * break parsing when "||" is used as a predicate in a JsonPath query.
+ */
+ @Test
+ public void addOrPredicateFilterTest() {
+ String topicFilterProp = "requestID=$.requestID||$.body.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ assertEquals(1, topicFilter.getFilterRule().size());
+ assertEquals("requestID", topicFilter.getFilterRule("requestID").getMdcKey());
+ assertEquals(Arrays.asList("$.requestID||$.body.requestID"), topicFilter
+ .getFilterRule("requestID").getPaths());
+ }
+
+ /**
+ * Tests getting all filter rules for a given topic.
+ */
+ @Test
+ public void getAllFilterRulesTest() {
+ String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
+ + "closedLoopControlName=$.closedLoopControlName";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ assertEquals(3, topicFilter.getFilterRule().size());
+ }
+
+ /**
+ * Tests getting a filter rule by its key.
+ */
+ @Test
+ public void getFilterRuleTest() {
+ String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
+ + "closedLoopControlName=$.closedLoopControlName";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ FilterRule rule = topicFilter.getFilterRule("requestID");
+ assertNotNull(rule);
+ }
+
+ /**
+ * Tests throwing an exception for passing in a null key.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void getFilterRuleNullKeyTest() {
+ String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.getFilterRule(null);
+ }
+
+ /**
+ * Tests throwing an exception for passing in an empty key.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void getFilterRuleEmptyKeyTest() {
+ String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.getFilterRule("");
+ }
+
+ /**
+ * Tests adding a filter rule with a single path.
+ */
+ @Test
+ public void addFilterRuleSinglePathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ String key = "subRequestID";
+ String path = "$.subRequestID";
+ FilterRule rule = topicFilter.addFilterRule(key, path);
+ assertEquals(topicFilter.getFilterRule(key), rule);
+ }
+
+ /**
+ * Tests adding a filter rule with multiple paths.
+ */
+ @Test
+ public void addFilterRuleMultiPathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ String key = "subRequestID";
+ List<String> paths = Arrays.asList("$.subRequestID", "$.sub-request-id");
+ FilterRule rule = topicFilter.addFilterRule(key, paths);
+ assertEquals(topicFilter.getFilterRule(key), rule);
+ }
+
+ /**
+ * Tests throwing an exception for passing a null key and a
+ * single path.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void addFilterRuleNullKeyStringPathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.addFilterRule(null, "$.subRequestID");
+ }
+
+ /**
+ * Tests throwing an exception for passing a null key and a list
+ * of paths.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void addFilterRuleNullKeyPathListTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.addFilterRule(null, Arrays.asList("$.subRequestID"));
+ }
+
+ /**
+ * Tests throwing an exception for passing an empty key and
+ * a single path.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void addFilterRuleEmptyKeyStringPathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.addFilterRule("", "$.subRequestID");
+ }
+
+ /**
+ * Tests throwing an exception for passing an empty key and
+ * a list of paths.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void addFilterRuleEmptyKeyPathListTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.addFilterRule("", Arrays.asList("$.subRequestID"));
+ }
+
+ /**
+ * Tests throwing an exception for passing an empty path string.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void addFilterRuleEmptyPathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.addFilterRule("subRequestID", "");
+ }
+
+ /**
+ * Tests throwing an exception for passing an empty paths list.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void addFilterRuleEmptyPathsTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.addFilterRule("subRequestID", Arrays.asList());
+ }
+
+ /**
+ * Tests throwing an exception for trying to add a filter with a key that
+ * already exists with a single filter.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void addExistingFilterRuleStringTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.addFilterRule("requestID", "$.test");
+ }
+
+ /**
+ * Tests throwing an exception for trying to add a filter with a key that
+ * already exists with a list of filters.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void addExistingFilterRuleListTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.addFilterRule("requestID", Arrays.asList("$.test"));
+ }
+
+ @Test
+ public void createFilterRuleExceptionTest() {
+ assertThatIllegalArgumentException().isThrownBy(() -> new MdcTopicFilter("invalid filter"))
+ .withMessage("could not parse filter rule");
+ }
+
+ /**
+ * Tests modifying a filter rule to add a new path.
+ */
+ @Test
+ public void modifyFilterRuleSinglePathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ FilterRule rule = topicFilter.modifyFilterRule("requestID", "$.request-id");
+ assertEquals(topicFilter.getFilterRule("requestID"), rule);
+ assertEquals(Arrays.asList("$.requestID", "$.request-id"), rule.getPaths());
+ }
+
+ /**
+ * Tests modifying a filter rule to add a list of new paths.
+ */
+ @Test
+ public void modifyFilterRuleMultiPathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ FilterRule rule = topicFilter.modifyFilterRule("requestID",
+ Arrays.asList("$.request-id", "$.requestId"));
+ assertEquals(topicFilter.getFilterRule("requestID"), rule);
+ assertEquals(
+ Arrays.asList("$.requestID", "$.request-id", "$.requestId"),
+ rule.getPaths());
+ }
+
+ @Test
+ public void modifyFilterRuleMultiPathExceptionTest() {
+ MdcTopicFilter filter = new MdcTopicFilter("abc=$a.value");
+ assertThatIllegalArgumentException()
+ .isThrownBy(() -> filter.modifyFilterRule("def", "abc", Arrays.asList("$.b", "$.c")))
+ .withMessage("a filter rule already exists for key: abc");
+ }
+
+ /**
+ * Tests modifying a filter rule key.
+ */
+ @Test
+ public void modifyFilterRuleKeyTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ FilterRule rule = topicFilter.modifyFilterRule("requestID",
+ "request-id", Arrays.asList("$.request-id"));
+ assertEquals(topicFilter.getFilterRule("request-id"), rule);
+ assertEquals("[$.request-id]", rule.getPaths().toString());
+ }
+
+ /**
+ * Tests throwing an exception when passing a null key and
+ * a single path.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleNullKeyStringPathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule(null, "$.request-id");
+ }
+
+ /**
+ * Tests throwing an exception when passing a null key and
+ * a list of multiple paths.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleNullKeyPathListTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule(null, Arrays.asList("$.request-id"));
+ }
+
+ /**
+ * Tests throwing an exception when passing an empty key and
+ * a single path.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleEmptyKeyStringPathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("", "$.request-id");
+ }
+
+ /**
+ * Tests throwing an exception when passing an empty key and
+ * a list of multiple paths.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleEmptyKeyPathListTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("", Arrays.asList("$.request-id"));
+ }
+
+ /**
+ * Tests throwing an exception when passing an empty string path.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleEmptyPathStringTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("requestID", "");
+ }
+
+ /**
+ * Tests throwing an exception when passing an empty list of paths.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleEmptyPathListTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("requestID", Arrays.asList());
+ }
+
+ /**
+ * Tests throwing an exception when passing a key that is
+ * not in the filter rules map and a string path.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleMissingKeyStringPathTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("request-id", "$.request-id");
+ }
+
+ /**
+ * Tests throwing an exception when passing a key that is
+ * not in the filter rules map and a list of paths.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleMissingKeyPathListTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("request-id", Arrays.asList("$.request-id"));
+ }
+
+
+ /**
+ * Tests throwing an exception when passing a null oldKey.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleNullOldKeyTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule(null, "request-id", Arrays.asList("$.request-id"));
+ }
+
+ /**
+ * Tests throwing an exception when passing an empty oldKey.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleEmptyOldKeyTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("", "request-id", Arrays.asList("$.request-id"));
+ }
+
+ /**
+ * Tests throwing an exception when passing a null newKey.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleNullNewKeyTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("requestID", null, Arrays.asList("$.request-id"));
+ }
+
+ /**
+ * Tests throwing an exception when passing an empty newKey.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleEmptyNewKeyTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("requestID", "", Arrays.asList("$.request-id"));
+ }
+
+ /**
+ * Tests throwing an exception when the old and new key are the same.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleSameKeyTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("requestID", "requestID",
+ Arrays.asList("$.request-id"));
+ }
+
+ /**
+ * Tests throwing an exception when passing an empty paths list.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleEmptyPathsTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("requestID", "request-id", Arrays.asList());
+ }
+
+ /**
+ * Tests throwing an exception when the old key doesn't exist
+ * in the rules map.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void modifyFilterRuleNonExistingOldKeyTest() {
+ String topicFilterProp = "requestID=$.requestID";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.modifyFilterRule("request-id", "id", Arrays.asList("$.request-id"));
+ }
+
+ /**
+ * Tests deleting all filter rules in the rules map.
+ */
+ @Test
+ public void deleteAllFilterRulesTest() {
+ String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
+ + "closedLoopControlName=$.closedLoopControlName";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ assertEquals(3, topicFilter.getFilterRule().size());
+ topicFilter.deleteFilterRule();
+ assertEquals(0, topicFilter.getFilterRule().size());
+ }
+
+ /**
+ * Tests deleting a single filter rule by its key from the rules map.
+ */
+ @Test
+ public void deleteFilterRuleTest() {
+ String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
+ + "closedLoopControlName=$.closedLoopControlName";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ assertEquals(3, topicFilter.getFilterRule().size());
+ topicFilter.deleteFilterRule("closedLoopControlName");
+ assertEquals(2, topicFilter.getFilterRule().size());
+ }
+
+ /**
+ * Tests throwing an exception if the key is null.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void deleteFilterRuleNullKeyTest() {
+ String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
+ + "closedLoopControlName=$.closedLoopControlName";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.deleteFilterRule(null);
+ }
+
+ /**
+ * Tests throwing an exception if the key is empty.
+ */
+ @Test(expected = IllegalArgumentException.class)
+ public void deleteFilterRuleEmptyKeyTest() {
+ String topicFilterProp = "requestID=$.requestID,subRequestID=$.subRequestID,"
+ + "closedLoopControlName=$.closedLoopControlName";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+ topicFilter.deleteFilterRule("");
+ }
+
+ /**
+ * Tests finding all results for each filter rule corresponding to a topic.
+ */
+ @Test
+ public void findAllTest() {
+ String message = "{\"requestID\":\"38adde30-cc22-11e8-a8d5-f2801f1b9fd1\",\"entity\":\"controller\","
+ + "\"controllers\":[{\"name\":\"test-controller\","
+ + "\"drools\":{\"groupId\":\"org.onap.policy.drools.test\","
+ + "\"artifactId\":\"test\",\"version\":\"0.0.1\"},\"operation\":\"update\"}]}";
+
+ String topicFilterProp = "requestID=$.requestID,controllerName=$.controllers[0].name,"
+ + "operation=$.controllers[0].operation";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ Map<String, List<String>> results = topicFilter.find(message);
+ assertEquals("38adde30-cc22-11e8-a8d5-f2801f1b9fd1",
+ results.get("requestID").get(0));
+ assertEquals("test-controller", results.get("controllerName").get(0));
+ assertEquals("update", results.get("operation").get(0));
+ }
+
+ @Test
+ public void findAllNotFoundTest() {
+ String message = "{\"requestID\":\"38adde30-cc22-11e8-a8d5-f2801f1b9fd1\",\"entity\":\"controller\","
+ + "\"controllers\":[{\"name\":\"test-controller\","
+ + "\"drools\":{\"groupId\":\"org.onap.policy.drools.test\","
+ + "\"artifactId\":\"test\",\"version\":\"0.0.1\"},\"operation\":\"update\"}]}";
+
+ String topicFilterProp = "requestID=$.requestID[3]";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ assertTrue(topicFilter.find(message).get("requestID").isEmpty());
+ }
+
+ /**
+ * Tests finding field matches for a filter rule corresponding to a topic.
+ */
+ @Test
+ public void findTest() {
+ String message = "{\"requestID\":\"38adde30-cc22-11e8-a8d5-f2801f1b9fd1\",\"entity\":\"controller\","
+ + "\"controllers\":[{\"name\":\"test-controller\","
+ + "\"drools\":{\"groupId\":\"org.onap.policy.drools.test\","
+ + "\"artifactId\":\"test\",\"version\":\"0.0.1\"},\"operation\":\"update\"}]}";
+
+ String topicFilterProp = "requestID=$.requestID,controllerName=$.controllers[0].name,"
+ + "operation=$.controllers[0].operation";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ List<String> results = topicFilter.find(message, "requestID");
+ assertEquals("38adde30-cc22-11e8-a8d5-f2801f1b9fd1", results.get(0));
+ }
+
+ @Test
+ public void findNotFoundTest() {
+ String message = "{\"requestID\":\"38adde30-cc22-11e8-a8d5-f2801f1b9fd1\",\"entity\":\"controller\","
+ + "\"controllers\":[{\"name\":\"test-controller\","
+ + "\"drools\":{\"groupId\":\"org.onap.policy.drools.test\","
+ + "\"artifactId\":\"test\",\"version\":\"0.0.1\"},\"operation\":\"update\"}]}";
+
+ String topicFilterProp = "requestID=$.requestID[3]";
+ MdcTopicFilter topicFilter = new MdcTopicFilter(topicFilterProp);
+
+ assertTrue(topicFilter.find(message, "requestID").isEmpty());
+ }
+
+ @Test
+ public void testFilterRuleStringString() {
+ FilterRule rule = new FilterRule("hello", "world");
+
+ assertEquals("hello", rule.getMdcKey());
+ assertEquals("[world]", rule.getPaths().toString());
+ }
+
+ @Test
+ public void testFilterRuleMdcKey() {
+ FilterRule rule = new FilterRule("abc", "def");
+
+ // check error cases first
+ assertThatIllegalArgumentException().isThrownBy(() -> rule.setMdcKey(null))
+ .withMessage(MdcTopicFilter.MDC_KEY_ERROR);
+ assertThatIllegalArgumentException().isThrownBy(() -> rule.setMdcKey(""))
+ .withMessage(MdcTopicFilter.MDC_KEY_ERROR);
+
+ // success cases
+ rule.setMdcKey("my-mdc-key");
+ assertEquals("my-mdc-key", rule.getMdcKey());
+ }
+
+ @Test
+ public void testFilterRulePaths() {
+ FilterRule rule = new FilterRule("abc", "def");
+
+ // check error cases first
+ assertThatIllegalArgumentException().isThrownBy(() -> rule.setPaths(null))
+ .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
+ assertThatIllegalArgumentException().isThrownBy(() -> rule.setPaths(Collections.emptyList()))
+ .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
+
+ assertThatIllegalArgumentException().isThrownBy(() -> rule.addPaths(null))
+ .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
+ assertThatIllegalArgumentException().isThrownBy(() -> rule.addPaths(Collections.emptyList()))
+ .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
+
+ assertThatIllegalArgumentException().isThrownBy(() -> rule.addPath(null))
+ .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
+ assertThatIllegalArgumentException().isThrownBy(() -> rule.addPath(""))
+ .withMessage(MdcTopicFilter.JSON_PATH_ERROR);
+
+ // success cases
+ rule.setPaths(new ArrayList<>(Arrays.asList("pathA", "pathB")));
+ assertEquals("[pathA, pathB]", rule.getPaths().toString());
+
+ rule.addPath("pathC");
+ assertEquals("[pathA, pathB, pathC]", rule.getPaths().toString());
+
+ rule.addPaths(Arrays.asList("pathD", "pathE"));
+ assertEquals("[pathA, pathB, pathC, pathD, pathE]", rule.getPaths().toString());
+ }
+}