Skip to content

Commit ae6c6a6

Browse files
authored
[hermes] Add docker.sh test runner (#3186)
gherrit-pr-id: Gyk2gmcgoj5je5p6tcbubsnoabjsbb575
1 parent 5a0a201 commit ae6c6a6

File tree

6 files changed

+503
-2915
lines changed

6 files changed

+503
-2915
lines changed

.github/workflows/hermes.yml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -48,22 +48,19 @@ jobs:
4848
cache-from: type=gha
4949
cache-to: type=gha,mode=max
5050

51-
- name: Create Hermes Cache Volume
52-
run: docker volume create hermes-cache
53-
5451
# Run unit tests separately, as they're much less likely to have bugs
5552
# during local development, and this makes the GitHub Actions output
5653
# easier to skim (in particular, it's clear at a glance whether a failure
5754
# is due to unit or integration tests).
5855
- name: Run unit tests
59-
run: docker run --rm -v $GITHUB_WORKSPACE/tools:/workspace -v hermes-cache:/root/.hermes hermes-ci:local cargo test --verbose --bin cargo-hermes
56+
run: docker run --rm -v $GITHUB_WORKSPACE/tools:/workspace hermes-ci:local cargo test --verbose --bin cargo-hermes
6057

6158
# We duplicate running unit tests since they're very cheap compared to
6259
# integration tests, and this way it's easier to be sure that we run all
6360
# tests instead of specifically trying to carve out unit tests and risk
6461
# missing test categories.
6562
- name: Run all tests
66-
run: docker run --rm -v $GITHUB_WORKSPACE/tools:/workspace -v hermes-cache:/root/.hermes hermes-ci:local cargo test --verbose
63+
run: docker run --rm -v $GITHUB_WORKSPACE/tools:/workspace hermes-ci:local cargo test --verbose
6764

6865
# Used to signal to branch protections that all other jobs have succeeded.
6966
all-jobs-succeed:

0 commit comments

Comments
 (0)