CLD-5770 - Fix smoketests workflow (#23713)

* fix: Fix proper status update since job.status depicts current job status which is always success
* Rename status check for smoketests
* Allow localhost origin for local testing
* Increase instance size for smoketest job, better status check name

---------
Co-authored-by: Antonis Stamatiou <stamatiou.antonis@gmail.com>
Этот коммит содержится в:
mvitale1989
2023-06-13 17:29:36 +02:00
коммит произвёл GitHub
родитель be590a1f0c
Коммит 0d81e5eb40
2 изменённых файлов: 23 добавлений и 6 удалений

28
.github/workflows/e2e-tests-ci.yml поставляемый
Просмотреть файл

@@ -22,7 +22,7 @@ jobs:
with:
repository_full_name: ${{ github.repository }}
commit_sha: ${{ inputs.commit_sha || github.sha }}
context: ci/e2e-tests
context: E2E Tests/smoketests
description: E2E tests for mattermost server app
status: pending
@@ -84,7 +84,7 @@ jobs:
npm run check
smoketests:
runs-on: ubuntu-22.04
runs-on: ubuntu-latest-8-cores
needs:
- cypress-check
- playwright-check
@@ -104,9 +104,9 @@ jobs:
echo "Cypress run completed with $CYPRESS_FAILURES failures"
[ "$CYPRESS_FAILURES" = "0" ]
update-final-status:
update-failure-final-status:
runs-on: ubuntu-22.04
if: always()
if: failure() || cancelled()
needs:
- smoketests
steps:
@@ -116,6 +116,22 @@ jobs:
with:
repository_full_name: ${{ github.repository }}
commit_sha: ${{ inputs.commit_sha || github.sha }}
context: ci/e2e-tests
context: E2E Tests/smoketests
description: E2E tests for mattermost server app
status: ${{ job.status }}
status: failure
update-success-final-status:
runs-on: ubuntu-22.04
if: success()
needs:
- smoketests
steps:
- uses: mattermost/actions/delivery/update-commit-status@main
env:
GITHUB_TOKEN: ${{ github.token }}
with:
repository_full_name: ${{ github.repository }}
commit_sha: ${{ inputs.commit_sha || github.sha }}
context: E2E Tests/smoketests
description: E2E tests for mattermost server app
status: success