-
Notifications
You must be signed in to change notification settings - Fork 4
Improve UI test cases #61
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
Conversation
…anagement options
…ling and error management
…ndling and user role validation
… input validation
…ation and error handling
Test Results502 tests +125 479 ✅ +131 17s ⏱️ +7s For more details on these failures, see this check. Results for commit b40ebb8. ± Comparison against base commit f63df46. This pull request removes 20 and adds 144 tests. Note that renamed tests count towards both. |
|
❌ Test or Coverage checks failed! Please check the workflow run for details. |
AimanYosofiUlt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job bro 👍
This pull request introduces comprehensive improvements to the test coverage of the UI components for authentication and project management. It replaces mocked annotations with
mockkobjects, adds new test cases for various scenarios, and ensures better handling of user roles and session states.Authentication Test Enhancements:
LoginUserUITest: Added new test cases to validate successful login, retry logic after failure, and handling of multiple login failures before success. These tests ensure robust behavior for different login scenarios.RegisterAdminUITest: Refactored to usemockkforConsoleIO, added detailed test cases for successful registration, retry logic after failure, and handling multiple registration failures. Enhanced password validation and error handling.Project Management Test Enhancements:
CreateProjectUITest: ReplacedMockKAnnotationswithmockkobjects, added test cases for admin and non-admin users, error handling during project creation, and state addition flow. Improved session management handling viaSessionManager.GetAllProjectsUITest: Added session-based role handling to differentiate behavior for admin and non-admin users. Introduced test cases to verify project display logic and improved mock setup for project data. [1] [2]These changes enhance the reliability and maintainability of the test suite by ensuring comprehensive coverage of edge cases and user role-specific behaviors.