fix: refactoring self user fetch [#WPB-15190]

[sbakhtiarov]

WPB-15190 [Android] race condition in getKnownUser causes multible requests to update

https://wearezeta.atlassian.net/browse/WPB-15190

What's new in this PR?

Issues

1. `observeSelfUser' is observing metadata and updating it internally. This caused multiple unnecessary network requests to fetch user.
2. Self user is requested multiple times on app start and on opening conversation. This causes multiple unnecessary network calls when user validity timeout expires and all requests try to update it simultaneously.

Solutions

1. Refactor observeSelfUser function to decouple it from userId fetch.
2. Create separate use cases for getting and observing self user.
3. Use deferred result for making only one network call to fetch user and suspend parallel requests.

[github-actions]

Test Results

3 401 tests  ±0   3 293 ✅ ±0   5m 35s ⏱️ -13s
  584 suites ±0     108 💤 ±0 
  584 files   ±0       0 ❌ ±0 

Results for commit f1b3ef1. ± Comparison against base commit b304f30.

♻️ This comment has been updated with latest results.

[github-actions]

Bencher Report

Branch	fix/self-user-refactoring
Testbed	ubuntu-latest

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

• Latency (nanoseconds (ns))

Click here to create a new Threshold
For more information, see the Threshold documentation.
To only post results if a Threshold exists, set the --ci-only-thresholds CLI flag.

Click to view all benchmark results

Benchmark	Latency	microseconds (µs)
com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInFiles	📈 view plot ⚠️ NO THRESHOLD	680.60
com.wire.kalium.benchmarks.logic.CoreLogicBenchmark.createObjectInMemory	📈 view plot ⚠️ NO THRESHOLD	404,007.65
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.messageInsertionBenchmark	📈 view plot ⚠️ NO THRESHOLD	1,332,864.91
com.wire.kalium.benchmarks.persistence.MessagesNoPragmaTuneBenchmark.queryMessagesBenchmark	📈 view plot ⚠️ NO THRESHOLD	21,771.99

🐰 View full continuous benchmarking report in Bencher

[datadog-wireapp]

Datadog Report

Branch report: fix/self-user-refactoring
Commit report: 0afc69b
Test service: kalium-jvm

✅ 0 Failed, 3293 Passed, 108 Skipped, 1m 2.78s Total Time

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 36.20690% with 37 lines in your changes missing coverage. Please review.

Project coverage is 54.44%. Comparing base (b304f30) to head (f1b3ef1).

Files with missing lines	Patch %	Lines
.../com/wire/kalium/logic/data/user/UserRepository.kt	48.83%	16 Missing and 6 partials ⚠️
...alium/logic/feature/user/ObserveSelfUserUseCase.kt	0.00%	7 Missing ⚠️
...lin/com/wire/kalium/persistence/dao/UserDAOImpl.kt	0.00%	5 Missing ⚠️
...re/kalium/logic/feature/user/GetSelfUserUseCase.kt	0.00%	2 Missing ⚠️
...in/com/wire/kalium/logic/feature/user/UserScope.kt	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3229      +/-   ##
===========================================
- Coverage    54.48%   54.44%   -0.04%     
===========================================
  Files         1269     1270       +1     
  Lines        36958    36984      +26     
  Branches      3745     3753       +8     
===========================================
+ Hits         20135    20136       +1     
- Misses       15410    15430      +20     
- Partials      1413     1418       +5     

Files with missing lines	Coverage Δ
.../kotlin/com/wire/kalium/persistence/dao/UserDAO.kt	96.33% <ø> (ø)
...in/com/wire/kalium/logic/feature/user/UserScope.kt	0.00% <0.00%> (ø)
...re/kalium/logic/feature/user/GetSelfUserUseCase.kt	0.00% <0.00%> (ø)
...lin/com/wire/kalium/persistence/dao/UserDAOImpl.kt	51.29% <0.00%> (-1.18%)	⬇️
...alium/logic/feature/user/ObserveSelfUserUseCase.kt	0.00% <0.00%> (ø)
.../com/wire/kalium/logic/data/user/UserRepository.kt	66.28% <48.83%> (-1.07%)	⬇️

... and 3 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b304f30...f1b3ef1. Read the comment docs.