update readme and comments

Author: vanajamukkara

integration-tests/README.md

@@ -18,8 +18,10 @@ Java integration tests cover the below use cases:
 
 Quick test use cases - 
 
-Operator Configuration - operator1 deployed in weblogic-operator1 namespace and manages domains in default and test1 namespaces
-Domain Configuration - Domain on PV using WLST, traefik load balancer
+|  |  |
+| --- | --- |
+| Operator Configuration | operator1 deployed in weblogic-operator1 namespace and manages domains in default and test1 namespaces |
+| Domain Configuration | Domain on PV using WLST, traefik load balancer |
 
 Basic Use Cases
 
@@ -238,36 +240,38 @@ Successful run will have the output like below:
 [INFO] Final Memory: 60M/1236M
 
 ```
-Failed run will have the output
+Failed run will have the output like
 ```
 
 [INFO] 
 [INFO] Results:
 [INFO] 
 [ERROR] Errors: 
-[ERROR]   ITOperator.testBCreateDomainWithDefaultValuesInSampleInputs:287->testAllUseCasesForADomain:303->BaseTest.testClusterScaling:241 ? Runtime
+[ERROR]   ITOperator.testDomainOnPVUsingWLST:145 ? Runtime FAILURE: Couldn't create serv...
 [INFO] 
-[ERROR] Tests run: 11, Failures: 0, Errors: 1, Skipped: 0
+[ERROR] Tests run: 9, Failures: 0, Errors: 1, Skipped: 0
 [INFO] 
 [INFO] 
 [INFO] --- maven-failsafe-plugin:2.20.1:verify (integration-tests) @ operator-integration-tests ---
 [INFO] ------------------------------------------------------------------------
 [INFO] Reactor Summary:
 [INFO] 
-[INFO] weblogic-kubernetes-operator ....................... SUCCESS [  1.669 s]
-[INFO] operator-model ..................................... SUCCESS [ 22.159 s]
-[INFO] operator-swagger ................................... SUCCESS [  1.426 s]
-[INFO] operator-runtime ................................... SUCCESS [ 54.559 s]
-[INFO] operator-integration-tests ......................... FAILURE [  01:03 h]
+[INFO] Build Tools ........................................ SUCCESS [  1.193 s]
+[INFO] weblogic-kubernetes-operator ....................... SUCCESS [  2.671 s]
+[INFO] json-schema ........................................ SUCCESS [ 14.917 s]
+[INFO] jsonschema-maven-plugin Maven Mojo ................. SUCCESS [  8.600 s]
+[INFO] operator-model ..................................... SUCCESS [ 23.065 s]
+[INFO] operator-swagger ................................... SUCCESS [  4.487 s]
+[INFO] operator-runtime ................................... SUCCESS [ 53.675 s]
+[INFO] operator-integration-tests ......................... FAILURE [41:09 min]
 [INFO] installation-tests ................................. SKIPPED
+[INFO] Project Reports .................................... SKIPPED
 [INFO] ------------------------------------------------------------------------
 [INFO] BUILD FAILURE
 [INFO] ------------------------------------------------------------------------
-[INFO] Total time: 01:04 h
-[INFO] Finished at: 2018-10-31T15:19:46-07:00
-[INFO] Final Memory: 48M/1275M
-[INFO] ------------------------------------------------------------------------
-
+[INFO] Total time: 42:58 min
+[INFO] Finished at: 2019-02-11T09:42:08-08:00
+[INFO] Final Memory: 124M/1534M
 ```
 JUnit test results can be seen at "integration-tests/target/failsafe-reports/TEST-oracle.kubernetes.operator.ITOperator.xml". This file shows how much time each test case took to run and the failed test results if any.
 

integration-tests/src/test/java/oracle/kubernetes/operator/ITOperator.java

@@ -200,11 +200,12 @@ public void testDomainOnPVUsingWDT() throws Exception {
   }
 
   /**
-   * Create two operators if they are not running. Create domain domain4 with dynamic cluster in
-   * default namespace, managed by operator1. Create domain domain5 with Configured cluster using
-   * WDT in test2 namespace, managed by operator2 Verify scaling for domain5 cluster from 2 to 3
-   * servers and back to 2, plus verify no impact on domain4 Cycle domain4 down and back up, plus
-   * verify no impact on domain5 shutdown by deleting both domain4 and domain5 CRD's
+   * Create two operators if they are not running. Create domain domain1 with dynamic cluster in
+   * default namespace, managed by operator1. Create domain domain2 with Configured cluster using
+   * WDT in test2 namespace, managed by operator2. Verify scaling for domain2 cluster from 2 to 3
+   * servers and back to 2, plus verify no impact on domain1. Cycle domain1 down and back up, plus
+   * verify no impact on domain2. shutdown by the domains using the delete resource script from
+   * samples.
    *
    * <p>ToDo: configured cluster support is removed from samples, modify the test to create
    *
@@ -215,7 +216,7 @@ public void testTwoDomainsManagedByTwoOperators() throws Exception {
     Assume.assumeFalse(QUICKTEST);
     String testMethodName = new Object() {}.getClass().getEnclosingMethod().getName();
     logTestBegin(testMethodName);
-    logger.info("Creating Domain domain4 & verifing the domain creation");
+    logger.info("Creating Domain domain1 & verifing the domain creation");
 
     logger.info("Checking if operator1 and domain1 are running, if not creating");
     if (operator1 == null) {
@@ -237,7 +238,7 @@ public void testTwoDomainsManagedByTwoOperators() throws Exception {
       if (operator2 == null) {
         operator2 = TestUtils.createOperator(operator2File);
       }
-      // create domain5 with configured cluster
+      // create domain2 with configured cluster
       // ToDo: configured cluster support is removed from samples, modify the test to create
       // configured cluster
       Map<String, Object> wdtDomainMap = TestUtils.loadYaml(domainonpvwdtFile);
@@ -256,7 +257,7 @@ public void testTwoDomainsManagedByTwoOperators() throws Exception {
       logger.info("Verify the only remaining running domain domain1 is unaffected");
       domain1.verifyDomainCreated();
 
-      logger.info("Destroy and create domain4 and verify no impact on domain2");
+      logger.info("Destroy and create domain1 and verify no impact on domain2");
       domain1.destroy();
       domain1.create();
 
@@ -299,7 +300,7 @@ public void testCreateDomainWithStartPolicyAdminOnly() throws Exception {
       operator1 = TestUtils.createOperator(operator1File);
     }
     logger.info("Creating Domain domain6 & verifing the domain creation");
-    // create domain6
+    // create domain
     Domain domain = null;
     boolean testCompletedSuccessfully = false;
     try {
@@ -332,8 +333,8 @@ public void testCreateDomainPVReclaimPolicyRecycle() throws Exception {
     if (operator1 == null) {
       operator1 = TestUtils.createOperator(operator1File);
     }
-    logger.info("Creating Domain domain7 & verifing the domain creation");
-    // create domain7
+    logger.info("Creating Domain domain & verifing the domain creation");
+    // create domain
     Domain domain = null;
 
     try {