Merge pull request #2 from SunZhaonan/master

Exclude tests that need configurations
datahub-project · Nov 19, 2015 · 9a315a7 · 9a315a7
2 parents d5c3d87 + b83c0a4
commit 9a315a7
Show file tree

Hide file tree

Showing 7 changed files with 15 additions and 25 deletions.
diff --git a/build.gradle b/build.gradle
@@ -24,6 +24,15 @@ subprojects {
       url 'https://repo.typesafe.com/typesafe/maven-releases/'
     }
   }
+  task excludegroup(type: Test) {
+    useTestNG() {
+      excludeGroups 'needConfig'
+    }
+
+    testLogging {
+      showStandardStreams = true
+    }
+  }
 
   ext.externalDependency = [//"mysql"              : "mysql:mysql-connector-java:5.1.36",
                             //"jython"             : "org.python:jython-standalone:2.7.0",

diff --git a/metadata-etl/build.gradle b/metadata-etl/build.gradle
@@ -58,12 +58,4 @@ jar {
   }
 }
 
-test {
-  useTestNG() {
-    excludeGroups 'needConfig'
-  }
 
-  testLogging {
-    showStandardStreams = true
-  }
-}
diff --git a/metadata-etl/src/test/java/metadata/etl/scheduler/azkaban/AzkabanExecEtlTest.java b/metadata-etl/src/test/java/metadata/etl/scheduler/azkaban/AzkabanExecEtlTest.java
@@ -13,15 +13,14 @@
  */
 package metadata.etl.scheduler.azkaban;
 
-import junit.framework.TestCase;
 import org.testng.annotations.BeforeTest;
 import org.testng.annotations.Test;
 
 
 /**
  * Created by zechen on 8/25/15.
  */
-public class AzkabanExecEtlTest extends TestCase {
+public class AzkabanExecEtlTest{
   AzkabanExecEtl az;
 
   @BeforeTest

diff --git a/metadata-etl/src/test/java/metadata/etl/scheduler/oozie/OozieExecEtlTest.java b/metadata-etl/src/test/java/metadata/etl/scheduler/oozie/OozieExecEtlTest.java
@@ -13,15 +13,14 @@
  */
 package metadata.etl.scheduler.oozie;
 
-import junit.framework.TestCase;
 import org.testng.annotations.BeforeTest;
 import org.testng.annotations.Test;
 
 
 /**
  * Created by zechen on 10/1/15.
  */
-public class OozieExecEtlTest extends TestCase {
+public class OozieExecEtlTest {
   OozieExecEtl oz;
 
   @BeforeTest

diff --git a/wherehows-common/build.gradle b/wherehows-common/build.gradle
@@ -8,13 +8,3 @@ dependencies {
   testCompile externalDependency.testng
   testCompile project(":metadata-etl")
 }
-
-test {
-  useTestNG() {
-    excludeGroups 'ignore'
-  }
-
-  testLogging {
-    showStandardStreams = true
-  }
-}
diff --git a/wherehows-common/src/test/java/DatasetPathTest.java b/wherehows-common/src/test/java/DatasetPathTest.java
@@ -35,7 +35,7 @@ public void separatedDataset() {
   }
 
   @Test
-  public void seteratedDatasetTest2() {
+  public void seperatedDatasetTest2() {
     String sample2 = "/.{/jobs/snusm/online/modeling/train-data/slot-1,/jobs/snusm/online/modeling/test-data/slot-1}";
     List<String> result2 = DatasetPath.separatedDataset(sample2);
     Assert.assertEquals(result2.get(0), sample2);

diff --git a/wherehows-common/src/test/java/LineageCombinerTest.java b/wherehows-common/src/test/java/LineageCombinerTest.java
@@ -29,6 +29,7 @@
 /**
  * Created by zsun on 9/13/15.
  */
+@Test(groups = {"wherehows.common"})
 public class LineageCombinerTest {
   LineageCombiner lineageCombiner;
   @BeforeTest
@@ -45,7 +46,7 @@ public void setUp()
     PathAnalyzer.initialize(conn);
   }
 
-  @Test
+  @Test(groups = {"needConfig"})
   public void lineageCombinerTest() {
     LineageRecord lineageRecord = new LineageRecord(0, 111L, "some_job_name", 111001L);
     lineageRecord.setDatasetInfo(0,
@@ -75,7 +76,7 @@ public void lineageCombinerTest() {
       "'some_app_name','0','111','0','111001','some_job_name','0','0','some_database_name','0','/data/hadoop/someDataSet','hdfs://eat1-nertznn01.grid.linkedin.com:9000/data/hadoop/someDataSet/daily/2015/09/01/part1','2015/09/01','2015/09/02','daily','1','','source','0','0','read','0','0','0','0',NULL,NULL");
   }
 
-  @Test
+  @Test(groups = {"needConfig"})
   public void analyzeTest() {
     String fullPath = "hdfs://lva1-warnn01.grid.linkedin.com:9000/jobs/siteflow/scoring/score/abook_decision/heathrowInactive/part-m-00004";
     DatasetPath datasetPath = PathAnalyzer.analyze(fullPath);