Simplify & remove duplicated testcases 63/50663/2
authorPawelSzalapski <pawel.szalapski@nokia.com>
Wed, 6 Jun 2018 12:05:57 +0000 (14:05 +0200)
committerPawelSzalapski <pawel.szalapski@nokia.com>
Wed, 6 Jun 2018 12:09:23 +0000 (14:09 +0200)
Testcases in those files were overengineered, plus there were
duplicates so they got simplified.
Obsolete classes that were unneeded were removed

Change-Id: I9fce325aed4f1a3a6073f0c564adac1ec5a6beae
Issue-ID: DCAEGEN2-524
Signed-off-by: PawelSzalapski <pawel.szalapski@nokia.com>
src/main/java/org/onap/dcae/commonFunction/CommonStartup.java
src/test/java/org/onap/dcae/vestest/InputJsonValidation.java [deleted file]
src/test/java/org/onap/dcae/vestest/TestDefaultConfiguration.java [moved from src/test/java/org/onap/dcae/vestest/VesCollectorJunitTestRunner.java with 50% similarity]
src/test/java/org/onap/dcae/vestest/TestJsonSchemaValidation.java [new file with mode: 0644]
src/test/java/org/onap/dcae/vestest/TestSchemaValidation.java [deleted file]
src/test/java/org/onap/dcae/vestest/VesCollectorJunitTest.java [deleted file]

index 83c8172..f16cdbb 100644 (file)
@@ -140,7 +140,7 @@ public class CommonStartup extends NsaBaseEndpoint implements Runnable {
                final String keyAlias = settings.getString(KSETTING_KEYALIAS, KDEFAULT_KEYALIAS);\r
 \r
                if (securePort > 0) {\r
-                       final String KSETTING_KEYSTOREPASS = readFile(keystorePasswordFile, Charset.defaultCharset());\r
+                       final String KSETTING_KEYSTOREPASS = readFile(keystorePasswordFile);\r
                        connectors.add(new ApiServerConnector.Builder(securePort).secure(true)\r
                                        .keystorePassword(KSETTING_KEYSTOREPASS).keystoreFile(keystoreFile).keyAlias(keyAlias).build());\r
 \r
@@ -265,7 +265,7 @@ public class CommonStartup extends NsaBaseEndpoint implements Runnable {
                }\r
        }\r
 \r
-       static String readFile(String path, Charset encoding) throws IOException {\r
+       static String readFile(String path) throws IOException {\r
                byte[] encoded = Files.readAllBytes(Paths.get(path));\r
                String pwd = new String(encoded);\r
                return pwd.substring(0, pwd.length() - 1);\r
diff --git a/src/test/java/org/onap/dcae/vestest/InputJsonValidation.java b/src/test/java/org/onap/dcae/vestest/InputJsonValidation.java
deleted file mode 100644 (file)
index 72ebdea..0000000
+++ /dev/null
@@ -1,151 +0,0 @@
-/*-
- * ============LICENSE_START=======================================================
- * PROJECT
- * ================================================================================
- * Copyright (C) 2017 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.dcae.vestest;
-
-import com.google.gson.JsonIOException;
-import com.google.gson.JsonParser;
-import com.google.gson.JsonSyntaxException;
-
-import java.io.BufferedReader;
-import java.io.FileNotFoundException;
-import java.io.FileReader;
-import java.io.IOException;
-import java.io.InputStreamReader;
-
-import org.json.simple.JSONObject;
-import org.json.simple.parser.JSONParser;
-import org.junit.Test;
-import org.onap.dcae.commonFunction.CommonStartup;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
-public class InputJsonValidation {
-
-    private static final Logger log = LoggerFactory.getLogger(InputJsonValidation.class);
-    static String valresult;
-
-
-    @Test
-    public void nonValidJsonValidation() {
-
-        JSONObject jsonObject;
-        JSONParser parser = new JSONParser();
-        Object obj = null;
-        //String jsonfilepath="C:/Users/vv770d/git/restfulcollector/src/test/resources/fujistu_non_valid_json.txt";
-        String jsonfilepath = "src/test/resources/VES_invalid.txt";
-        String retValue = "false";
-        try {
-
-            obj = parser.parse(new FileReader(jsonfilepath));
-        } catch (Exception e) {
-
-            log.info("Exception while opening the file");
-        }
-        jsonObject = (JSONObject) obj;
-
-        String schema = null;
-        try {
-            schema = new JsonParser().parse(new FileReader("etc/CommonEventFormat_27.2.json"))
-                .toString();
-            //log.info("Schema value: " + schema.toString());
-        } catch (JsonIOException | JsonSyntaxException | FileNotFoundException e) {
-            // TODO Auto-generated catch block
-            log.error(e.getLocalizedMessage(), e);
-        }
-
-        if (schema != null) {
-            retValue = CommonStartup.schemavalidate(jsonObject.toString(), schema);
-        }
-        //return retValue;
-        VesCollectorJunitTest.output = retValue;
-    }
-
-
-    // The below test case meant for verifying json schema on provided json file
-    @Test
-    public void validJsonValidation() {
-
-        JSONObject jsonObject;
-        JSONParser parser = new JSONParser();
-        Object obj = null;
-
-        String jsonfilepath = "src/test/resources/VES_valid.txt";
-        String retValue = "false";
-        try {
-
-            obj = parser.parse(new FileReader(jsonfilepath));
-        } catch (Exception e) {
-            log.info("Exception while opening the file");
-        }
-        jsonObject = (JSONObject) obj;
-        String schema = null;
-        try {
-
-            log.info("XX debug" + VesCollectorJunitTest.schemaFile);
-            schema = new JsonParser().parse(new FileReader("etc/CommonEventFormat_27.2.json"))
-                .toString();
-        } catch (JsonIOException | JsonSyntaxException | FileNotFoundException e) {
-            // TODO Auto-generated catch block
-            log.error(e.getLocalizedMessage(), e);
-        }
-
-        if (schema != null) {
-            retValue = CommonStartup.schemavalidate(jsonObject.toString(), schema);
-        }
-        VesCollectorJunitTest.output = retValue;
-        //return retValue;
-    }
-
-
-    //validating valid json reception and its posting to DMAP.
-    @Test
-    public void eventReception() {
-
-        String testCurlCommand = "curl -i -X POST -d @C:/Users/vv770d/git/restfulcollector/src/test/resources/fujistu-3.txt --header \"Content-Type: application/json\" http://localhost:8080/eventListener/v1";
-
-        //final Process terminal = curlCommand.start();
-        try {
-            Process process = Runtime.getRuntime().exec(testCurlCommand);
-            BufferedReader stdInput = new BufferedReader(new
-                InputStreamReader(process.getInputStream()));
-
-            BufferedReader stdError = new BufferedReader(new
-                InputStreamReader(process.getErrorStream()));
-
-            // read the output from the command
-
-            String str;
-            while ((str = stdInput.readLine()) != null) {
-                if (str.contains("HTTP/1.1 200 OK")) {
-
-                    //return "true";
-                    VesCollectorJunitTest.output = "true";
-                }
-            }
-        } catch (IOException e) {
-            // TODO Auto-generated catch block
-            log.error(e.getLocalizedMessage(), e);
-        }
-
-        //return "false";
-    }
-}
-
@@ -7,9 +7,9 @@
  * 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.
  */
 
 /*
- * Purpose: CommonCollectorJunitTestRunner is the main class where test suit execution starts its
- * test cases execution the common collector test suit has been written in order to incorporate
- * functional and logical testing of collector features.
+ *
+ * Purpose: CommonCollectorJunitTest is the wrapper class to invoke all prescribed Junit test cases.
+ *
  */
 
 package org.onap.dcae.vestest;
 
-import org.junit.runner.JUnitCore;
-import org.junit.runner.Result;
-import org.junit.runner.notification.Failure;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
-public class VesCollectorJunitTestRunner {
-
-    private static final Logger log = LoggerFactory.getLogger(VesCollectorJunitTestRunner.class);
-
-    /**
-     * Runner for test case.
-     *
-     * @param args command line arguments
-     */
-    public static void main(String[] args) {
-
-        log.info("STARTING TEST SUITE EXECUTION.....");
-
-        Result result = JUnitCore.runClasses(VesCollectorJunitTest.class);
-
-        for (Failure failure : result.getFailures()) {
-            log.info(failure.toString());
+import java.io.BufferedReader;
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.Paths;
+import java.util.Properties;
+import org.json.JSONObject;
+import org.junit.Test;
+
+public class TestDefaultConfiguration {
+
+    @Test
+    public void shouldDefaultCollectorSchemaFileBeAValidJson() throws IOException {
+        try (BufferedReader bufferedReader = Files.newBufferedReader(Paths.get("etc/collector.properties"))) {
+            Properties properties = new Properties();
+            properties.load(bufferedReader);
+            new JSONObject(properties.getProperty("collector.schema.file"));
         }
-
-        log.info("Execution Final result : " + result.wasSuccessful());
     }
 }
+
diff --git a/src/test/java/org/onap/dcae/vestest/TestJsonSchemaValidation.java b/src/test/java/org/onap/dcae/vestest/TestJsonSchemaValidation.java
new file mode 100644 (file)
index 0000000..2b39206
--- /dev/null
@@ -0,0 +1,57 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * PROJECT
+ * ================================================================================
+ * Copyright (C) 2017 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.dcae.vestest;
+
+import static java.nio.file.Files.readAllBytes;
+import static junit.framework.Assert.assertEquals;
+
+import com.google.gson.JsonObject;
+import com.google.gson.JsonParser;
+import java.io.IOException;
+import java.nio.file.Paths;
+import org.junit.Test;
+import org.onap.dcae.commonFunction.CommonStartup;
+
+public class TestJsonSchemaValidation {
+
+    @Test
+    public void shouldValidEventPassSchema_27_2() throws IOException {
+        String result = CommonStartup.schemavalidate(
+            readJSONFromFile("src/test/resources/VES_valid.txt").toString(),
+            readJSONFromFile("etc/CommonEventFormat_27.2.json").toString());
+        assertEquals(result, "true");
+    }
+
+
+    @Test
+    public void shouldInvalidEventDoesNotPassSchema_27_2() throws IOException {
+        String result = CommonStartup.schemavalidate(
+            readJSONFromFile("src/test/resources/VES_invalid.txt").toString(),
+            readJSONFromFile("etc/CommonEventFormat_27.2.json").toString());
+        assertEquals(result, "false");
+    }
+
+
+    private static JsonObject readJSONFromFile(String path) throws IOException {
+        return (JsonObject) new JsonParser().parse(new String(readAllBytes(Paths.get(path))));
+    }
+}
+
diff --git a/src/test/java/org/onap/dcae/vestest/TestSchemaValidation.java b/src/test/java/org/onap/dcae/vestest/TestSchemaValidation.java
deleted file mode 100644 (file)
index 816b328..0000000
+++ /dev/null
@@ -1,116 +0,0 @@
-/*-\r
- * ============LICENSE_START=======================================================\r
- * PROJECT\r
- * ================================================================================\r
- * Copyright (C) 2017-2018 AT&T Intellectual Property. All rights reserved.\r
- * ================================================================================\r
- * Licensed under the Apache License, Version 2.0 (the "License");\r
- * you may not use this file except in compliance with the License.\r
- * You may obtain a copy of the License at\r
- *\r
- *      http://www.apache.org/licenses/LICENSE-2.0\r
- *\r
- * Unless required by applicable law or agreed to in writing, software\r
- * distributed under the License is distributed on an "AS IS" BASIS,\r
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r
- * See the License for the specific language governing permissions and\r
- * limitations under the License.\r
- * ============LICENSE_END=========================================================\r
- */\r
-package org.onap.dcae.vestest;\r
-\r
-import java.io.ByteArrayInputStream;\r
-import java.io.FileNotFoundException;\r
-import java.io.FileReader;\r
-import java.io.InputStream;\r
-import java.nio.charset.StandardCharsets;\r
-\r
-import org.junit.After;\r
-import org.junit.Assert;\r
-import org.junit.Before;\r
-import org.junit.Test;\r
-import org.onap.dcae.commonFunction.CommonStartup;\r
-import org.onap.dcae.restapi.endpoints.EventReceipt;\r
-\r
-import com.google.gson.JsonParser;\r
-\r
-public class TestSchemaValidation {\r
-\r
-        CommonStartup cl;\r
-        String schema = null;\r
-        String payload = null;\r
-        String payloadinvalid = null;\r
-\r
-        @Before\r
-        public void setUp() throws Exception {\r
-\r
-                schema = new JsonParser().parse(new FileReader("etc/CommonEventFormat_27.2.json")).toString();\r
-                payload = new JsonParser().parse(new FileReader("src/test/resources/VES_valid.txt")).toString();\r
-                payloadinvalid = new JsonParser().parse(new FileReader("src/test/resources/VES_invalid.txt")).toString();\r
-\r
-        }\r
-\r
-        @After\r
-        public void tearDown() throws Exception {\r
-        }\r
-\r
-        @Test\r
-        public void testsuccessfulschemavalidation() {\r
-\r
-                String valresult = CommonStartup.schemavalidate(payload, schema);\r
-                System.out.println("testsuccessfulschemavalidation:" + valresult);\r
-                Assert.assertEquals(valresult, "true");\r
-        }\r
-\r
-        @Test\r
-        public void testunsuccessfulschemavalidation() {\r
-                String valresult = null;\r
-                valresult = CommonStartup.schemavalidate(payloadinvalid, schema);\r
-                System.out.println("testunsuccessfulschemavalidation:" + valresult);\r
-                Assert.assertFalse(valresult.equals("true"));\r
-\r
-        }\r
-\r
-/*\r
-        @Test\r
-        public void testeventReceipt() {\r
-\r
-                DrumlinServlet webServlet = new DrumlinServlet();\r
-                DrumlinRequestContext ctx = new DrumlinRequestContext(webServlet, null, null, null, null, null);\r
-                EventReceipt er= new EventReceipt();\r
-                try {\r
-                       EventReceipt.receiveVESEvent(null);\r
-                } catch ( NullPointerException   e) {\r
-                     \r
-                       e.printStackTrace();\r
-                }\r
-                Assert.assertEquals("true", "true");\r
-\r
-        }*/\r
-\r
-        @Test\r
-        public void testsafeclosefr() {\r
-\r
-\r
-                FileReader fr;\r
-                try {\r
-                        fr = new FileReader("etc/CommonEventFormat_27.2.json");\r
-                        EventReceipt.safeClose(fr);\r
-                } catch (FileNotFoundException e) {\r
-                        // TODO Auto-generated catch block\r
-                        e.printStackTrace();\r
-                }\r
-                Assert.assertEquals("true", "true");\r
-\r
-        }\r
-\r
-        @Test\r
-        public void testsafecloseis() {\r
-\r
-                        InputStream is = new ByteArrayInputStream(StandardCharsets.UTF_16.encode("randomstring").array());\r
-                        EventReceipt.safeClose(is);\r
-                        Assert.assertEquals("true", "true");\r
-\r
-        }\r
-}\r
-\r
diff --git a/src/test/java/org/onap/dcae/vestest/VesCollectorJunitTest.java b/src/test/java/org/onap/dcae/vestest/VesCollectorJunitTest.java
deleted file mode 100644 (file)
index 88ea192..0000000
+++ /dev/null
@@ -1,95 +0,0 @@
-/*-
- * ============LICENSE_START=======================================================
- * PROJECT
- * ================================================================================
- * Copyright (C) 2017 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=========================================================
- */
-
-/*
- * 
- * Purpose: CommonCollectorJunitTest is the wrapper class to invoke all prescribed Junit test cases.
- * 
- */
-
-package org.onap.dcae.vestest;
-
-import static org.junit.Assert.assertEquals;
-
-import java.io.FileInputStream;
-import java.io.IOException;
-import java.io.InputStream;
-import java.util.Properties;
-import org.json.JSONObject;
-import org.junit.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
-public class VesCollectorJunitTest {
-
-    private static final Logger log = LoggerFactory.getLogger(VesCollectorJunitTest.class);
-
-    public static String schemaFile = "etc/CommonEventFormat_27.2.json";
-    public static String output;
-
-
-    String message = "true";
-    InputJsonValidation messageUtil = new InputJsonValidation();
-
-    @Test
-    public void validJsonValidation() {
-
-        output = "true";
-        testHelper(new Properties());
-
-        assertEquals("true", output);
-    }
-
-
-    @Test
-    public void nonValidJsonValidation() {
-        output = "false";
-        testHelper(new Properties());
-        //assertEquals("false",messageUtil.nonValidJsonValidation());
-        assertEquals("false", output);
-    }
-
-    private void testHelper(Properties prop) {
-        try (InputStream input = new FileInputStream("etc/collector.properties")) {
-            prop.load(input);
-            //schemaFile=prop.getProperty("collector.schema.file");
-
-            JSONObject schemaFileJson = new JSONObject(
-                prop.getProperty("collector.schema.file"));
-            log.info("JSON Schemafile" + schemaFileJson);
-            //schemaFile = schemaFileJson.getString("v4");
-
-            log.info("Schema file location: " + schemaFile);
-        } catch (IOException e) {
-            // TODO Auto-generated catch block
-            log.error(e.getLocalizedMessage(), e);
-        }
-    }
-
-    //The test case requires common collector running in the environment prior to start execution of JUNIT test cases
-    /*
-    @Test
-    public void testValidJSONObjectReception() {
-
-        assertEquals("true",messageUtil.eventReception());
-        assertEquals("true",output);
-    }*/
-}
-