Re-land "Run unit tests on all cros-workon packages in full builders."
Currently, unit tests are only run on packages whose names start with
'chromeos-base'. This CL updates the full builders to run unit tests for
all cros-workon ebuilds.
As-is, this CL enables unit tests on full builders for 7 ebuilds that
previously only ran unit tests on the commit queue.
- chromeos-base/chromite
- chromeos-base/cros-devutils
- dev-util/quipper
- media-sound/adhd
- sys-apps/dtc
- sys-apps/flashmap
- sys-kernel/chromeos-kernel
BUG=chromium-os:39038
TEST=Run paladin, full, and canary builders.
Change-Id: I8c2b4d9a660dffbc00aa3b8b11a83ee38495aa68
Original-Change-Id: Ibf54ad59e615e317e358148f5f736e95185d5ec3
Previously-Reviewed-on: https://gerrit.chromium.org/gerrit/43547
CQ-DEPEND=CL:43619, CL:43621
Reviewed-on: https://gerrit.chromium.org/gerrit/43622
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: David James <davidjames@chromium.org>
Tested-by: David James <davidjames@chromium.org>
Reviewed-by: Peter Mayo <petermayo@chromium.org>
diff --git a/cros_run_unit_tests b/cros_run_unit_tests
index 6716ff9..7b5c12b 100755
--- a/cros_run_unit_tests
+++ b/cros_run_unit_tests
@@ -66,9 +66,10 @@
# If we didn't specify packages, find all packages.
if [ -z "${FLAGS_package_file}" -a -z "${FLAGS_packages}" ]; then
- PACKAGE_LIST=$( /usr/lib/crosutils/get_package_list chromeos \
- --board="${FLAGS_board}" |
- egrep '^chromeos-base' )
+ PACKAGE_LIST=$( cd /usr/local/portage/chromiumos;
+ git grep -lE '^[[:space:]]*inherit.*\<cros-workon\>' | \
+ xargs git grep -lE '^src_test()' | \
+ xargs -n 1 dirname | grep '/' | sort -u )
fi
BLACK_LIST_FILE="/usr/share/crostestutils/unit_test_black_list.txt"
@@ -87,9 +88,9 @@
warn "${package} not found"
if [ -n "${EBUILD_PATH}" ]; then
if check_src_test "${EBUILD_PATH}"; then
- TEST_PACKAGE_LIST="${TEST_PACKAGE_LIST} ${package}"
+ TEST_PACKAGE_LIST+=" ${package}"
else
- NO_UNITTESTS="${NO_UNITTESTS} ${package}"
+ NO_UNITTESTS+=" ${package}"
fi
fi
done
