From 09aa7715d4bfc03e8df548f9e0c179ba49402a3e Mon Sep 17 00:00:00 2001 From: Ananth Suryanarayana Date: Tue, 13 Sep 2016 11:48:24 -0700 Subject: [PATCH] Fix coverage build error No need to run coverage on flaky tests suite, if it is empty Change-Id: Ib63f52f84478d79221ee67d6e3f76ab976c71d3a Partial-Bug: 1596630 --- src/io/test/SConscript | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/src/io/test/SConscript b/src/io/test/SConscript index ca9fc1981bf..6573515388a 100644 --- a/src/io/test/SConscript +++ b/src/io/test/SConscript @@ -124,13 +124,14 @@ else: env.AlwaysBuild(run_coverage_cmd) env.Alias('controller/src/io:test', run_coverage_cmd) - flaky_test = env.TestSuite('io-flaky-test', flaky_test_suite) - clean_coverage_cmd = env.Command('clean_coverage', '', clean_coverage) - env.Depends(flaky_test, clean_coverage_cmd[0].name) - env.AlwaysBuild(clean_coverage_cmd) - run_coverage_cmd = env.Command('run_coverage', '', run_coverage) - env.Depends(run_coverage_cmd, flaky_test[0]) - env.AlwaysBuild(run_coverage_cmd) - env.Alias('controller/src/io:flaky-test', run_coverage_cmd) + if len(flaky_test_suite) != 0: + flaky_test = env.TestSuite('io-flaky-test', flaky_test_suite) + clean_coverage_cmd = env.Command('clean_coverage', '', clean_coverage) + env.Depends(flaky_test, clean_coverage_cmd[0].name) + env.AlwaysBuild(clean_coverage_cmd) + run_coverage_cmd = env.Command('run_coverage', '', run_coverage) + env.Depends(run_coverage_cmd, flaky_test[0]) + env.AlwaysBuild(run_coverage_cmd) + env.Alias('controller/src/io:flaky-test', run_coverage_cmd) Return('test_suite')