Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/15428


Change subject: [gradle] remove failOnNoMatchingTests
......................................................................

[gradle] remove failOnNoMatchingTests

I found it confusing when gradle reports success on executing test
targets which don't exist.  Prior to this patch, the following returned
success and claimed successfully executing tasks:

  $ ./gradlew :kudu-client:test --tests org.apache.kudu.client.Nonsense

  BUILD SUCCESSFUL in 1s
  15 actionable tasks: 1 executed, 14 up-to-date

This patch fixes it by removing failOnNoMatchingTests.

It seems there were some ramifications on the pattern matching for
dist-test some time ago, but I verified prior and after this patch
and saw that the following works and breaks the same way:

works:
  ../build-support/dist_test.py java loop -n 8 '*ITClient'
  ../build-support/dist_test.py java loop -n 8 '*ITClient*'
  ../build-support/dist_test.py java loop -n 8 
org/apache/kudu/client/TestConnectToCluster

doesn't work:
  ../build-support/dist_test.py java loop -n 8 '*ITClient.class'
  ../build-support/dist_test.py java loop -n 8 '*ITClient*X'
  ../build-support/dist_test.py java loop -n 8 
org/apache/kudu/client/TestConnectToCluster.class

Change-Id: Iabb0f72fb9481c775ebcc9881a39180fb0413154
---
M java/gradle/tests.gradle
1 file changed, 0 insertions(+), 7 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/28/15428/1
--
To view, visit http://gerrit.cloudera.org:8080/15428
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iabb0f72fb9481c775ebcc9881a39180fb0413154
Gerrit-Change-Number: 15428
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to