You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ds...@apache.org on 2015/11/03 00:46:24 UTC

[4/8] incubator-geode git commit: GEODE-487: Remove integrationTest and distributedTest from build task.

GEODE-487: Remove integrationTest and distributedTest from build task.

This commit removes integrationTest and distributedTest tasks from the
build task lifecycle. The tasks were associated with the 'check' task,
which is in the 'build' lifecycle. After removing the tasks, only the
'test' task is run when build task is executed.

Tested by running 'clean build'

Review available at https://reviews.apache.org/r/39664


Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/b2dea99c
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/b2dea99c
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/b2dea99c

Branch: refs/heads/feature/GEODE-409
Commit: b2dea99cbb909edf0ccca9d117147316c44e87bb
Parents: 28eb309
Author: Mark Bretl <mb...@pivotal.io>
Authored: Mon Oct 26 11:26:54 2015 -0700
Committer: Mark Bretl <mb...@pivotal.io>
Committed: Tue Oct 27 10:54:18 2015 -0700

----------------------------------------------------------------------
 build.gradle | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/b2dea99c/build.gradle
----------------------------------------------------------------------
diff --git a/build.gradle b/build.gradle
index 980eba8..7e477f3 100755
--- a/build.gradle
+++ b/build.gradle
@@ -409,10 +409,13 @@ subprojects {
     }
   })
 
-  check.dependsOn integrationTest
-  check.dependsOn distributedTest 
+  // Make precheckin task run all validation tests for checking in code.
+  task precheckin (dependsOn: [ build, integrationTest, distributedTest ]) {
+    description 'Run this task before checking in code to validate changes. This task combines the following tasks: build, integrationTest, and distributedTest'
+  }
+
   check.dependsOn checkMissedTests
-  combineReports.mustRunAfter check, test, integrationTest, distributedTest,checkMissedTests
+  combineReports.mustRunAfter check, test, integrationTest, distributedTest, checkMissedTests
   build.finalizedBy combineReports
   check.finalizedBy combineReports
   test.finalizedBy combineReports