From 950fe7dcd47cf4ab7b09b46bbef992f1d5b7fe0d Mon Sep 17 00:00:00 2001 From: jhh Date: Thu, 10 Feb 2022 13:11:01 -0600 Subject: restore coders toolset toString This is to reduce verbosity introduced by lombok @ToString, Issue-ID: POLICY-3895 Signed-off-by: jhh Change-Id: I7b6b3490b508e6ccb094da87d6f0f64a7e859277 Signed-off-by: jhh --- .../drools/protocol/coders/EventProtocolCoder.java | 16 +++-- .../protocol/coders/ProtocolCoderToolsetTest.java | 74 ++++++++++++---------- 2 files changed, 53 insertions(+), 37 deletions(-) diff --git a/policy-management/src/main/java/org/onap/policy/drools/protocol/coders/EventProtocolCoder.java b/policy-management/src/main/java/org/onap/policy/drools/protocol/coders/EventProtocolCoder.java index 278ea41b..88b25255 100644 --- a/policy-management/src/main/java/org/onap/policy/drools/protocol/coders/EventProtocolCoder.java +++ b/policy-management/src/main/java/org/onap/policy/drools/protocol/coders/EventProtocolCoder.java @@ -2,7 +2,7 @@ * ============LICENSE_START======================================================= * ONAP * ================================================================================ - * Copyright (C) 2017-2019 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2017-2019,2022 AT&T Intellectual Property. All rights reserved. * Modifications Copyright(C) 2018 Samsung Electronics Co., Ltd. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); @@ -25,7 +25,6 @@ import java.util.List; import lombok.AllArgsConstructor; import lombok.Getter; import lombok.Setter; -import lombok.ToString; import org.onap.policy.drools.controller.DroolsController; /** @@ -35,7 +34,6 @@ public interface EventProtocolCoder { @Getter @Setter - @ToString @AllArgsConstructor public static class CoderFilters { @@ -53,13 +51,23 @@ public interface EventProtocolCoder { * classloader hash. */ protected int modelClassLoaderHash; + + @Override + public String toString() { + return "CoderFilters [factClass=" + + factClass + + ", filter=" + + filter + + ", modelClassLoaderHash=" + + modelClassLoaderHash + + "]"; + } } /** * Adds a Decoder class to decode the protocol over this topic. * * @param eventProtocolParams parameter object for event protocol - * @throw IllegalArgumentException if an invalid parameter is passed */ void addDecoder(EventProtocolParams eventProtocolParams); diff --git a/policy-management/src/test/java/org/onap/policy/drools/protocol/coders/ProtocolCoderToolsetTest.java b/policy-management/src/test/java/org/onap/policy/drools/protocol/coders/ProtocolCoderToolsetTest.java index 7920723e..6d1ec3d3 100644 --- a/policy-management/src/test/java/org/onap/policy/drools/protocol/coders/ProtocolCoderToolsetTest.java +++ b/policy-management/src/test/java/org/onap/policy/drools/protocol/coders/ProtocolCoderToolsetTest.java @@ -2,7 +2,7 @@ * ============LICENSE_START======================================================= * ONAP * ================================================================================ - * Copyright (C) 2018-2021 AT&T Intellectual Property. All rights reserved. + * Copyright (C) 2018-2021-2022 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. @@ -20,6 +20,13 @@ package org.onap.policy.drools.protocol.coders; +import static org.assertj.core.api.Assertions.assertThat; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertNull; +import static org.junit.Assert.assertSame; + import com.google.gson.Gson; import com.google.gson.GsonBuilder; import java.io.IOException; @@ -29,7 +36,6 @@ import java.util.Properties; import lombok.AllArgsConstructor; import lombok.Getter; import org.junit.After; -import org.junit.Assert; import org.junit.Before; import org.junit.BeforeClass; import org.junit.Test; @@ -55,7 +61,7 @@ public class ProtocolCoderToolsetTest { private static final String JUNIT_PROTOCOL_CODER_TOPIC = JUNIT_PROTOCOL_CODER_ARTIFACT_ID; private static final String CONTROLLER_ID = "blah"; - private static Logger logger = LoggerFactory.getLogger(ProtocolCoderToolset.class); + private static final Logger logger = LoggerFactory.getLogger(ProtocolCoderToolset.class); private static volatile ReleaseId releaseId; @@ -110,8 +116,11 @@ public class ProtocolCoderToolsetTest { .eventClass(ThreeStrings.class.getName()).protocolFilter(protocolFilter) .customGsonCoder(null).modelClassLoaderHash(12345678).build(), CONTROLLER_ID); - Assert.assertNotNull(gsonToolset.getEncoder()); - Assert.assertNotNull(gsonToolset.getDecoder()); + assertNotNull(gsonToolset.getEncoder()); + assertNotNull(gsonToolset.getDecoder()); + assertThat(gsonToolset.toString()).startsWith("GsonProtocolCoderToolset ["); + assertThat(gsonToolset.toString()).contains("=ProtocolCoderToolset ["); + assertThat(gsonToolset.toString()).contains("[CoderFilters ["); testToolset(protocolFilter, gsonToolset); @@ -146,81 +155,80 @@ public class ProtocolCoderToolsetTest { private void decode(JsonProtocolFilter protocolFilter, ProtocolCoderToolset coderToolset, ThreeStrings triple, String tripleEncoded) { - ThreeStrings tripleDecoded = null; try { - tripleDecoded = (ThreeStrings) coderToolset.decode(tripleEncoded); + coderToolset.decode(tripleEncoded); } catch (UnsupportedOperationException e) { /* OK */ logger.trace("Junit expected exception - decode does not pass filtering", e); } CoderFilters coderFilters = coderToolset.getCoder(ThreeStrings.class.getName()); - Assert.assertSame(coderFilters.getFactClass(), ThreeStrings.class.getName()); - Assert.assertSame(coderFilters.getFilter(), protocolFilter); - Assert.assertNotNull(coderFilters.getFilter().getRule()); + assertSame(coderFilters.getFactClass(), ThreeStrings.class.getName()); + assertSame(coderFilters.getFilter(), protocolFilter); + assertNotNull(coderFilters.getFilter().getRule()); coderFilters.getFilter().setRule("[?($.second =~ /^v2$/ && $.third =~ /.*v3.*/)]"); - tripleDecoded = (ThreeStrings) coderToolset.decode(tripleEncoded); + ThreeStrings tripleDecoded = (ThreeStrings) coderToolset.decode(tripleEncoded); - Assert.assertEquals(triple.getFirst(), tripleDecoded.getFirst()); - Assert.assertEquals(triple.getSecond(), tripleDecoded.getSecond()); - Assert.assertEquals(triple.getThird(), tripleDecoded.getThird()); + assertEquals(triple.getFirst(), tripleDecoded.getFirst()); + assertEquals(triple.getSecond(), tripleDecoded.getSecond()); + assertEquals(triple.getThird(), tripleDecoded.getThird()); coderFilters.getFilter().setRule(null); - Assert.assertEquals("[?($ =~ /.*/)]", coderFilters.getFilter().getRule()); + assertEquals("[?($ =~ /.*/)]", coderFilters.getFilter().getRule()); tripleDecoded = (ThreeStrings) coderToolset.decode(tripleEncoded); - Assert.assertEquals(tripleDecoded.getFirst(), triple.getFirst()); - Assert.assertEquals(tripleDecoded.getSecond(), triple.getSecond()); - Assert.assertEquals(tripleDecoded.getThird(), triple.getThird()); + assertEquals(tripleDecoded.getFirst(), triple.getFirst()); + assertEquals(tripleDecoded.getSecond(), triple.getSecond()); + assertEquals(tripleDecoded.getThird(), triple.getThird()); coderFilters.getFilter().setRule("[?($.third =~ /.*v3.*/)]"); } private String encode(ProtocolCoderToolset coderToolset, ThreeStrings triple) { String tripleEncoded = coderToolset.encode(triple); - Assert.assertTrue(!tripleEncoded.isEmpty()); + assertFalse(tripleEncoded.isEmpty()); return tripleEncoded; } private void addRemoveCoder(ProtocolCoderToolset coderToolset) { coderToolset.addCoder(this.getClass().getName(), new JsonProtocolFilter("[?($.second =~ /.*/)]"), 654321); - Assert.assertEquals(2, coderToolset.getCoders().size()); + assertEquals(2, coderToolset.getCoders().size()); coderToolset.removeCoders(this.getClass().getName()); - Assert.assertEquals(1, coderToolset.getCoders().size()); + assertEquals(1, coderToolset.getCoders().size()); } private void updateCoderFilterRule(ProtocolCoderToolset coderToolset) { coderToolset.addCoder(ThreeStrings.class.getName(), new JsonProtocolFilter("[?($.third =~ /.*/)]"), 654321); - Assert.assertEquals(1, coderToolset.getCoders().size()); + assertEquals(1, coderToolset.getCoders().size()); - Assert.assertEquals(654321, coderToolset.getCoder(ThreeStrings.class.getName()).getModelClassLoaderHash()); + assertEquals(654321, coderToolset.getCoder(ThreeStrings.class.getName()).getModelClassLoaderHash()); - Assert.assertNotNull(coderToolset.getCoder(ThreeStrings.class.getName()).getFilter().getRule()); + assertNotNull(coderToolset.getCoder(ThreeStrings.class.getName()).getFilter().getRule()); - Assert.assertEquals("[?($.third =~ /.*/)]", + assertEquals("[?($.third =~ /.*/)]", coderToolset.getCoder(ThreeStrings.class.getName()).getFilter().getRule()); } private void validateInitialization(JsonProtocolFilter protocolFilter, ProtocolCoderToolset coderToolset) { - Assert.assertEquals(CONTROLLER_ID, coderToolset.getControllerId()); - Assert.assertEquals(releaseId.getGroupId(), coderToolset.getGroupId()); - Assert.assertEquals(releaseId.getArtifactId(), coderToolset.getArtifactId()); - Assert.assertNull(coderToolset.getCustomCoder()); + assertEquals(CONTROLLER_ID, coderToolset.getControllerId()); + assertEquals(releaseId.getGroupId(), coderToolset.getGroupId()); + assertEquals(releaseId.getArtifactId(), coderToolset.getArtifactId()); + assertNull(coderToolset.getCustomCoder()); - Assert.assertEquals(1, coderToolset.getCoders().size()); + assertEquals(1, coderToolset.getCoders().size()); CoderFilters coderFilters = coderToolset.getCoder(CONTROLLER_ID); - Assert.assertNull(coderFilters); + assertNull(coderFilters); coderFilters = coderToolset.getCoder(ThreeStrings.class.getName()); - Assert.assertNotNull(coderFilters); + assertNotNull(coderFilters); - Assert.assertEquals(coderFilters.getFilter(), protocolFilter); + assertEquals(coderFilters.getFilter(), protocolFilter); } private DroolsController createController() { -- cgit 1.2.3-korg