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

EES-5742 Fix UI tests #5561

Merged
merged 1 commit into from
Jan 28, 2025
Merged

EES-5742 Fix UI tests #5561

merged 1 commit into from
Jan 28, 2025

Conversation

mmyoungman
Copy link
Collaborator

@mmyoungman mmyoungman commented Jan 28, 2025

This PR changes the headless argument we use when opening google-chrome in the UI tests to not use "old".

We did this because we started seeing failures like this:

2025-01-28T10:26:13.8418368Z UI Tests.Glossary Page                                                        
2025-01-28T10:26:13.8418684Z ==============================================================================
2025-01-28T10:26:13.8418983Z Navigate to glossary page                                             | FAIL |
2025-01-28T10:26:13.8419220Z Parent suite setup failed:
2025-01-28T10:26:13.8419591Z SessionNotCreatedException: Message: session not created: probably user data directory is already in use, please specify a unique value for --user-data-dir argument, or don't use --user-data-dir

It seems like newer version of google-chrome and chromedriver have broken "headless=old".

@benoutram benoutram self-requested a review January 28, 2025 12:00
@mmyoungman mmyoungman merged commit 6bcde6d into dev Jan 28, 2025
10 checks passed
@mmyoungman mmyoungman deleted the EES-5742 branch January 28, 2025 12:10
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