Skip to content
This repository has been archived by the owner on Apr 25, 2024. It is now read-only.

Fix e2e test for usermenu #553

Merged
merged 2 commits into from
Jan 15, 2024
Merged

Fix e2e test for usermenu #553

merged 2 commits into from
Jan 15, 2024

Conversation

sebxian
Copy link
Collaborator

@sebxian sebxian commented Jan 15, 2024

This fixes the e2e test for the user menu.

It failed on my system and when investigating, I found that no namespace is created before performing this test. So if performed in isolation, the onboarding page will be rendered and the user menu will not be accessible. Probably the test didn't fail in the past because namespaces from other tests existed.

Signed-off-by: Sebastian Armbrust <[email protected]>
@sebxian sebxian changed the title Create namespace for test Fix e2e test for usermenu Jan 15, 2024
@sebxian sebxian requested a review from stefan-kracht January 15, 2024 10:07
Copy link
Member

@stefan-kracht stefan-kracht left a comment

Choose a reason for hiding this comment

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

I think I did that on purpose because I thought there is no isolated namespace needed for that test, but I did not think about having no namespaces at all.

@sebxian sebxian merged commit 156ea3c into main Jan 15, 2024
1 check passed
@sebxian sebxian deleted the sebastianarmbrust/e2e-usermenu-fix branch January 15, 2024 11:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants