Add convert() to Coder

This addresses Liam's review comment about moving the "translate"
method from the actor Util class into policy-common.
Added a method to Coder to convert from one object type to
another (e.g., from a Map to a POJO, or vice versa).

Issue-ID: POLICY-2363
Signed-off-by: Jim Hahn <jrh3@att.com>
Change-Id: I2a0b5ab4ce4b0eeda216a57cbe23a8bb64f64940

2 files changed, 161 insertions, 1 deletions

diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/CoderTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/CoderTest.java
new file mode 100644
index 00000000..01821504
--- /dev/null
+++ b/utils/src/test/java/org/onap/policy/common/utils/coder/CoderTest.java
@@ -0,0 +1,129 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * ONAP
+ * ================================================================================
+ * Copyright (C) 2020 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.coder;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNull;
+
+import java.io.File;
+import java.io.InputStream;
+import java.io.OutputStream;
+import java.io.Reader;
+import java.io.Writer;
+import org.junit.Before;
+import org.junit.Test;
+
+public class CoderTest {
+    private static final Long LONG = 10L;
+    private static final Integer INTEGER = 10;
+    private static final String INT_TEXT = INTEGER.toString();
+    private static final String TEXT = "some text";
+    private static final String ENCODED = "encoded value";
+    private static final String DECODED = "decoded value";
+
+    private MyCoder coder;
+
+    @Before
+    public void setUp() {
+        coder = new MyCoder();
+    }
+
+    @Test
+    public void testConvert() throws CoderException {
+        assertNull(coder.convert(null, String.class));
+
+        // same class of object
+        assertEquals(TEXT, coder.convert(TEXT, String.class));
+        assertEquals(INTEGER, coder.convert(INTEGER, Integer.class));
+
+        // source is a string
+        assertEquals(INTEGER, coder.convert(TEXT, Integer.class));
+
+        // target is a string
+        assertEquals(INT_TEXT, coder.convert(INTEGER, String.class));
+
+        // source and target are different types, neither is a string
+        assertEquals(INTEGER, coder.convert(LONG, Integer.class));
+    }
+
+    private static class MyCoder implements Coder {
+        @Override
+        public String encode(Object object) throws CoderException {
+            return (object.getClass() == String.class ? ENCODED : INT_TEXT);
+        }
+
+        @Override
+        public String encode(Object object, boolean pretty) throws CoderException {
+            // unused
+            return null;
+        }
+
+        @Override
+        public void encode(Writer target, Object object) throws CoderException {
+            // unused
+        }
+
+        @Override
+        public void encode(OutputStream target, Object object) throws CoderException {
+            // unused
+        }
+
+        @Override
+        public void encode(File target, Object object) throws CoderException {
+            // unused
+        }
+
+        @Override
+        public <T> T decode(String json, Class<T> clazz) throws CoderException {
+            return (clazz == String.class ? clazz.cast(DECODED) : clazz.cast(INTEGER));
+        }
+
+        @Override
+        public <T> T decode(Reader source, Class<T> clazz) throws CoderException {
+            // unused
+            return null;
+        }
+
+        @Override
+        public <T> T decode(InputStream source, Class<T> clazz) throws CoderException {
+            // unused
+            return null;
+        }
+
+        @Override
+        public <T> T decode(File source, Class<T> clazz) throws CoderException {
+            // unused
+            return null;
+        }
+
+        @Override
+        public StandardCoderObject toStandard(Object object) throws CoderException {
+            // unused
+            return null;
+        }
+
+        @Override
+        public <T> T fromStandard(StandardCoderObject sco, Class<T> clazz) throws CoderException {
+            // unused
+            return null;
+        }
+    }
+}
diff --git a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderTest.java b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderTest.java
index d5cde55a..ad4382ce 100644
--- a/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderTest.java
+++ b/utils/src/test/java/org/onap/policy/common/utils/coder/StandardCoderTest.java
@@ -23,6 +23,8 @@ package org.onap.policy.common.utils.coder;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertSame;
 import static org.mockito.Mockito.doThrow;
 import static org.mockito.Mockito.spy;
 import static org.mockito.Mockito.when;
@@ -66,6 +68,35 @@ public class StandardCoderTest {
     }
 
     @Test
+    public void testConvert() throws CoderException {
+        // null source
+        assertNull(coder.convert(null, StandardCoderObject.class));
+
+        // same class of object
+        StandardCoderObject sco = new StandardCoderObject();
+        assertSame(sco, coder.convert(sco, StandardCoderObject.class));
+
+        // source is a string
+        assertEquals(Integer.valueOf(10), coder.convert("10", Integer.class));
+
+        // target is a string
+        assertEquals("10", coder.convert(10, String.class));
+
+        // source and target are different types, neither is a string
+        sco = coder.convert(Map.of("hello", "world"), StandardCoderObject.class);
+        assertEquals("world", sco.getString("hello"));
+
+        // throw an exeception
+        coder = new StandardCoder() {
+            @Override
+            protected <T> T fromJson(JsonElement json, Class<T> clazz) {
+                throw jpe;
+            }
+        };
+        assertThatThrownBy(() -> coder.convert(10, Long.class)).isInstanceOf(CoderException.class).hasCause(jpe);
+    }
+
+    @Test
     public void testEncodeObject() throws Exception {
         List<Integer> arr = Arrays.asList(1100, 1110);
         assertEquals("[1100,1110]", coder.encode(arr));
@@ -312,7 +343,7 @@ public class StandardCoderTest {
 
         // test when decoding into a map
         @SuppressWarnings("unchecked")
-        Map<String,Object> map2 = coder.decode("{'intValue':10, 'dblVal':20.1}", TreeMap.class);
+        Map<String, Object> map2 = coder.decode("{'intValue':10, 'dblVal':20.1}", TreeMap.class);
         assertEquals("{dblVal=20.1, intValue=10}", map2.toString());
     }