Improved test coverage.

Reorganized some of the tests due to comments

Issue-ID: SDC-2238

Change-Id: I6b3bcc674c792f3c5c06e851a2cfe2d92eead07c
Signed-off-by: James Guistwite <jguistwite@iconectiv.com>

4 files changed, 84 insertions, 4 deletions

diff --git a/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ConfigurationTest.java b/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ConfigurationTest.java
new file mode 100644
index 0000000000..f637632835
--- /dev/null
+++ b/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ConfigurationTest.java
@@ -0,0 +1,66 @@
+/*
+ * Copyright © 2019 iconectiv
+ *
+ * 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.
+ */
+
+package org.openecomp.core.externaltesting.api;
+
+import com.fasterxml.jackson.databind.ObjectMapper;
+import org.junit.Assert;
+import org.junit.Test;
+import org.onap.sdc.tosca.services.YamlUtil;
+
+import java.io.File;
+import java.io.FileInputStream;
+import java.io.InputStream;
+
+public class ConfigurationTest {
+
+  @Test
+  public void testClientConfig() {
+    // a brain dead test of the setter and getter.
+    // future tests for more complex config to come.
+    ClientConfiguration cc = new ClientConfiguration();
+    cc.setEnabled(true);
+    Assert.assertTrue("client configuration setter", cc.isEnabled());
+    cc.setEnabled(false);
+    Assert.assertFalse("client configuration setter", cc.isEnabled());
+  }
+
+  @Test
+  public void testEndpointDefinition() {
+    RemoteTestingEndpointDefinition def = new RemoteTestingEndpointDefinition();
+    def.setId("vtp");
+    def.setEnabled(true);
+    def.setTitle("VTP");
+    def.setApiKey("FOOBARBAZ");
+    def.setUrl("http://example.com/vtptesting");
+    def.setScenarioFilter("c.*");
+    def.setConfig("vtp,VTP,true,http://example.com/vtptesting,c.*,FOO");
+
+    Assert.assertEquals("code", "VTP", def.getTitle());
+    Assert.assertEquals("API keys equals", "FOOBARBAZ", def.getApiKey());
+    Assert.assertEquals("code equals", "VTP", def.getTitle());
+    Assert.assertEquals("url equals", "http://example.com/vtptesting", def.getUrl());
+    Assert.assertEquals("filter equals", "c.*", def.getScenarioFilter());
+    Assert.assertTrue("enabled", def.isEnabled());
+    Assert.assertEquals("id equals", "vtp", def.getId());
+    Assert.assertNotNull("config has val", def.getConfig());
+
+
+    boolean matches = def.getScenarioFilterPattern().matcher("certification").matches();
+    Assert.assertTrue("pattern", matches);
+
+  }
+}
diff --git a/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ErrorBodyTests.java b/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ErrorBodyTest.java
index 216ff3599f..788e5d6cf3 100644
--- a/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ErrorBodyTests.java
+++ b/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ErrorBodyTest.java
@@ -19,7 +19,7 @@ package org.openecomp.core.externaltesting.api;
 import org.junit.Assert;
 import org.junit.Test;
 
-public class ErrorBodyTests {
+public class ErrorBodyTest {
 
   @Test
   public void testErrorBody() {
diff --git a/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ExecutionRequestTests.java b/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ExecutionRequestTest.java
index bfa07ecc0c..541f636841 100644
--- a/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ExecutionRequestTests.java
+++ b/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ExecutionRequestTest.java
@@ -25,7 +25,7 @@ import java.io.IOException;
 import java.util.Map;
 import java.util.UUID;
 
-public class ExecutionRequestTests {
+public class ExecutionRequestTest {
 
   @Test
   public void testTestCase() throws Exception {
@@ -89,6 +89,19 @@ public class ExecutionRequestTests {
     Assert.assertEquals("COMPLETED", rsp.getStatus());
     Assert.assertNotNull(rsp.getStartTime());
     Assert.assertNotNull(rsp.getEndTime());
+
+    rsp = mapper.readValue(new File("src/test/data/failedexecution.json"), VtpTestExecutionResponse.class);
+    Assert.assertEquals("F-1131", rsp.getCode());
+    Assert.assertEquals("Failure reason", rsp.getMessage());
+    Assert.assertEquals(500, rsp.getHttpStatus().intValue());
+  }
+
+  @Test
+  public void testTreeConstructor() {
+    // test constructor.
+    TestTreeNode tree = new TestTreeNode("root", "Root");
+    Assert.assertEquals("root", tree.getName());
+    Assert.assertEquals("Root", tree.getDescription());
   }
 
   @Test
diff --git a/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ExternalTestingApiTests.java b/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ExternalTestingApiTests.java
index 62b129c7c0..67cf71702d 100644
--- a/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ExternalTestingApiTests.java
+++ b/openecomp-be/lib/openecomp-sdc-externaltesting-lib/openecomp-sdc-externaltesting-api/src/test/java/org/openecomp/core/externaltesting/api/ExternalTestingApiTests.java
@@ -21,8 +21,9 @@ import org.junit.runners.Suite;
 
 @RunWith(Suite.class)
 @Suite.SuiteClasses({
-    ExecutionRequestTests.class,
-    ErrorBodyTests.class
+    ConfigurationTest.class,
+    ExecutionRequestTest.class,
+    ErrorBodyTest.class
 })
 public class ExternalTestingApiTests {
   // nothing to do - just a placeholder.