From 98a4da643c738a4246cc4cc4aa9c9f21ae47cff8 Mon Sep 17 00:00:00 2001 From: Jim Hahn Date: Tue, 6 Aug 2019 12:17:29 -0400 Subject: Add ApiUtils to facilitate looping over features A number of places in the code (common & drools-pdp) loop over features, giving each provider a chance to act on something. Created a common method in a new ApiUtils class to faciliate it. Renamed to FeatureApiUtils per review comment. Change-Id: I0c2cf0f33854cb5b6921b394f64de498a6234909 Issue-ID: POLICY-1968 Signed-off-by: Jim Hahn --- .../features/NetLoggerFeatureProviders.java | 2 +- .../common/endpoints/utils/NetLoggerUtil.java | 34 +++------ .../common/utils/services/FeatureApiUtils.java | 61 ++++++++++++++++ .../common/utils/services/FeatureApiUtilsTest.java | 82 ++++++++++++++++++++++ 4 files changed, 155 insertions(+), 24 deletions(-) create mode 100644 utils/src/main/java/org/onap/policy/common/utils/services/FeatureApiUtils.java create mode 100644 utils/src/test/java/org/onap/policy/common/utils/services/FeatureApiUtilsTest.java diff --git a/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/features/NetLoggerFeatureProviders.java b/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/features/NetLoggerFeatureProviders.java index db2b05d1..8b09f386 100644 --- a/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/features/NetLoggerFeatureProviders.java +++ b/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/features/NetLoggerFeatureProviders.java @@ -32,7 +32,7 @@ public class NetLoggerFeatureProviders { * Feature providers implementing this interface. */ @Getter - private static OrderedServiceImpl providers = + private static final OrderedServiceImpl providers = new OrderedServiceImpl<>(NetLoggerFeatureApi.class); private NetLoggerFeatureProviders() { diff --git a/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/utils/NetLoggerUtil.java b/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/utils/NetLoggerUtil.java index d8f48a26..c7dd516f 100644 --- a/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/utils/NetLoggerUtil.java +++ b/policy-endpoints/src/main/java/org/onap/policy/common/endpoints/utils/NetLoggerUtil.java @@ -21,8 +21,8 @@ package org.onap.policy.common.endpoints.utils; import org.onap.policy.common.endpoints.event.comm.Topic.CommInfrastructure; -import org.onap.policy.common.endpoints.features.NetLoggerFeatureApi; import org.onap.policy.common.endpoints.features.NetLoggerFeatureProviders; +import org.onap.policy.common.utils.services.FeatureApiUtils; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -111,17 +111,11 @@ public class NetLoggerUtil { */ private static boolean featureBeforeLog(Logger eventLogger, EventType type, CommInfrastructure protocol, String topic, String message) { - for (NetLoggerFeatureApi feature : NetLoggerFeatureProviders.getProviders().getList()) { - try { - if (feature.beforeLog(eventLogger, type, protocol, topic, message)) { - return true; - } - } catch (Exception e) { - logger.error("feature {} before-log failure because of {}", feature.getClass().getName(), - e.getMessage(), e); - } - } - return false; + + return FeatureApiUtils.apply(NetLoggerFeatureProviders.getProviders().getList(), + feature -> feature.beforeLog(eventLogger, type, protocol, topic, message), + (feature, ex) -> logger.error("feature {} before-log failure because of {}", + feature.getClass().getName(), ex.getMessage(), ex)); } /** @@ -138,17 +132,11 @@ public class NetLoggerUtil { */ private static boolean featureAfterLog(Logger eventLogger, EventType type, CommInfrastructure protocol, String topic, String message) { - for (NetLoggerFeatureApi feature : NetLoggerFeatureProviders.getProviders().getList()) { - try { - if (feature.afterLog(eventLogger, type, protocol, topic, message)) { - return true; - } - } catch (Exception e) { - logger.error("feature {} after-log failure because of {}", feature.getClass().getName(), e.getMessage(), - e); - } - } - return false; + + return FeatureApiUtils.apply(NetLoggerFeatureProviders.getProviders().getList(), + feature -> feature.afterLog(eventLogger, type, protocol, topic, message), + (feature, ex) -> logger.error("feature {} after-log failure because of {}", + feature.getClass().getName(), ex.getMessage(), ex)); } } diff --git a/utils/src/main/java/org/onap/policy/common/utils/services/FeatureApiUtils.java b/utils/src/main/java/org/onap/policy/common/utils/services/FeatureApiUtils.java new file mode 100644 index 00000000..e88361ea --- /dev/null +++ b/utils/src/main/java/org/onap/policy/common/utils/services/FeatureApiUtils.java @@ -0,0 +1,61 @@ +/* + * ============LICENSE_START======================================================= + * ONAP + * ================================================================================ + * 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.common.utils.services; + +import java.util.List; +import java.util.function.BiConsumer; +import java.util.function.Predicate; + +/** + * Utilities for use with "feature APIs". + */ +public class FeatureApiUtils { + + private FeatureApiUtils() { + // do nothing + } + + /** + * Applies a function on each feature provider, stopping as soon as one returns true. + * + * @param providers list of feature providers + * @param predicate function to be applied to each provider + * @param handleEx function to handle any exception generated by the predicate + * (typically, by logging the message) + * @return {@code true} if one of the providers returned {@code true}, {@code false} + * otherwise + */ + public static boolean apply(List providers, Predicate predicate, + BiConsumer handleEx) { + + for (T feature : providers) { + try { + if (predicate.test(feature)) { + return true; + } + } catch (RuntimeException e) { + handleEx.accept(feature, e); + } + } + + return false; + } +} diff --git a/utils/src/test/java/org/onap/policy/common/utils/services/FeatureApiUtilsTest.java b/utils/src/test/java/org/onap/policy/common/utils/services/FeatureApiUtilsTest.java new file mode 100644 index 00000000..232d3409 --- /dev/null +++ b/utils/src/test/java/org/onap/policy/common/utils/services/FeatureApiUtilsTest.java @@ -0,0 +1,82 @@ +/*- + * ============LICENSE_START======================================================= + * ONAP + * ================================================================================ + * 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.common.utils.services; + +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertTrue; + +import java.util.Arrays; +import java.util.LinkedList; +import java.util.List; +import java.util.function.Predicate; +import org.junit.Before; +import org.junit.Test; + +public class FeatureApiUtilsTest { + private static final String HANDLED = "handled"; + + private MyPred pred; + private List tried; + private List errors; + + /** + * Initializes fields. + */ + @Before + public void setUp() { + tried = new LinkedList<>(); + errors = new LinkedList<>(); + pred = new MyPred(); + } + + @Test + public void testApplyFeatureTrue() { + assertTrue(FeatureApiUtils.apply(Arrays.asList("exceptT0", "falseT1", HANDLED, "falseT2", HANDLED), pred, + (str, ex) -> errors.add(str))); + + assertEquals("[exceptT0, falseT1, handled]", tried.toString()); + assertEquals("[exceptT0]", errors.toString()); + } + + @Test + public void testApplyFeatureFalse() { + List lst = Arrays.asList("falseF1", "exceptF2", "falseF3"); + + assertFalse(FeatureApiUtils.apply(lst, pred, (str,ex) -> errors.add(str))); + assertEquals(lst.toString(), tried.toString()); + assertEquals("[exceptF2]", errors.toString()); + } + + private class MyPred implements Predicate { + + @Override + public boolean test(String data) { + tried.add(data); + + if (data.startsWith("except")) { + throw new IllegalArgumentException("expected exception"); + } + + return data.equals(HANDLED); + } + } +} -- cgit 1.2.3-korg