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.

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

Fix broken iFrame e2e test #7359

Merged
merged 1 commit into from
Sep 27, 2021
Merged

Fix broken iFrame e2e test #7359

merged 1 commit into from
Sep 27, 2021

Conversation

christian-bromann
Copy link
Member

This patch b93e7f4 disabled an e2e test that seem to start failing for unknown reasons. Running the test locally seems fine. I suggest instead of using a 3rd party page to replace it with an own blank page that allows us to verify that switching to the iframe was successul.

@christian-bromann christian-bromann added help wanted Issues that are free to take by anyone interested good first pick a reasonable task to start getting familiar with the code base labels Aug 30, 2021
@christian-bromann christian-bromann changed the base branch from master to main September 27, 2021 10:16
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 27, 2021

CLA Signed

The committers are authorized under a signed CLA.

@christian-bromann christian-bromann added PR: Internal 🏠 PRs that contain changes internals and removed help wanted Issues that are free to take by anyone interested good first pick a reasonable task to start getting familiar with the code base labels Sep 27, 2021
Copy link
Member

@erwinheitzman erwinheitzman left a comment

Choose a reason for hiding this comment

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

Nice going Christian

@christian-bromann christian-bromann merged commit 6ee2eff into main Sep 27, 2021
@christian-bromann christian-bromann deleted the cb-fix-e2e-test branch September 27, 2021 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Internal 🏠 PRs that contain changes internals
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants