Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[e2e][pipeline] Add support for e2e tests on pipeline #1067

Open
wants to merge 74 commits into
base: main
Choose a base branch
from
Open

[e2e][pipeline] Add support for e2e tests on pipeline #1067

wants to merge 74 commits into from

Conversation

tanner-ricks
Copy link
Contributor

@tanner-ricks tanner-ricks commented Nov 19, 2024 •
edited
Loading

Adding playwright e2e test runs to the pipeline

Changes

  • Added new test job to run the playwright end-to-end tests from the pipeline
  • Updated playwright reporting to produce more types of output
  • Added steps to save the artifacts of the test runs including the docker logs and playwright results
  • Found and fixed test failures that had been introduced since starting this work

How to test this PR

  1. Read the notes
  2. Verify that the playwright job ran and produced artifacts

Notes

  • The TSC linting failure is unrelated to this work

@tanner-ricks
Testing multi checkouts
2954fd3
@tanner-ricks
calling only playwright test
b6111c6
@tanner-ricks
calling only playwright test
84ab5e1
@tanner-ricks
calling only playwright test
69717ec
@tanner-ricks
testing checkouts
ed9422d
@tanner-ricks
testing checkouts
d580dd9
@tanner-ricks
testing checkouts
5e5a22c
@tanner-ricks
testing checkouts
35c71d5
@tanner-ricks
testing checkouts
4ea8fd7
@tanner-ricks
testing checkouts
895fd4a
@tanner-ricks
testing checkouts
f0f875d
@tanner-ricks
testing checkouts
9cf6635
@tanner-ricks
testing checkouts
3bda8b9
@tanner-ricks
testing starting docker
c4a7001
@tanner-ricks
testing docker builds
517c75d
@tanner-ricks
testing docker builds
55201d1
@tanner-ricks
testing docker builds
051e5b4
@tanner-ricks
checking registry
fed54fc
@tanner-ricks
checking registry
868c081
@tanner-ricks
checking registry
bbf3a8b
@tanner-ricks
checking registry
a5d7c01
@tanner-ricks
Attempt playwright run
783e0ac
@tanner-ricks
Attempt playwright run
b334e1a
@tanner-ricks
Testing Playwright
8f97762
@tanner-ricks
Testing Playwright
e618590
@tanner-ricks
Testing Playwright
583b4ff
@tanner-ricks
Testing Playwright
0541498
@tanner-ricks
Testing Playwright
0d1c7e7
@tanner-ricks
Testing Playwright
6b6ac14
@tanner-ricks
Checking Playwright
fc16579
@tanner-ricks tanner-ricks changed the title Draft: [e2e][pipeline] Add support for e2e tests on pipeline [e2e][pipeline] Add support for e2e tests on pipeline Nov 21, 2024
@tanner-ricks tanner-ricks marked this pull request as ready for review November 21, 2024 22:21
This was linked to issues Nov 21, 2024
[e2e][pipeline] Add support for e2e tests on pipeline #1064
Open
[e2e] Fix cleanup api calls in e2e tests #1063
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@billhimmelsbach billhimmelsbach Awaiting requested review from billhimmelsbach billhimmelsbach is a code owner

@meissadia meissadia Awaiting requested review from meissadia meissadia is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[e2e][pipeline] Add support for e2e tests on pipeline [e2e] Fix cleanup api calls in e2e tests
1 participant
@tanner-ricks