Rename tests and minor refactor

Author: pavolloffay

all-in-one/src/test/java/io/jaegertracing/kubernetes/AllInOneETest.java

@@ -14,9 +14,11 @@
 package io.jaegertracing.kubernetes;
 
 import io.jaegertracing.kubernetes.deployment.BaseETest;
+import org.junit.Ignore;
 
 /**
  * @author Pavol Loffay
  */
+@Ignore
 public class AllInOneETest extends BaseETest {
 }

jaegertracing-kubernetes-deployment-itest/src/main/java/io/jaegertracing/kubernetes/deployment/BaseETest.java

@@ -67,13 +67,13 @@ public class BaseETest {
   @Named(COLLECTOR_SERVICE_NAME)
   @PortForward
   @ArquillianResource
-  protected URL collectorUrl;
+  private URL collectorUrl;
 
   @Port(9411)
   @Named(ZIPKIN_SERVICE_NAME)
   @PortForward
   @ArquillianResource
-  protected URL zipkinUrl;
+  private URL zipkinUrl;
 
   @Test
   public void testUiResponds() throws IOException, InterruptedException {

production-elasticsearch/src/test/java/io/jaegertracing/kubernetes/ProductionETest.java renamed to production-elasticsearch/src/test/java/io/jaegertracing/kubernetes/ElasticearchETest.java

@@ -20,13 +20,14 @@
 import okhttp3.Request;
 import okhttp3.Response;
 import org.junit.Before;
+import org.junit.Ignore;
 
 import static org.awaitility.Awaitility.await;
 
 /**
  * @author Pavol Loffay
  */
-public class ProductionETest extends BaseETest {
+public class ElasticearchETest extends BaseETest {
 
   /**
    * We need to initialize ES storage, before we proceed to tests for two reasons:
@@ -53,6 +54,7 @@ public void before() {
     });
   }
 
+  @Ignore("It requires spark job")
   public void testDependencyLinks() throws IOException, InterruptedException {
   }
 }

production/src/test/java/io/jaegertracing/kubernetes/ProductionETest.java renamed to production/src/test/java/io/jaegertracing/kubernetes/CassandraETest.java

@@ -20,9 +20,9 @@
 /**
  * @author Pavol Loffay
  */
-public class ProductionETest extends BaseETest {
+public class CassandraETest extends BaseETest {
 
-  @Ignore("dependency links returns 404 because of old Cassandra image")
+  @Ignore("It requires spark job")
   public void testDependencyLinks() throws IOException, InterruptedException {
   }
 }