Skip to content

Conversation

@daferya
Copy link
Collaborator

@daferya daferya commented Dec 18, 2024

This pull request includes several changes to the Android test and main source files to improve the testing framework and refactor the navigation graph. The most important changes involve adding a new test class for the chat screen, removing the ModeViewModel from various test and main source files, and updating the navigation graph to include the ChatViewModel.

Improvements to testing framework:

  • Added a new test class ChatsScreenTest in app/src/androidTest/java/com/arygm/quickfix/ui/chat/ChatsScreenTest.kt to test chat screen functionalities.
  • Added log statements in MainActivityTest for better debugging. [1] [2]

Refactoring:

  • Removed ModeViewModel from MessageUserNoModeScreenTest and other test files. [1] [2] [3] [4] [5] [6] [7] [8] [9]
  • Updated AppContentNavGraph to include ChatViewModel and QuickFixViewModel.
  • Updated UserModeNavGraph to remove ModeViewModel. [1] [2]
  • Updated WorkerModeNavGraph to include ChatViewModel and QuickFixViewModel, and replaced MessagesScreen with ChatsScreen. [1] [2] [3] [4]

These changes enhance the testing capabilities and streamline the navigation graph by removing unnecessary view models and adding necessary ones.

@daferya daferya added prio:high This issue or pull request should be handled as soon as possible feature Add a new feature labels Dec 18, 2024
@daferya daferya added this to the Sprint 10 milestone Dec 18, 2024
@daferya daferya self-assigned this Dec 18, 2024
@daferya daferya linked an issue Dec 18, 2024 that may be closed by this pull request
@daferya daferya force-pushed the 255-implement-messages-screen branch 3 times, most recently from d201717 to ae8fa18 Compare December 19, 2024 06:38
@daferya daferya force-pushed the 255-implement-messages-screen branch from ae8fa18 to 9c2aef3 Compare December 19, 2024 06:44
@mohagpt mohagpt force-pushed the 255-implement-messages-screen branch 2 times, most recently from b082c8f to 73a3bd0 Compare December 19, 2024 09:29
@sonarqubecloud
Copy link

@daferya daferya force-pushed the 255-implement-messages-screen branch from 73a3bd0 to 1e550d2 Compare December 19, 2024 22:48
@daferya daferya force-pushed the 255-implement-messages-screen branch from 97f2b53 to 88d6cc1 Compare December 19, 2024 22:56
@daferya daferya force-pushed the 255-implement-messages-screen branch 2 times, most recently from ac3a8dd to fa6f555 Compare December 20, 2024 02:36
@daferya daferya force-pushed the 255-implement-messages-screen branch from fa6f555 to 9442b2f Compare December 20, 2024 04:21
@daferya daferya force-pushed the 255-implement-messages-screen branch from 9f51b50 to e99f7cf Compare December 20, 2024 06:06
@daferya daferya changed the title 255 implement messages screen 255 implement chat screen of the worker and update the accountpicture Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Add a new feature prio:high This issue or pull request should be handled as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

implement messages screen

3 participants