From e6cd0e70a9c8014b6bbb3758369e82c8514ec417 Mon Sep 17 00:00:00 2001 From: Sree Kuchibhotla Date: Thu, 22 Oct 2015 15:56:07 -0700 Subject: [PATCH] Return success if there are no test cases to run --- test/cpp/interop/stress_test.cc | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/test/cpp/interop/stress_test.cc b/test/cpp/interop/stress_test.cc index 7f5c494b390..91ef92af378 100644 --- a/test/cpp/interop/stress_test.cc +++ b/test/cpp/interop/stress_test.cc @@ -186,8 +186,15 @@ int main(int argc, char** argv) { // Parse test cases and weights if (FLAGS_test_cases.length() == 0) { - gpr_log(GPR_INFO, "Cannot start the tests. Test cases string is empty"); - return 1; + gpr_log(GPR_INFO, "Not running tests. The 'test_cases' string is empty"); + + // TODO(sreek): stress_tests is currently being run by run_tests.py in + // jenkins. There does not seem to be a straightforward way to skip this. + // So, for now, return 0 (i.e success) to keep jenkins build happy. Moreover + // we don't want to run stress_tests (for now) in jenkins anyway. + // Once we figure out a good way to skip this tests in run_tests.py, I will + // change this to 'return 1' + return 0; } vector> tests;