From 770f804325dbbf07a9d84d989a6d7982e95d293e Mon Sep 17 00:00:00 2001 From: Emily Rockman Date: Thu, 17 Aug 2023 07:20:07 -0500 Subject: [PATCH] Fix test failures (#8432) * fail job when anything fails in previous matrix * tweak wording * PR feedback --- .github/workflows/main.yml | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index 233bebbf743..a1a12d41514 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -227,13 +227,21 @@ jobs: flags: integration integration-report: + if: ${{ always() }} name: Integration Test Suite runs-on: ubuntu-latest needs: integration steps: - - name: "[Notification] Integration test suite passes" + - name: "Integration Tests Failed" + if: ${{ contains(needs.integration.result, 'failure') || contains(needs.integration.result, 'cancelled') }} + # when this is true the next step won't execute run: | - echo "::notice title="Integration test suite passes"" + echo "::notice title='Integration test suite failed'" + exit 1 + + - name: "Integration Tests Passed" + run: | + echo "::notice title='Integration test suite passed'" build: name: build packages