[Dynamic Dashboard] Reviews card: business logic

[hichamboushaba]

Part of: #11474

Description

This PR adds the business logic part of the reviews card, I wanted to create this PR in this state to keep it small, and to have a chance to discuss how to approach the new cards with the feature flag.
The UI, the navigation to review details and handling of error state will be added on separate PRs.

Testing instructions

1. Open the app.
2. Open the widget editor screen.
3. Confirm that a new card "Most recent reviews" was added, and that it's unselected.
4. Select it.
5. Go back.
6. Confirm reviews are loaded (ignore the appearance).

Images/gif

• I have considered if this change warrants release notes and have added them to RELEASE-NOTES.txt if necessary. Use the "[Internal]" label for non-user-facing changes.

[dangermattic]

	3 Warnings
⚠️	Class DashboardReviewsViewModel is missing tests, but unit-tests-exemption label was set to ignore this.
⚠️	Class Success is missing tests, but unit-tests-exemption label was set to ignore this.
⚠️	PR is not assigned to a milestone.

	1 Message
📖	This PR contains changes to Tracks-related logic. Please ensure (author and reviewer) the following are completed: The tracks events must be validated in the Tracks system. Verify the internal Tracks spreadsheet has also been updated. Please consider registering any new events. The PR must be assigned the category: tracks label.

Generated by 🚫 Danger

[wpmobilebot]

📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.

	App Name	WooCommerce Android
	Flavor	Jalapeno
	Build Type	Debug
	Commit	f873268
	Direct Download	woocommerce-prototype-build-pr11477-f873268.apk

[hichamboushaba]

This logic follows approach 2 on what we decided, as it's quite simple to implement for cards with existing caching, I think it would make sense to use it even if iOS team don't, WDYT?

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 1.69492% with 116 lines in your changes are missing coverage. Please review.

Project coverage is 40.78%. Comparing base (e734909) to head (f873268).

Files	Patch %	Lines
...ommerce/android/ui/reviews/ReviewListRepository.kt	0.00%	41 Missing ⚠️
...droid/ui/dashboard/reviews/DashboardReviewsCard.kt	0.00%	34 Missing ⚠️
.../ui/dashboard/reviews/DashboardReviewsViewModel.kt	0.00%	34 Missing ⚠️
...ommerce/android/ui/dashboard/DashboardContainer.kt	0.00%	3 Missing ⚠️
...ce/android/ui/dashboard/data/DashboardDataStore.kt	0.00%	3 Missing ⚠️
...e/android/ui/dashboard/data/DashboardRepository.kt	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##              trunk   #11477      +/-   ##
============================================
- Coverage     40.84%   40.78%   -0.06%     
  Complexity     5180     5180              
============================================
  Files          1068     1070       +2     
  Lines         62283    62374      +91     
  Branches       8495     8521      +26     
============================================
+ Hits          25437    25438       +1     
- Misses        34559    34649      +90     
  Partials       2287     2287              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[0nko]

When I test the PR, even after logging out and clearing the storage the Reviews card is always shown by default.

[hichamboushaba]

When I test the PR, even after logging out and clearing the storage the Reviews card is always shown by default.

@0nko Yes, this is fixed on the next PR, this happens because the logic of adding only M1 cards by default wasn't agreed on, and it wasn't added to this PR.

[0nko]

Okay, I just mentioned that because the test instructions say Confirm that a new card "Most recent reviews" was added, and that it's unselected.

[0nko]