Always run //Documentation:check_licenses on PolyGerrit validations
The recent breakage on Gerrit master (see [1]) has highlighted the need
of running the check of the license file also for the PolyGerrit-only
changes. In [1] the removal of a library (moment.js) was not accompanied
with the update of the Gerrit licenses and therefore broke the build.
The breakage wasn't reported by the CI because the total set of tests
is skipped for PolyGerrit-only changes.
The license file generated and included in the Gerrit documentation set
is dependent on the NodeJS packages included in the PolyGerrit components.
[1] https://gerrit-review.googlesource.com/c/gerrit/+/266035
Change-Id: I8c5de7fe1478f6c924dd9e38420a7bdb68ac4d3f
diff --git a/jenkins/gerrit-bazel-verifier-test.sh b/jenkins/gerrit-bazel-verifier-test.sh
index cb89104..85382ef 100755
--- a/jenkins/gerrit-bazel-verifier-test.sh
+++ b/jenkins/gerrit-bazel-verifier-test.sh
@@ -39,6 +39,10 @@
echo 'Running local tests...'
bash ./polygerrit-ui/app/run_test.sh || touch ~/polygerrit-failed
fi
+
+ echo 'Running license verification...'
+ bazelisk test $BAZEL_OPTS //Documentation:check_licenses
+
if [ -z "$SAUCE_USERNAME" ] || [ -z "$SAUCE_ACCESS_KEY" ]
then
echo 'Not running on Sauce Labs because env vars are not set.'
