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

Test fixes, improved Playwright tests #496

Merged
merged 1 commit into from
Mar 23, 2023
Merged

Test fixes, improved Playwright tests #496

merged 1 commit into from
Mar 23, 2023

Conversation

lslezak
Copy link
Contributor

@lslezak lslezak commented Mar 23, 2023 •
edited
Loading

Improved Tests

  • Fixed randomly failing builds when refreshing the repositories in the zypper install call
    • The unused repositories are disabled now for faster refresh
    • Repeat zypper ref several times if it fails (just a workaround 🤔 )
  • Fixed failing integration test, the user configuration has been changed
  • Improved documentation in README.md to describe how to use Playwright locally
  • Added login form autodetection to check whether the user login is needed or not
  • The Playwright tests can be now also used with the webpack development server

@coveralls
Copy link

coveralls commented Mar 23, 2023 •
edited
Loading

Coverage Status

Coverage: 75.56%. Remained the same when pulling a880232 on playwright_fixes into bc900ec on master.

@lslezak lslezak merged commit 84b7090 into master Mar 23, 2023
@lslezak lslezak deleted the playwright_fixes branch March 23, 2023 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@imobachgs imobachgs imobachgs approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@lslezak @coveralls @imobachgs