Guillaume lepin test/end to end tests #8
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ewModel/tests/preview - Implement FakeBookingRepository to satisfy BookingRepository interface and provide deterministic demo bookings for tests and previews. - Update MyBookingsViewModel to accept a BookingRepository and userId, use repo.getBookingsByUserId(...) in refresh(), and expose deterministic demo() results for previews/tests. - Update unit and Robolectric tests to construct MyBookingsViewModel with FakeBookingRepository and the required user id; adjust TestHost wrappers to avoid duplicate test tags. - Update MyBookingsScreen preview to use the repository-backed ViewModel (calls refresh() for realistic data). BREAKING CHANGE: MyBookingsViewModel constructor now requires a BookingRepository and a userId. Tests and callers must be updated to pass these arguments.
Add a bottom navigation bar and a top app bar to the main application structure. Create placeholder screens for the home, search, profile and settings sections to allow for testing the navigation flow between them. This setup provides the foundational UI structure for future feature development.
Add a bottom navigation bar and a top app bar to the main application structure. Create placeholder screens for the home, search, profile and settings sections to allow for testing the navigation flow between them. This setup provides the foundational UI structure for future feature development. # Conflicts: # app/src/androidTest/java/com/android/sample/navigation/NavigationTestsWithPlaceHolderScreens.kt # app/src/main/java/com/android/sample/ui/components/TopAppBar.kt # app/src/main/java/com/android/sample/ui/navigation/NavGraph.kt # app/src/main/java/com/android/sample/ui/screens/HomePlaceholder.kt # app/src/main/java/com/android/sample/ui/screens/ProfilePlaceholder.kt # app/src/main/java/com/android/sample/ui/screens/SettingsPlaceholder.kt # app/src/main/java/com/android/sample/ui/screens/SkillsPlaceholder.kt
Delete generated test files that were failing due to referencing removed composable components. The tests were checking for UI elements from the original project template that are no longer present in the application after implementing the custom navigation framework. This eliminates false CI failures and removes outdated test dependencies on template code that has been replaced.
- Fix navigation behavior to match PR review requirements - Add code comments to clarify key implementation details - Implement test coverage for all new screen functionality - Include secondary screen navigation in test suite
Create a GitHub Actions workflow that generates APK files on demand through manual triggering. This allows team members to build release artifacts without going through the full CI pipeline, useful for testing and demonstration purposes. The workflow produces signed APKs that can be directly installed on test devices, streamlining the development and QA processes.
Temporarily add push trigger to the APK generation workflow to enable testing before merging into main. This allows the workflow to execute on branch push, making it visible in GitHub Actions for verification and debugging purposes. The push trigger will be removed once the workflow is validated and ready for production use with manual triggers only.
… from base64 before restoring in APK generation workflow
Remove the push trigger that was temporarily added for testing the APK generation workflow. The workflow now only supports manual triggering as intended for production use. This change follows successful validation of the workflow functionality during testing. The workflow is ready for use in the main branch and can be triggered manually through GitHub Actions when APK builds are needed.
-Preparing the Listing structure to associate with the new Booking structure
-Enhancing the strcuture of the data type to implement complex rating logic -Can rate tutors & students and proposal & requests with this structure
…st files - Add unit test for MyBookingsViewModel.refresh() to assert mapping of Booking -> BookingCardUi (duration, price label, date format, id/tutor mapping). - Merge BookingCard compose interaction test into MyBookingsRobolectricTest to verify rendering and details button behavior without creating new test files. - Expand android instrumentation/compose tests in MyBookingsTest to cover empty state + integration scenarios using .
…db-repository-logic' into NedenSinir-feature/db-repository-logic
…itory-logic Firestore repositories for data types
…st authentication
…ening-fix Fixx app not working by adding repository providers in main app and test authentication
…ile-tests Nahuel arthur fix/fix my profile tests
…tore-for-firebase add keystore so that anyone can use the firebase functions
…tion-tests Bjlpedersen test/fix navigation tests
…/end-to-end-tests' into GuillaumeLepin-test/end-to-end-tests
- Modified existing end-to-end tests to better reflect current app flow - Adjusted test logic to handle missing text fields and updated navigation behavior - Improved test stability and coverage for key user interactions
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.