Skip to content

Commit

Permalink
Make changes to tests trigger running all tests
Browse files Browse the repository at this point in the history
Needed to change workflow so that changes inside the tests folder also trigger running all tests.
  • Loading branch information
GuiMacielPereira committed Jul 23, 2024
1 parent 54ab628 commit f9c1791
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/pr_workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,11 +35,11 @@ jobs:

- name: Check for analysis files changed
run: |
echo "analysis-changed=$(git diff --name-only ${{ github.event.before }} ${{ github.sha }} | grep -q '^src/' && echo true || echo false)" >> $GITHUB_ENV
echo "analysis-changed=$(git diff --name-only ${{ github.event.before }} ${{ github.sha }} | grep -q -e '^src/' -e '^tests/' && echo true || echo false)" >> $GITHUB_ENV
- name: Check for calibration files changed
run: |
echo "calibration-changed=$(git diff --name-only ${{ github.event.before }} ${{ github.sha }} | grep -q '^tools/calibration_scripts/' && echo true || echo false)" >> $GITHUB_ENV
echo "calibration-changed=$(git diff --name-only ${{ github.event.before }} ${{ github.sha }} | grep -q -e '^tools/calibration_scripts/' -e '^tests/' && echo true || echo false)" >> $GITHUB_ENV
- name: Run mvesuvio Analysis Unit Tests
if: ${{ env.analysis-changed == 'true'}}
Expand Down
1 change: 0 additions & 1 deletion tests/system/analysis/test_analysis.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@
ip_folder=str(Path(handle_config.VESUVIO_PACKAGE_PATH).joinpath("config", "ip_files")),
inputs_file=str(Path(__file__).absolute().parent.parent / "data" / "inputs" / "sample_test.py")
)

ipFilesPath = Path(handle_config.read_config_var("caching.ipfolder"))


Expand Down

0 comments on commit f9c1791

Please sign in to comment.