Refacto 2025
Description
- Introduce Stores class which is the class we are mainly going to update to add a new Store backend
- Get rid off a bunch of conditions and start looping on available Stores instead
- Remove Origin class which was redundant with the Source class
- Sanitize the StoreRepository interface
Screenshots
Technical details
Tests
Issues
https://gitlab.e.foundation/e/os/backlog/-/issues/2903
10 commandments of code reviews
Summary by CodeRabbit
-
New Features
- Introduced a new
Storesclass for managing store repositories. - Updated
getHomeScreenDatamethods across multiple repositories to accept a list ofHomeobjects and return a list ofHomeinstead of a map.
- Introduced a new
-
Bug Fixes
- Enhanced error handling in the
getHomeScreenDatamethod ofPlayStoreRepository.
- Enhanced error handling in the
-
Refactor
- Simplified data retrieval logic in
HomeApiImpland related classes by removing unnecessary parameters and methods. - Transitioned several methods in
ApplicationDataManagerand related classes from asynchronous to synchronous execution.
- Simplified data retrieval logic in
-
Tests
- Expanded test coverage for
AppsApifunctionality. - Removed the
HomeApiTestclass, indicating a shift in testing strategy. - Updated dependency management in tests related to
ApplicationDataManager.
- Expanded test coverage for
Edited by Jonathan Klee