From 9f0385738187dc18fa479b8498d63e382723f5ea Mon Sep 17 00:00:00 2001 From: Ioannis Sotiropoulos Date: Thu, 26 Jul 2018 10:01:12 +0100 Subject: Increase code coverage Add tests to increase code coverage above 50 percent. Change-Id: Ie4f7517d7984b301e2bea7a2262307032c820700 Issue-ID: AAI-1425 Signed-off-by: Ioannis Sotiropoulos --- .../onap/aai/spike/event/OffsetManagerTest.java | 90 -------------------- .../spike/event/incoming/GizmoGraphEventTest.java | 98 ++++++++++++++++++++++ .../spike/event/incoming/OffsetManagerTest.java | 90 ++++++++++++++++++++ .../spike/event/outgoing/SpikeGraphEventTest.java | 79 +++++++++++++++++ .../spike/schema/GraphEventTransformerTest.java | 67 ++++++++++++++- 5 files changed, 331 insertions(+), 93 deletions(-) delete mode 100644 src/test/java/org/onap/aai/spike/event/OffsetManagerTest.java create mode 100644 src/test/java/org/onap/aai/spike/event/incoming/GizmoGraphEventTest.java create mode 100644 src/test/java/org/onap/aai/spike/event/incoming/OffsetManagerTest.java create mode 100644 src/test/java/org/onap/aai/spike/event/outgoing/SpikeGraphEventTest.java (limited to 'src/test/java') diff --git a/src/test/java/org/onap/aai/spike/event/OffsetManagerTest.java b/src/test/java/org/onap/aai/spike/event/OffsetManagerTest.java deleted file mode 100644 index 7975d6f..0000000 --- a/src/test/java/org/onap/aai/spike/event/OffsetManagerTest.java +++ /dev/null @@ -1,90 +0,0 @@ -/** - * ============LICENSE_START======================================================= - * org.onap.aai - * ================================================================================ - * Copyright © 2017-2018 AT&T Intellectual Property. All rights reserved. - * Copyright © 2017-2018 Amdocs - * ================================================================================ - * 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.aai.spike.event; - -import static org.junit.Assert.assertTrue; -import static org.junit.Assert.fail; - -import org.junit.Test; -import org.onap.aai.spike.event.incoming.OffsetManager; - -public class OffsetManagerTest { - - /** - * This test validates that as events are cached, and flagged as published, that the 'safe' offset - * is advanced correctly. - */ - @Test - public void testOffsetAdvancement() throws Exception { - - final Long offsetPeriod = 50L; // ms - - // Create an instance of the offset manager. - OffsetManager offsetManager = new OffsetManager(10, offsetPeriod); - - // Now, cache some events as if we had consumed them and they - // are in flight. - final int event1Index = offsetManager.cacheEvent("1", 1); - final int event2Index = offsetManager.cacheEvent("2", 2); - final int event3Index = offsetManager.cacheEvent("3", 3); - final int event4Index = offsetManager.cacheEvent("4", 4); - - // Mark some of them as 'published' - offsetManager.markAsPublished(event1Index); - offsetManager.markAsPublished(event2Index); - offsetManager.markAsPublished(event4Index); - - // Validate that the offset manager reported the expected offset (ie: we can only commit up - // to event2, event though event4 has been processed, since event3 is still in flight). - Long nextOffset = waitForOffsetUpdate(null, offsetPeriod, offsetManager); - assertTrue("Unexpected 'next offset' value. Expected=2, actual=" + nextOffset, nextOffset == 2); - - // Now, let's mark event3 as 'published'. We should see the next safe offset - // advance to event4 (since is was previously flagged as 'published'). - offsetManager.markAsPublished(event3Index); - - nextOffset = waitForOffsetUpdate(nextOffset, offsetPeriod, offsetManager); - assertTrue("Unexpected 'next offset' value. Expected=4, actual=" + nextOffset, nextOffset == 4); - } - - private Long waitForOffsetUpdate(Long currentOffset, Long offsetPeriod, OffsetManager offsetManager) - throws InterruptedException { - - Long newOffset = currentOffset; - int retries = 3; - while (currentOffset == newOffset) { - - // Wait long enough for the offset manager to have hopefully kicked it's offset - // update task. - Thread.sleep(offsetPeriod); - newOffset = offsetManager.getNextOffsetToCommit(); - - // We might have just missed the update due to timing, so we will retry a - // few times before giving up... - retries--; - if (retries == 0) { - fail("Safe offset was not updated as expected"); - } - } - return newOffset; - } - -} diff --git a/src/test/java/org/onap/aai/spike/event/incoming/GizmoGraphEventTest.java b/src/test/java/org/onap/aai/spike/event/incoming/GizmoGraphEventTest.java new file mode 100644 index 0000000..6c35ffe --- /dev/null +++ b/src/test/java/org/onap/aai/spike/event/incoming/GizmoGraphEventTest.java @@ -0,0 +1,98 @@ +/** + * ============LICENSE_START======================================================= + * org.onap.aai + * ================================================================================ + * Copyright © 2017-2018 AT&T Intellectual Property. All rights reserved. + * Copyright © 2017-2018 Amdocs + * ================================================================================ + * 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.aai.spike.event.incoming; + +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNull; +import java.io.IOException; +import java.net.URISyntaxException; +import org.junit.Test; +import org.onap.aai.spike.event.envelope.EventEnvelopeParser; +import org.onap.aai.spike.event.outgoing.SpikeGraphEvent; +import org.onap.aai.spike.exception.SpikeException; +import org.onap.aai.spike.test.util.TestFileReader; + +public class GizmoGraphEventTest { + + @Test + public void TestToSpikeGraphEvent() throws SpikeException, IOException, URISyntaxException { + String champNotification = + TestFileReader.getFileAsString("event/champ-update-notification-raw-with-relationship.json"); + + GizmoGraphEvent gizmoGraphEvent = new EventEnvelopeParser().parseEvent(champNotification); + SpikeGraphEvent spikeGraphEvent = gizmoGraphEvent.toSpikeGraphEvent(); + + assertEquals("b9c7d24a-64a5-4b89-a10a-a89ce58b1caa", spikeGraphEvent.getRelationship().getId()); + assertEquals("537494bd-1e8a-4198-9712-8cefa0f80457", spikeGraphEvent.getRelationship().getSource().getId()); + assertEquals("981c0494-c742-4d75-851c-8194bbbd8a96", spikeGraphEvent.getRelationship().getTarget().getId()); + } + + @Test + public void TestGetObjectKey() { + String objectKey; + GizmoGraphEvent gizmoGraphEvent; + + gizmoGraphEvent = new GizmoGraphEvent(); + GizmoVertex vertex = new GizmoVertex(); + vertex.setId("addfff68"); + gizmoGraphEvent.setVertex(vertex); + objectKey = gizmoGraphEvent.getObjectKey(); + assertEquals("addfff68", objectKey); + + gizmoGraphEvent = new GizmoGraphEvent(); + GizmoEdge relationship = new GizmoEdge(); + relationship.setId("909d"); + + gizmoGraphEvent.setRelationship(relationship); + objectKey = gizmoGraphEvent.getObjectKey(); + assertEquals("909d", objectKey); + + gizmoGraphEvent = new GizmoGraphEvent(); + objectKey = gizmoGraphEvent.getObjectKey(); + assertNull(objectKey); + } + + @Test + public void TestGetObjectType() { + String objectType; + String type = "pserver"; + GizmoGraphEvent gizmoGraphEvent; + + gizmoGraphEvent = new GizmoGraphEvent(); + GizmoVertex vertex = new GizmoVertex(); + vertex.setType(type); + gizmoGraphEvent.setVertex(vertex); + objectType = gizmoGraphEvent.getObjectType(); + assertEquals("Vertex->" + type, objectType); + + gizmoGraphEvent = new GizmoGraphEvent(); + GizmoEdge relationship = new GizmoEdge(); + relationship.setType(type); + + gizmoGraphEvent.setRelationship(relationship); + objectType = gizmoGraphEvent.getObjectType(); + assertEquals("Relationship->" + type, objectType); + + gizmoGraphEvent = new GizmoGraphEvent(); + objectType = gizmoGraphEvent.getObjectType(); + assertNull(objectType); + } +} diff --git a/src/test/java/org/onap/aai/spike/event/incoming/OffsetManagerTest.java b/src/test/java/org/onap/aai/spike/event/incoming/OffsetManagerTest.java new file mode 100644 index 0000000..e2a622b --- /dev/null +++ b/src/test/java/org/onap/aai/spike/event/incoming/OffsetManagerTest.java @@ -0,0 +1,90 @@ +/** + * ============LICENSE_START======================================================= + * org.onap.aai + * ================================================================================ + * Copyright © 2017-2018 AT&T Intellectual Property. All rights reserved. + * Copyright © 2017-2018 Amdocs + * ================================================================================ + * 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.aai.spike.event.incoming; + +import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; + +import org.junit.Test; +import org.onap.aai.spike.event.incoming.OffsetManager; + +public class OffsetManagerTest { + + /** + * This test validates that as events are cached, and flagged as published, that the 'safe' offset + * is advanced correctly. + */ + @Test + public void testOffsetAdvancement() throws Exception { + + final Long offsetPeriod = 50L; // ms + + // Create an instance of the offset manager. + OffsetManager offsetManager = new OffsetManager(10, offsetPeriod); + + // Now, cache some events as if we had consumed them and they + // are in flight. + final int event1Index = offsetManager.cacheEvent("1", 1); + final int event2Index = offsetManager.cacheEvent("2", 2); + final int event3Index = offsetManager.cacheEvent("3", 3); + final int event4Index = offsetManager.cacheEvent("4", 4); + + // Mark some of them as 'published' + offsetManager.markAsPublished(event1Index); + offsetManager.markAsPublished(event2Index); + offsetManager.markAsPublished(event4Index); + + // Validate that the offset manager reported the expected offset (ie: we can only commit up + // to event2, event though event4 has been processed, since event3 is still in flight). + Long nextOffset = waitForOffsetUpdate(null, offsetPeriod, offsetManager); + assertTrue("Unexpected 'next offset' value. Expected=2, actual=" + nextOffset, nextOffset == 2); + + // Now, let's mark event3 as 'published'. We should see the next safe offset + // advance to event4 (since is was previously flagged as 'published'). + offsetManager.markAsPublished(event3Index); + + nextOffset = waitForOffsetUpdate(nextOffset, offsetPeriod, offsetManager); + assertTrue("Unexpected 'next offset' value. Expected=4, actual=" + nextOffset, nextOffset == 4); + } + + private Long waitForOffsetUpdate(Long currentOffset, Long offsetPeriod, OffsetManager offsetManager) + throws InterruptedException { + + Long newOffset = currentOffset; + int retries = 3; + while (currentOffset == newOffset) { + + // Wait long enough for the offset manager to have hopefully kicked it's offset + // update task. + Thread.sleep(offsetPeriod); + newOffset = offsetManager.getNextOffsetToCommit(); + + // We might have just missed the update due to timing, so we will retry a + // few times before giving up... + retries--; + if (retries == 0) { + fail("Safe offset was not updated as expected"); + } + } + return newOffset; + } + +} diff --git a/src/test/java/org/onap/aai/spike/event/outgoing/SpikeGraphEventTest.java b/src/test/java/org/onap/aai/spike/event/outgoing/SpikeGraphEventTest.java new file mode 100644 index 0000000..b64e487 --- /dev/null +++ b/src/test/java/org/onap/aai/spike/event/outgoing/SpikeGraphEventTest.java @@ -0,0 +1,79 @@ +/** + * ============LICENSE_START======================================================= + * org.onap.aai + * ================================================================================ + * Copyright © 2017-2018 AT&T Intellectual Property. All rights reserved. + * Copyright © 2017-2018 Amdocs + * ================================================================================ + * 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.aai.spike.event.outgoing; + +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNull; +import org.junit.Test; + +public class SpikeGraphEventTest { + @Test + public void testGetObjectKey() { + String objectKey; + + SpikeGraphEvent spikeGraphEvent; + + spikeGraphEvent = new SpikeGraphEvent(); + SpikeVertex vertex = new SpikeVertex(); + vertex.setId("addfff68"); + spikeGraphEvent.setVertex(vertex); + objectKey = spikeGraphEvent.getObjectKey(); + assertEquals("addfff68", objectKey); + + spikeGraphEvent = new SpikeGraphEvent(); + SpikeEdge relationship = new SpikeEdge(); + relationship.setId("909d"); + + spikeGraphEvent.setRelationship(relationship); + objectKey = spikeGraphEvent.getObjectKey(); + assertEquals("909d", objectKey); + + spikeGraphEvent = new SpikeGraphEvent(); + objectKey = spikeGraphEvent.getObjectKey(); + assertNull(objectKey); + } + + @Test + public void testGetObjectType() { + String objectType; + String type = "pserver"; + SpikeGraphEvent spikeGraphEvent; + + spikeGraphEvent = new SpikeGraphEvent(); + SpikeVertex vertex = new SpikeVertex(); + vertex.setType(type); + spikeGraphEvent.setVertex(vertex); + objectType = spikeGraphEvent.getObjectType(); + assertEquals("Vertex->" + type, objectType); + + spikeGraphEvent = new SpikeGraphEvent(); + SpikeEdge relationship = new SpikeEdge(); + relationship.setType(type); + + spikeGraphEvent.setRelationship(relationship); + objectType = spikeGraphEvent.getObjectType(); + assertEquals("Relationship->" + type, objectType); + + spikeGraphEvent = new SpikeGraphEvent(); + objectType = spikeGraphEvent.getObjectType(); + assertNull(objectType); + } +} diff --git a/src/test/java/org/onap/aai/spike/schema/GraphEventTransformerTest.java b/src/test/java/org/onap/aai/spike/schema/GraphEventTransformerTest.java index f58bbfc..1e14e5e 100644 --- a/src/test/java/org/onap/aai/spike/schema/GraphEventTransformerTest.java +++ b/src/test/java/org/onap/aai/spike/schema/GraphEventTransformerTest.java @@ -20,18 +20,20 @@ */ package org.onap.aai.spike.schema; +import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; import static org.junit.Assert.fail; - import java.io.BufferedReader; import java.io.File; import java.io.FileReader; import java.io.IOException; -import javax.xml.bind.JAXBException; +import java.net.URISyntaxException; import org.junit.Before; import org.junit.Test; +import org.onap.aai.spike.event.envelope.EventEnvelopeParser; import org.onap.aai.spike.event.incoming.GizmoGraphEvent; import org.onap.aai.spike.exception.SpikeException; +import org.onap.aai.spike.test.util.TestFileReader; /** @@ -39,7 +41,6 @@ import org.onap.aai.spike.exception.SpikeException; * compliant representations of generic graph entities. */ public class GraphEventTransformerTest { - /** * Performs all setup steps expected to be performed prior to each test. */ @@ -118,4 +119,64 @@ public class GraphEventTransformerTest { } } } + + @Test + public void TestValidateFieldType() throws SpikeException { + String value = null; + Class clazz; + Object obj; + + clazz = Integer.class; + value = "5"; + obj = GraphEventTransformer.validateFieldType(value, clazz); + assertEquals(new Integer(5), obj); + + clazz = Long.class; + value = "5"; + obj = GraphEventTransformer.validateFieldType(value, clazz); + assertEquals(new Long(5), obj); + + clazz = Float.class; + value = "5.5"; + obj = GraphEventTransformer.validateFieldType(value, clazz); + assertEquals(new Float(5.5), obj); + + clazz = Double.class; + value = "5"; + obj = GraphEventTransformer.validateFieldType(value, clazz); + assertEquals(new Double(5), obj); + + clazz = Boolean.class; + value = "true"; + obj = GraphEventTransformer.validateFieldType(value, clazz); + assertEquals(new Boolean(true), obj); + + value = "NotValidBoolean"; + try { + GraphEventTransformer.validateFieldType(value, clazz); + } catch (SpikeException e) { + assertEquals("Invalid property value: " + value, e.getMessage()); + } + + clazz = String.class; + value = "11"; + try { + GraphEventTransformer.validateFieldType(value, clazz); + } catch (SpikeException e) { + assertEquals("Invalid property value: " + value, e.getMessage()); + } + } + + @Test + public void TestPopulateUUID() throws IOException, URISyntaxException, SpikeException { + String champNotification = + TestFileReader.getFileAsString("event/champ-update-notification-raw-with-relationship.json"); + + GizmoGraphEvent gizmoGraphEvent = new EventEnvelopeParser().parseEvent(champNotification); + GraphEventTransformer.populateUUID(gizmoGraphEvent); + + assertEquals("b9c7d24a-64a5-4b89-a10a-a89ce58b1caa", gizmoGraphEvent.getRelationship().getId()); + assertEquals("537494bd-1e8a-4198-9712-8cefa0f80457", gizmoGraphEvent.getRelationship().getSource().getId()); + assertEquals("981c0494-c742-4d75-851c-8194bbbd8a96", gizmoGraphEvent.getRelationship().getTarget().getId()); + } } -- cgit 1.2.3-korg