Skip to content

Commit

Permalink
Fix: Add TEST_PROJECT injected var to make-unit
Browse files Browse the repository at this point in the history
This was missed in the last commit, when make-unit.sh was modified
so that it could be set to something that isn't the triggering
gerrit project.

This commit also improves the feedback from make-unit.sh if the
Makefile isn't found.

Signed-off-by: Eric Ball <[email protected]>
Change-Id: I12b78b94edffb1a80e462450cc2e5b8883e10de2
  • Loading branch information
eb-oss committed Oct 31, 2024
1 parent 43254c1 commit fdfc3ec
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions jjb/make-unit.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@
UNIT_TEST_TARGETS={unit-test-targets}
UNIT_TEST_KEEP_GOING={unit-test-keep-going}
GOPROXY=https://proxy.golang.org
TEST_PROJECT={project}
- shell: !include-raw-escape: shell/make-unit.sh

publishers:
Expand Down
2 changes: 1 addition & 1 deletion jjb/shell/make-unit.sh
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ UNIT_TEST_TARGETS=${UNIT_TEST_TARGETS:-test}
UNIT_TEST_KEEP_GOING=${UNIT_TEST_KEEP_GOING:-false}

if [ ! -f "$test_path/Makefile" ]; then
echo "Makefile not found at $test_path!"
echo "Makefile not found at ${test_path}/Makefile"
exit 1
else
pushd "$test_path"
Expand Down

0 comments on commit fdfc3ec

Please sign in to comment.