Skip to content

Conversation

@crypto-a
Copy link
Contributor

@crypto-a crypto-a commented Dec 3, 2024

No description provided.

crypto-a and others added 30 commits November 11, 2024 23:36
Modified checkstyle for Package Names to accommodate our naming conventions.

Renamed other packages to abide by this convention.
I also added various classes to support this View (non-exhaustive).
…ad buttons and incorportate clean architecture for the interface adapters
…luding a view button), and added TODO's for Robert

-fixed checkstyle mostly
Added a barebones view for the "send message" user case.
Updated Javadocs for some classes associated with the Send Message use case.

Deleted LoggedInView, as we will be replacing it with ThreadsView.
Changes list type from array to array list in ThreadPane (this can be changed back later, just ot make it compile)
Made it so that ThreadsView compiles.
Added all the necessary Application-level files for the send-message use case.
Implemented the "Return To Threads Button."

Had to change the name of the view name in ThreadsViewModel from "ThreadsViewState" to "threads"
Updated Javadocs for some classes associated with the Send Message use case.

Deleted LoggedInView, as we will be replacing it with ThreadsView.
Added all the necessary Application-level files for the send-message use case.
Implemented the "Return To Threads Button."

Had to change the name of the view name in ThreadsViewModel from "ThreadsViewState" to "threads"
Roppax and others added 28 commits December 2, 2024 15:12
- Implemented multiple test cases to validate GetThreadsInteractor functionality, including success and error scenarios.
- Added mock DAOs (InMemoryThreadDataAccess and InMemoryThreadDataAccessWithError) to simulate server behaviors.
- Fully implemented all methods required by GetThreadsThreadDataAccessInterface in the mock DAOs.
- Enhanced test coverage for switching views (chat and add thread view).
- Ensured all interface methods are properly handled to prevent compile-time issues.
GetThreads Interactor Test + minor fixes
Modificarions to ChatView(can now scorll to bottom)
Revert "Modificarions to ChatView(can now scorll to bottom)"
… method implementations. Added 2 new test cases for better coverage.
Improved Code Coverage Test Class
@crypto-a crypto-a merged commit 73279c0 into main Dec 3, 2024
1 check failed
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.

8 participants