Skip to content

Commit

Permalink
Tidy up CI test changes.
Browse files Browse the repository at this point in the history
Perserve try-catch logic since that seems to have worked well. The TestResults directory is no longer needed.
  • Loading branch information
Tkael authored and bcthund committed Jun 26, 2024
1 parent 36cd89e commit 8288219
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
9 changes: 8 additions & 1 deletion .github/actions/Test/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,12 @@ runs:
run: |
$runTestCmd = "${{ inputs.target-path }} ${{ inputs.target-args }}"
ECHO Invoking... $runTestCmd
Invoke-Expression $runTestCmd
# Wrap the invokation so that we can act on its success or failure
try {
Invoke-Expression $runTestCmd
} catch {
# seems to have failed
ECHO "::error::Tests failed - " + $_.Exception.Message
EXIT 1
}
shell: pwsh
2 changes: 0 additions & 2 deletions .github/workflows/ContinuousIntegration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,6 @@ jobs:
# Perform unit testing
- name: Add VSTest.console.exe to PATH
uses: darenm/Setup-VSTest@d9a5dffa3f11d9c27ec42eb69515d3aaeaad9ef8
- name: Make a TestResults directory
run: mkdir -p ./TestResults
- name: Test our solution
uses: ./.github/actions/Test
with:
Expand Down

0 comments on commit 8288219

Please sign in to comment.