Add ability for cbuildbot to only run unit tests for revved packages.

cros_run_unit_tests

 #!/bin/bash
 
 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # This script builds and runs Chromium OS unit tests.  Note that this script
 # utilizes the src_test stanza in chromeos-base packages.  These stanzas
 # should both build and run the unit tests.
 
 # This script requires that you run build_packages first.
 
 . "$(dirname "$0")/common.sh"
 
 get_default_board
 
 # Flags
 DEFINE_string board "${DEFAULT_BOARD}" \
   "Target board of which tests were built"
 DEFINE_string build_root "${DEFAULT_BUILD_ROOT}" \
   "Root of build output"
+DEFINE_string package_file "" \
+  "File with space-separated list of packages to run unit tests" f
 DEFINE_string packages "" \
   "Optional space-separated list of packages to run unit tests" p
 
 # Total count of packages with unit tests.
 TEST_COUNT=0
 # Number of unit test failures.
 TEST_FAILURES=0
 # List of packages with no unit tests.
 NO_UNITTESTS=""
 # List of packages that have unit test failures.
@@ skipping 16 matching lines @@
 # Parse command line and die if unexpected parameters given.
 FLAGS_HELP="usage: ${0} [flags]"
 FLAGS "${@}" || exit 1
 eval set -- "${FLAGS_ARGV}"
 check_flags_only_and_allow_null_arg "${@}" && set --
 
 set -e
 
 [ -z "${FLAGS_board}" ] && die "--board required"
 
-# If no packages are specified we run all unit tests for chromeos-base
-# packages.
-if [ -n "${FLAGS_packages}" ]; then
-  PACKAGE_LIST="${FLAGS_packages}"
-else
+# Create package list from package file and list of packages.
+[ -n "${FLAGS_package_file}" ] && PACKAGE_LIST="$(cat ${FLAGS_package_file})"
+[ -n "${FLAGS_packages}" ] && PACKAGE_LIST="${PACKAGE_LIST} ${FLAGS_packages}"
+
+# If we didn't specify packages, find all packages.
+if [ -z "${FLAGS_package_file}" -a -z "${FLAGS_packages}" ]; then

[petkov, 2010/11/22 20:05:23]

can't you just -z "${PACKAGE_LIST}" here?

   PACKAGE_LIST=$( ./get_package_list chromeos --board="${FLAGS_board}" |
                   egrep '^chromeos-base' )
 fi
 
 BLACK_LIST_FILE="$(dirname "$0")/unit_test_black_list.txt"
 
 for package in ${PACKAGE_LIST}; do
   if grep -xq "${package}" "${BLACK_LIST_FILE}"; then
     warn "Skipping package ${package} since it is blacklisted."
     continue
@@ skipping 14 matching lines @@
   warn "${NO_UNITTESTS}"
 fi
 
 if [ ${TEST_FAILURES} -ne 0 ]; then
   error "Run completed with ${TEST_FAILURES}/${TEST_COUNT} test failures."
   error "Following packages had failing tests:"
   die "${FAILED_PACKAGES}"
 else
   info "All unit tests passed."
 fi