feat(neon_framework): add account_repository

[Leptopoda]

Finally a end to end unit tested login flow.

1. I did not add a separate conventional commit type for the account_repository. As it is mostly going to be used by the framework itself, I feel like neon_framework is good enough. I don't want too many types (open for discussions)
2. This is still a draft, as I was not able to get all tests to pass. Especially the widget tests of all pages have issues with the widget finder. I have no idea what is causing this and will continue to look into it.

[provokateurin]

I only reviewed the account_repository so far.
I'll review the neon_framework changes when the existing comments have been sorted out.
IMO you should add a verify for all the API calls that are made in the tests.

[Leptopoda]

I've dropped the neon_framework changes for easier reviewability.

[Leptopoda]

IMO you should add a verify for all the API calls that are made in the tests.

Thanks for mentioning. I've added them everywhere.

[Leptopoda]

Please review my latest changes.
I added a testing library so our createModel can be reused by other tests

[provokateurin]

In many places you are using await expectLater(), but it is unnecessary because you already match using completion().

[Leptopoda]

In many places you are using await expectLater(), but it is unnecessary because you already match using completion().

No this will not work as the verify calls would be invoked before the actual call completes.

For the [completes] and [completion] matchers, as well as [throwsA] and related matchers when they're matched against a [Future], the returned future completes when the matched future completes.

[provokateurin]

Oh but then there are some places were you use completion() without await expectLater()

[codecov]

Codecov Report

Attention: Patch coverage is 98.93238% with 3 lines in your changes missing coverage. Please review.

Project coverage is 26.40%. Comparing base (5bd82e1) to head (be66e4b).
Report is 6 commits behind head on main.

Files	Patch %	Lines
...ackages/neon_framework/lib/src/blocs/accounts.dart	0.00%	2 Missing ⚠️
...nt_repository/lib/src/testing/testing_account.dart	87.50%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2403      +/-   ##
==========================================
+ Coverage   26.32%   26.40%   +0.08%     
==========================================
  Files         362      370       +8     
  Lines      136541   136662     +121     
==========================================
+ Hits        35940    36086     +146     
+ Misses     100601   100576      -25     

Flag	Coverage Δ		*Carryforward flag
account_repository	99.64% <99.64%> (?)
cookie_store	100.00% <ø> (ø)		Carriedforward from 5bd82e1
dashboard_app	96.05% <ø> (ø)
dynamite	31.08% <ø> (ø)		Carriedforward from 5bd82e1
dynamite_end_to_end_test	61.84% <ø> (ø)		Carriedforward from 5bd82e1
dynamite_runtime	85.40% <ø> (ø)		Carriedforward from 5bd82e1
neon_dashboard	96.05% <ø> (ø)		Carriedforward from 5bd82e1
neon_framework	43.56% <33.33%> (-1.76%)	⬇️
neon_http_client	93.28% <ø> (ø)		Carriedforward from 5bd82e1
neon_notifications	100.00% <ø> (ø)		Carriedforward from 5bd82e1
neon_talk	99.45% <ø> (ø)		Carriedforward from 5bd82e1
nextcloud	22.34% <ø> (ø)		Carriedforward from 5bd82e1
notifications_app	100.00% <ø> (ø)
sort_box	90.90% <ø> (ø)		Carriedforward from 5bd82e1
talk_app	99.09% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

Files	Coverage Δ
...lib/src/settings/utils/settings_export_helper.dart	100.00% <100.00%> (ø)
...account_repository/lib/src/account_repository.dart	100.00% <100.00%> (ø)
...es/account_repository/lib/src/account_storage.dart	100.00% <100.00%> (ø)
...account_repository/lib/src/models/credentials.dart	100.00% <100.00%> (ø)
...count_repository/lib/src/models/credentials.g.dart	100.00% <100.00%> (ø)
...count_repository/lib/src/models/login_qr_code.dart	100.00% <100.00%> (ø)
...account_repository/lib/src/models/serializers.dart	100.00% <100.00%> (ø)
...epository/lib/src/testing/testing_credentials.dart	100.00% <100.00%> (ø)
...epository/lib/src/utils/authentication_client.dart	100.00% <100.00%> (ø)
..._repository/lib/src/utils/http_client_builder.dart	100.00% <100.00%> (ø)
... and 2 more

... and 2 files with indirect coverage changes