diff options
| author | Martin Roth <gaumless@gmail.com> | 2022-11-19 15:53:11 -0700 |
|---|---|---|
| committer | Martin L Roth <gaumless@gmail.com> | 2022-11-21 20:04:53 +0000 |
| commit | b3f60a53dacc6529f6d4606c1d5388f4c6fc923f (patch) | |
| tree | c6efd5d410fa7e20773988036a677de9488c70aa /util/testing | |
| parent | 2852cd2b22c27f0574968c6b238b4bbea01746f7 (diff) | |
util/testing: Call test-lint target from what-jenkins-does
Instead of having duplicate lines in the what-jenkins-does target and
the test-lint target, make test-lint with the --junit argument from
what-jenkins-does.
Now there's only one place to update when changing the call.
Signed-off-by: Martin Roth <gaumless@gmail.com>
Change-Id: I2f90df76126f453fbcd91f4c4af5d784ac2dbe88
Reviewed-on: https://review.coreboot.org/c/coreboot/+/69835
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Diffstat (limited to 'util/testing')
| -rw-r--r-- | util/testing/Makefile.inc | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/util/testing/Makefile.inc b/util/testing/Makefile.inc index dfb66bef0b..652adc4f6b 100644 --- a/util/testing/Makefile.inc +++ b/util/testing/Makefile.inc @@ -88,10 +88,9 @@ validate_sec_tools: exit 1; \ fi -ifneq ($(JENKINS_SKIP_UNIT_TESTS),y) - util/lint/lint lint-stable --junit - util/lint/lint lint-extended --junit what-jenkins-does: test-cleanup validate_sec_tools +ifneq ($(JENKINS_SKIP_LINT_TESTS),y) + JUNIT=--junit $(MAKE) test-lint endif cd 3rdparty/intel-sec-tools/ ; go mod vendor cd util/goswid ; go mod vendor @@ -116,8 +115,8 @@ endif test-basic: test-lint test-tools test-abuild test-payloads test-cleanup test-lint: - util/lint/lint lint-stable - util/lint/lint lint-extended + util/lint/lint lint-stable $(JUNIT) + util/lint/lint lint-extended $(JUNIT) test-abuild: util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/chromeos -B -e $(if $(TEST_NOCCACHE),,-y) -c $(CPUS) -p $(TEST_PAYLOAD) -x |