Skip to content

Commit ac79b78

Browse files
valentin-seitzValentin SeitzMakisH
authored
Added a test workflow (#309)
Co-authored-by: Valentin Seitz <[email protected]> Co-authored-by: Gerasimos Chourdakis <[email protected]>
1 parent a353484 commit ac79b78

File tree

2 files changed

+39
-0
lines changed

2 files changed

+39
-0
lines changed

.github/workflows/system-tests.yaml

+29
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
name: System tests
2+
3+
on:
4+
pull_request:
5+
types: [labeled]
6+
7+
jobs:
8+
gather-refs:
9+
if: ${{ github.event.label.name == 'trigger-system-tests' }}
10+
runs-on: ubuntu-latest
11+
outputs:
12+
reftutorials: ${{ steps.reftutorials.outputs.shorthash }}
13+
steps:
14+
- id: reftutorials
15+
uses: nmbgeek/github-action-get-latest-commit@main
16+
with:
17+
owner: precice
18+
repo: tutorials
19+
branch: develop
20+
21+
run-system-tests:
22+
if: ${{ github.event.label.name == 'trigger-system-tests' }}
23+
needs: gather-refs
24+
uses: precice/tutorials/.github/workflows/run_testsuite_workflow.yml@develop
25+
with:
26+
suites: openfoam_adapter_pr
27+
build_args: TUTORIALS_REF:${{ needs.gather-refs.outputs.reftutorials }},PRECICE_REF:v3.0.0,OPENFOAM_EXECUTABLE:openfoam2306,OPENFOAM_ADAPTER_REF:${{ github.event.pull_request.head.sha }}
28+
systests_branch: develop
29+
loglevel: "DEBUG"

CONTRIBUTING.md

+10
Original file line numberDiff line numberDiff line change
@@ -27,3 +27,13 @@ There are also a few additional workflows that can be triggered manually:
2727
- `Check links`: checks the links in all markdown files to verify if they are still reachable.
2828

2929
Members of the repository can trigger these workflows in the "Actions" tab.
30+
31+
## System tests
32+
33+
For non-trivial pull requests, we also need to execute [system regression tests](https://precice.org/dev-docs-system-tests.html),
34+
to ensure that complete simulations still run and give the same results.
35+
Because these take long, run on an external system, and consume significant resources,
36+
we are only triggering these on demand. Add (or ask a maintainer to add) the
37+
`trigger-system-tests` label to the pull request to trigger them.
38+
The tests will only run once, so that further commits don't consume additional
39+
resources: In case you want to re-trigger them, remove and add the label again.

0 commit comments

Comments
 (0)