Skip to content

[SDPTA-1525] gha pw#81

Draft
klaukl wants to merge 10 commits intomainfrom
SDPTA-1525-gha-pw
Draft

[SDPTA-1525] gha pw#81
klaukl wants to merge 10 commits intomainfrom
SDPTA-1525-gha-pw

Conversation

@klaukl
Copy link
Copy Markdown
Contributor

@klaukl klaukl commented Apr 17, 2026

No description provided.

@klaukl
Copy link
Copy Markdown
Contributor Author

klaukl commented Apr 17, 2026

image Max failures example

@klaukl
Copy link
Copy Markdown
Contributor Author

klaukl commented Apr 17, 2026

Comment thread .github/workflows/test_e2e_be.yml Outdated
TESTID: ${{ inputs.test_id }}
NIGHTWATCH_HEADLESS: true
NIGHTWATCH_OUTPUT: true
# PW Settings
Copy link
Copy Markdown
Contributor Author

@klaukl klaukl Apr 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tim-yao Do we want to make parallel / retry take env vars in case want to change in CI or just leave it hardcoded.
I.e. env:
TEST_WORKERS: 5
TEST_MAX_FAILURES: 5

Otherwise leave it as input but I think we said no need

I will be removing hardcode
     --workers=6 \
      --max-failures=10

@klaukl klaukl changed the title Sdpta 1525 gha pw [SDPTA-1525] gha pw Apr 20, 2026
@klaukl klaukl requested a review from tim-yao April 22, 2026 03:56
name: Run Backend tests
run: |
npm run e2e:backend -- \
--workers=5 \
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will remove this after PW max retry merge

Comment on lines +86 to +87
NIGHTWATCH_HEADLESS: true
NIGHTWATCH_OUTPUT: true
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can be removed

Comment on lines +83 to +84
BROWSERSTACK_USER: ${{ secrets.TEST_BROWSERSTACK_USER }}
BROWSERSTACK_KEY: ${{ secrets.TEST_BROWSERSTACK_KEY }}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can be removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants