Skip to content

Conversation

@rudream
Copy link
Contributor

@rudream rudream commented Apr 17, 2025

Purpose

This PR introduces the setup for end-to-end testing in the Web UI using Playwright against a real running Teleport instance and a test for the signup flow.

@rudream rudream added the no-changelog Indicates that a PR does not require a changelog entry label Apr 17, 2025
@rudream rudream requested a review from gzdunek April 17, 2025 00:51
@rudream rudream marked this pull request as ready for review April 23, 2025 04:55
@rudream rudream requested a review from gzdunek April 23, 2025 04:55
@github-actions github-actions bot requested a review from avatus April 23, 2025 04:56
@rudream rudream requested a review from hugoShaka April 23, 2025 18:47
@rudream rudream requested review from gzdunek and hugoShaka May 7, 2025 15:29
@rudream rudream force-pushed the yassine/e2e-3 branch 2 times, most recently from 3a30b54 to 34fb975 Compare May 7, 2025 15:46
Copy link
Contributor

@gzdunek gzdunek left a comment

Choose a reason for hiding this comment

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

The last three comments from the previous batch are marked as resolved, but it doesn't look like they've been addressed. Do you plan to address them?

@rudream rudream requested a review from gzdunek May 9, 2025 12:04
@rudream rudream force-pushed the yassine/e2e-3 branch 2 times, most recently from 7acbff5 to d234075 Compare May 14, 2025 09:49
@rudream rudream requested a review from ravicious as a code owner May 14, 2025 09:49
Copy link
Contributor

@gzdunek gzdunek left a comment

Choose a reason for hiding this comment

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

Left a few final comments.

Copy link
Contributor

@hugoShaka hugoShaka left a comment

Choose a reason for hiding this comment

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

LGTM with minor nits, I only looked at the go bits

@rudream rudream force-pushed the yassine/e2e-3 branch 2 times, most recently from e61f786 to 2f63e2f Compare May 16, 2025 18:23
@rudream rudream enabled auto-merge May 16, 2025 18:26
@rudream rudream added this pull request to the merge queue May 16, 2025
Merged via the queue into master with commit 999533b May 16, 2025
44 checks passed
@rudream rudream deleted the yassine/e2e-3 branch May 16, 2025 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/md ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants