Avoid creating multiple clients if the NSE is invoked more than once …

[stefanceriu]

…on the same process

This came out of a discussion with @bnjbvr from which I quote:

it's of uttermost importance that there's at most one Client per process ... We put all our context in there, viz. locks, viz. OIDC data, so there needs to be at most one per different account

[github-actions]

	Warnings
⚠️	Please add a changelog.
⚠️	Some of the commits are missing ticket numbers. Please consider squashing all commits that don't have a tracking number.

Generated by 🚫 Danger Swift against 29b9e36

[bnjbvr]

Sweet, thanks!

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 18 lines in your changes missing coverage. Please review.

Project coverage is 78.34%. Comparing base (b94111a) to head (985b51a).
Report is 1 commits behind head on develop.

❗ Current head 985b51a differs from pull request most recent head 29b9e36

Please upload reports for the commit 29b9e36 to get more accurate results.

Files	Patch %	Lines
NSE/Sources/NotificationServiceExtension.swift	0.00%	18 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2944      +/-   ##
===========================================
+ Coverage    78.31%   78.34%   +0.02%     
===========================================
  Files          691      691              
  Lines        54028    54146     +118     
===========================================
+ Hits         42313    42420     +107     
- Misses       11715    11726      +11     

Flag	Coverage Δ
unittests	68.99% <0.00%> (+0.05%)	⬆️

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.

[sonarcloud]

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 SonarCloud