SAMZA-1451: Disable integration tests conditionally in build.
authorShanthoosh Venkataraman <svenkataraman@linkedin.com>
Wed, 11 Oct 2017 20:56:24 +0000 (13:56 -0700)
committerPrateek Maheshwari <pmaheshw@linkedin.com>
Wed, 11 Oct 2017 20:56:24 +0000 (13:56 -0700)
Remove runIntegrationTests gradle property added as a part of SAMZA-1355 and introduce skipIntegrationTests property(which inverts it).
If skipIntegrationTests gradle project property is enabled, execution of all tests in samza-test project will be skipped from the build.

Author: Shanthoosh Venkataraman <svenkataraman@linkedin.com>

Reviewers: Prateek Maheshwari <pmaheshwari@apache.org>

Closes #319 from shanthoosh/skip_integration_tests

README.md
bin/check-all.sh
docs/contribute/tests.md
settings.gradle

index 0164dd3..334c7b4 100644 (file)
--- a/README.md
+++ b/README.md
@@ -34,7 +34,7 @@ Samza builds with [Scala](http://www.scala-lang.org/) 2.10 or 2.11 and [YARN](ht
 
 ### Testing Samza
 
-To run all unit tests:
+To run all tests:
 
     ./gradlew clean test
 
index ceb1e99..2f9f03c 100755 (executable)
@@ -81,7 +81,7 @@ do
     for yarn_version in "${YARNs[@]}"
     do
       echo "------------- Running check task against JDK${jdk_number}/Scala ${scala_version}/YARN ${yarn_version}"
-      ${gradle_file} -PscalaVersion=${scala_version} -PyarnVersion=${yarn_version} -Dorg.gradle.java.home=${!i} -PrunIntegrationTests clean check $@
+      ${gradle_file} -PscalaVersion=${scala_version} -PyarnVersion=${yarn_version} -Dorg.gradle.java.home=${!i} clean check $@
       echo "------------- Finished running check task against JDK${jdk_number}/Scala ${scala_version}/YARN ${yarn_version}"
     done
   done
index 048e3be..9fe728c 100644 (file)
@@ -25,7 +25,7 @@ Samza's unit tests are written on top of [JUnit](http://junit.org/), and license
 
 To run all tests, and license checks:
 
-    ./gradlew clean check -PrunIntegrationTests
+    ./gradlew clean check
 
 To run a single test:
 
index 603cd35..4c81d9c 100644 (file)
@@ -48,18 +48,17 @@ rootProject.children.each {
 }
 
 /**
- * Skips execution of all integration tests in project 'samza-test'.
- * To run integration tests in samza-test: './gradlew clean build -PrunIntegrationTests'
+ * To skip integration tests in samza-test: './gradlew clean build -PskipIntegrationTests'
  */
 gradle.taskGraph.whenReady { taskGraph ->
   taskGraph.getAllTasks().each { task ->
     def project = task.getProject()
     task.onlyIf {
       /**
-       * Behaves as pass through filter for all tasks when `runIntegrationTests` property is turned on.
+       * Behaves as pass through filter for all tasks when `skipIntegrationTests` property is turned off.
        * Filters 'test' task of 'samza-test' project otherwise.
        */
-      project.hasProperty("runIntegrationTests") || !(project.getName().contains("samza-test") && task.getName() == "test")
+      !project.hasProperty("skipIntegrationTests") || !(project.getName().contains("samza-test") && task.getName() == "test")
     }
   }
 }