Read user avatar from cache

[bmarty]

Type of change

• Feature
• Bugfix
• Technical
• Other :

Content

Ensure the user avatar in the room list is rendered even if there is no network. The avatar will also be rendered faster in case of slow network, since it's now read from cache.

In this PR the user profile will be read from cache, previously 2 network requests was trigger on in received Push. This will speed up the display of the notification and reduce the network request.

Motivation and context

Fast UI

Closes #2488

Screenshots / GIFs

Tests

• Login to an account with a user which has an avatar: the avatar is rendered in the room list at top start.
• Kill the app
• enter airplane mode, so the device has no way to reach the homeserver
• start the app
• observe that the user avatar is rendered.

Previously the avatar was not rendered in this case.

Tested devices

• Physical
• Emulator
• OS version(s):

Checklist

• Changes have been tested on an Android device or Android emulator with API 23
• UI change has been tested on both light and dark themes
• Accessibility has been taken into account. See https://github.com/element-hq/element-x-android/blob/develop/CONTRIBUTING.md#accessibility
• Pull request is based on the develop branch
• Pull request includes a new file under ./changelog.d. See https://github.com/element-hq/element-x-android/blob/develop/CONTRIBUTING.md#changelog
• Pull request includes screenshots or videos if containing UI changes
• Pull request includes a sign off
• You've made a self review of your PR

[github-actions]

📱 Scan the QR code below to install the build (arm64 only) for this PR.

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/NtyUaZ

[jmartinesp]

Thanks! The code looks good and the issue is fixed 👍 .

[codecov]

Codecov Report

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

Project coverage is 73.12%. Comparing base (5169ed3) to head (a51855a).
Report is 5 commits behind head on develop.

Files	Patch %	Lines
.../notifications/DefaultNotificationDrawerManager.kt	14.28%	11 Missing and 1 partial ⚠️
...tures/preferences/impl/root/PreferencesRootView.kt	50.00%	1 Missing ⚠️
.../android/libraries/matrix/test/FakeMatrixClient.kt	87.50%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2565      +/-   ##
===========================================
- Coverage    73.14%   73.12%   -0.02%     
===========================================
  Files         1408     1407       -1     
  Lines        34103    34092      -11     
  Branches      6620     6617       -3     
===========================================
- Hits         24945    24931      -14     
- Misses        5698     5703       +5     
+ Partials      3460     3458       -2     

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

[bmarty]

a51855a should fix the coverage drop.

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud