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: flaky tests #950

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

fix: flaky tests #950

wants to merge 1 commit into from

Conversation

web-mi
Copy link
Contributor

@web-mi web-mi commented Jun 5, 2023

Fixes #

Proposed Changes

@what-the-diff
Copy link

what-the-diff bot commented Jun 5, 2023

PR Summary

  • Added a new test case for default login
    A new test case has been added to ensure the default login is functioning correctly.

  • Optimized existing tests using cypress commands
    Some tests have been updated to use cypress commands rather than direct DOM manipulation for better testing efficiency.

  • Fixed appointment deletion issue
    An issue with appointment deletion has been addressed, ensuring all appointments can be deleted without causing errors in headless mode or CI/CD pipelines.

  • Removed unneeded wait command in messages.cy
    An unnecessary waiting command in messages.cy has been removed to optimize testing.

  • Test case added for session expiration redirection
    A new test checks if the user is redirected to the login page when their session expires during page navigation.

  • Enhanced willReturn command for object property overriding
    The willReturn command has been updated to work with the extend option, allowing for better customization in test cases.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 5, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@web-mi web-mi marked this pull request as ready for review June 5, 2023 16:24
@web-mi web-mi requested review from RunzelRosinchen and CarlosSoares and removed request for CarlosSoares June 5, 2023 16:24
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.

1 participant