Skip to content

Commit

Permalink
[ci] Allow RBE build to skip known-bad tests
Browse files Browse the repository at this point in the history
  • Loading branch information
shs96c committed Aug 1, 2023
1 parent 6f8ac08 commit 9868754
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 2 additions & 0 deletions .skipped-tests
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
-//java/test/org/openqa/selenium:FormHandlingTest-chrome
-//java/test/org/openqa/selenium/federatedcredentialmanagement:FederatedCredentialManagementTest
7 changes: 4 additions & 3 deletions scripts/github-actions/ci-build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,15 +7,15 @@ set -x
# The ruby version may have been set by the CI runner. Stash
# changes while we check to see if we need to reformat the
# code.
git stash
git commit -am 'Temp commit to allow format to run cleanly'

# Fail the build if the format script needs to be re-run
./scripts/format.sh
git diff --exit-code

# Now we're made it out, reapply changes made by the build
# runner
git stash apply
git reset --soft HEAD^

# The NPM repository rule wants to write to the HOME directory
# but that's configured for the remote build machines, so run
Expand All @@ -25,4 +25,5 @@ bazel query @npm//:all >/dev/null

# Now run the tests. The engflow build uses pinned browsers
# so this should be fine
bazel test --config=remote-ci --keep_going //java/...
# shellcheck disable=SC2046
bazel test --config=remote-ci --test_tag_filters=-exclusive-if-local,-skip-remote --keep_going --flaky_test_attempts=2 //java/... -- $(cat .skipped-tests | tr '\n' ' ')

0 comments on commit 9868754

Please sign in to comment.