diff options
author | Vic (Chun-Ju) Yang <victoryang@chromium.org> | 2014-02-06 16:45:28 +0800 |
---|---|---|
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-02-06 19:27:18 +0000 |
commit | 798df2a6d077cb973cb60c5401d0ec037006fe9a (patch) | |
tree | 0c92b15cb6967d26b456ae99d9c716ca2b813e23 /util | |
parent | 66eec4415a0cf8a74aa9dd26c957423b63681e6b (diff) | |
download | chrome-ec-798df2a6d077cb973cb60c5401d0ec037006fe9a.tar.gz |
Add presubmit check for unit test
This enforces that "make buildall" runs at least once after the last
file change.
TEST=Try to upload without running "make buildall"
TEST=Change a file without re-running "make buildall", and try to
upload.
BUG=None
BRANCH=None
Change-Id: Ia4abb3c0e17cf4d559975574f398d74c7986c89f
Signed-off-by: Vic (Chun-Ju) Yang <victoryang@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/185116
Reviewed-by: Dave Parker <dparker@chromium.org>
Diffstat (limited to 'util')
-rwxr-xr-x | util/presubmit_check.sh | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/util/presubmit_check.sh b/util/presubmit_check.sh new file mode 100755 index 0000000000..220522083e --- /dev/null +++ b/util/presubmit_check.sh @@ -0,0 +1,17 @@ +#!/bin/bash +# +# Copyright (c) 2014 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. + +if [[ ! -e .tests-passed ]]; then + echo 'Unit tests have not passed. Please run "make buildall -j".' + exit 1 +fi +changed=$(find ${PRESUBMIT_FILES} -newer .tests-passed) +if [[ -n "${changed}" ]]; then + echo "Files have changed since last time unit tests passed:" + echo "${changed}" | sed -e 's/^/ /' + echo 'Please run "make buildall -j".' + exit 1 +fi |