Better handling for editing alias in case of different HS

[Velin92]

We only update the alias if the new alias belongs to the same homeserver or one did not exist yet, otherwise we add it in the list of the alternative ones

[codecov]

Codecov Report

Attention: Patch coverage is 95.04132% with 6 lines in your changes missing coverage. Please review.

Project coverage is 78.68%. Comparing base (48e530f) to head (751cb3e).
Report is 1 commits behind head on develop.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...AddressScreen/EditRoomAddressScreenViewModel.swift	82.35%	6 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3695      +/-   ##
===========================================
- Coverage    78.68%   78.68%   -0.01%     
===========================================
  Files          792      792              
  Lines        67742    67844     +102     
===========================================
+ Hits         53306    53382      +76     
- Misses       14436    14462      +26     

Flag	Coverage Δ
unittests	70.20% <95.04%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[stefanceriu]

Makes sense to me 👍
Would be useful to update the EditRoomAddressScreenViewModelTests.testCorrectMethodsCalledOnSave test as well now.

[Velin92]

Makes sense to me 👍 Would be useful to update the EditRoomAddressScreenViewModelTests.testCorrectMethodsCalledOnSave test as well now.

Just added a bunch of tests with expectation for all the possible cases!

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
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