Skip to content

Commit

Permalink
report full path during benchmark
Browse files Browse the repository at this point in the history
  • Loading branch information
bdemann committed Nov 1, 2024
1 parent f1a9cce commit d547311
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/benchmark_parallel.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,10 @@ jobs:
ref: ${{ inputs.base-branch }}
token: ${{ secrets.LASTMJS_GITHUB_TOKEN || github.token }}

- name: Report full path of test
# Just in case the path isn't obvious from the name, this will remove ambiguity
run: echo ${{matrix.test.path}}

- uses: ./.github/actions/setup_node

- uses: ./.github/actions/setup_dfx
Expand Down
16 changes: 8 additions & 8 deletions .github/workflows/run_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ on:

jobs:
run-test:
name: '${{matrix.tests.name}} | ${{matrix.tests.displayPath}} | ${{matrix.azle_source}}'
name: '${{matrix.test.name}} | ${{matrix.test.displayPath}} | ${{matrix.azle_source}}'
runs-on: ${{ matrix.os }}
env:
ETHEREUM_URL: ${{ secrets.ETHEREUM_URL }}
Expand All @@ -35,7 +35,7 @@ jobs:
os: [ubuntu-latest]
azle_source:
- ${{ inputs.include_npm == true && 'npm' || 'repo' }}
tests: ${{ fromJSON(inputs.test_infos) }}
test: ${{ fromJSON(inputs.test_infos) }}
steps:
- uses: actions/checkout@v4

Expand All @@ -60,7 +60,7 @@ jobs:
- name: Report full path of test
# Just in case the path isn't obvious from the name, this will remove ambiguity
run: echo ${{matrix.tests.path}}
run: echo ${{matrix.test.path}}

- uses: ./.github/actions/setup_node

Expand All @@ -82,7 +82,7 @@ jobs:
npm run lint
shell: bash -l {0}

- name: Run pre-test setup for ${{ matrix.tests.name }}
- name: Run pre-test setup for ${{ matrix.test.name }}
run: |
npm install
Expand All @@ -91,17 +91,17 @@ jobs:
fi
npx azle install-dfx-extension
working-directory: ${{ matrix.tests.path }}
working-directory: ${{ matrix.test.path }}
shell: bash -l {0}

- name: Start dfx with artificial delay 0
if: ${{ steps.set-conditions.outputs.is_feature_branch_pr == 'true' || steps.set-conditions.outputs.is_feature_branch_draft_pr == 'true' }}
working-directory: ${{ matrix.tests.path }}
working-directory: ${{ matrix.test.path }}
run: dfx start --clean --background --host 127.0.0.1:8000 --artificial-delay 0

- name: Start dfx
if: ${{ steps.set-conditions.outputs.is_release_branch_pr == 'true' || steps.set-conditions.outputs.is_main_branch_push == 'true' || steps.set-conditions.outputs.is_main_branch_push_from_release_merge == 'true' }}
working-directory: ${{ matrix.tests.path }}
working-directory: ${{ matrix.test.path }}
run: dfx start --clean --background --host 127.0.0.1:8000

- name: Calculate number of test runs
Expand Down Expand Up @@ -136,7 +136,7 @@ jobs:
- name: Run tests
run: npm test
shell: bash -l {0}
working-directory: ${{ matrix.tests.path }}
working-directory: ${{ matrix.test.path }}
env:
AZLE_PROPTEST_NUM_RUNS: ${{ steps.calc-runs.outputs.runs }}
AZLE_PROPTEST_VERBOSE: true
Expand Down

0 comments on commit d547311

Please sign in to comment.