Refactor app config to source, check validity and improve edgecase handling (EXPOSUREAPP-3774,EXPOSUREAPP-3771) (#1623)

* Refactor app config in to source: remote, local and default.
Check validity on local config before requesting one from remote.

* Handle JSON/GSON related edge-case behavior.

* Set local time offset for the default config to 0.

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

* Don't limit the DownloadDiagnosisKeysTask to once a day (EXPOSUREAPP-3771)

* Start DownloadDiagnosisKeysTask only once every hour (EXPOSUREAPP-3771)

* Fix initial state after fresh install (EXPOSUREAPP-3771)

* please ktlint and small fix

* Remove debug notification

* Use task state instead of a dedicated retrieving status indicator

* remove empty line

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>
Co-authored-by: Matthias Urhahn <matthias.urhahn@sap.com>

Co-authored-by: Ralf Gehrer <ralfgehrer@users.noreply.github.com>

Cannot delete tests older than 21 days - DE: Tests die älter als 21 Tage sind, können nicht gelöscht werden (EXPOSUREAPP-2398) (#1586)

* 1. Replaced redundant uiStateState during test result fetch with a NetworkRequestWrapper that is able to represent multiple states. This allows for better request state handling throughout the app
2. Completely removed the SubmissionViewModel
3. Failure to fetch the test should now always display the invalid test card on the home screen

Signed-off-by: Kolya Opahle <k.opahle@sap.com>

* Keep displaying fetching card on server errors as this usually does not indicate any problems with the test itself

Signed-off-by: Kolya Opahle <k.opahle@sap.com>

* Fixing unit tests (mostly just syntax, one test case changed due to new card)

Signed-off-by: Kolya Opahle <k.opahle@sap.com>

* Promoted unknown host exception to server exception

Co-authored-by: chris-cwa <69595386+chris-cwa@users.noreply.github.com>
Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

Merge 1.6.x into 1.7.x

Co-authored-by: Kolya Opahle <k.opahle@sap.com>
Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

# Conflicts:
#	Corona-Warn-App/src/main/java/de/rki/coronawarnapp/nearby/modules/detectiontracker/ExposureDetectionTrackerStorage.kt
#	Corona-Warn-App/src/main/java/de/rki/coronawarnapp/util/flow/HotDataFlow.kt
#	Corona-Warn-App/src/test/java/de/rki/coronawarnapp/util/flow/HotDataFlowTest.kt
#	gradle.properties

Co-authored-by: Ralf Gehrer <ralfgehrer@users.noreply.github.com>

* Make config sharing behavior `SharingStarted.Lazily`. (start once, never release)
Otherwise classes that are just interested in the current config, if there is no other subscriber active,
would launch the `startValueProvider` and pull a config each time.

The ExposureDetectionTracker is such a candidate that would like to use the latest config every ~3min.

While this already no longer happens since "Track config changes" (4d1ecb54),
because there is no a class that is permanently subscribed to `currentConfig`, we should not rely on side-effects,
so let's make the sharing behavior explicit.

* Fix test regression.

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

* Fix HotDataFlow behavior.
* Prevent re-execution of past submissions when the observable goes cold.
* Guard internal value updates with a mutex in case the value update is not as thread-safe as we think it is.

* Fix problematic testcases.

* Update RiskLevelRepository.kt

* Removed comment

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

* Flush the json data after writing it, otherwise if the app process dies, we end up with an empty file.
GSON deserializes this into a null object that looks like a non-null object until it is evaluated some time later.

Force an evaluation via `require(it.size >= 0)` and delete the corrupt data file when catching the exception.

* Show a visible error dialog if we fail to save data.

* Used buffered versions of readers and writer.

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

* Catch cancellation exceptions for coroutines launched on the viewModelScope.

* Use the viewmodel provided launch method to use common handling.

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>
Co-authored-by: chris-cwa <69595386+chris-cwa@users.noreply.github.com>

Change-Id: I64e8b3a17346d3e14e3c7306c89100633afc104e

Co-authored-by: Ralf Gehrer <ralfgehrer@users.noreply.github.com>

* + config changed detector

* tests

* ConfigChangeDetector.kt improvements.
* Fix config collecting scoping
* Move the identifier update into the RiskLevelTask
* Add test coverage

* Logs NO-OPs too.

* Filter emissions early if the value didn't change.

* extracted constant

Co-authored-by: Matthias Urhahn <matthias.urhahn@sap.com>

* Don't rename the APK on `device` flavor builds.
It breaks internal pipelines.

* Better log output.

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

* add TracingStatus check that forces No_Calculation option if exposure logging stopped

* detekt adjustments

* Added test with Rituraj for tracing status when tracing status is inactive

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

* If we revoked any cached keys we need to force an index lookup.
+ Additional tests to check that forced index lookup is decided on correctly.

* Fix merge regressions in tests.

The timeout checker doesn't need to force an app-config refresh, it's fine working with the latest available one. (#1573)

Co-authored-by: Ralf Gehrer <ralfgehrer@users.noreply.github.com>

* Show more specific error message when quota limit is reached and remove "Update ever 24h text".

* Show custom title in error dialog and make stack trace selectable.

* LINTs

* Add a log statement so that in all cases the the base error reporting is made visible.

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

* improved logging for Bakcground Worker

* loggin added in updateRiskLevelScore

Notification Permissions logged and notification sent is loggged

* merge conflict resolution

* uniformity in Log tags

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

* Update from https://github.com/corona-warn-app/cwa-protocol-buffers/commit/9586b423837c598edc76c2ddf08baebcce1b1c36



* Adjust API to new protobuf names.
Update default config.

* Adjust function naming invalid -> revoked.

Co-authored-by: Matthias Urhahn <matthias.urhahn@sap.com>

* Replaced KeyFileDownloader.kt in favor of KeyFileSyncTool.kt.
Supports new logic for more frequent downloads
First draft.

* Change "country" to location, which is more fitting.

* Split sync success tracking into days and hours, track them separately.

* Fix stale location data not being cleaned up.

* Test fragment, first drafts.
Remove 24 hour mode options, no longer used.

* Implemented metered connection check for hourly key download.

* Provide timeout values for download transaction and individual downloads via app config.

* Add clear+download actions, and current network state display to the key download test fragment.

* Complete unit tests for KeyDownloadTool.kt

* Complete unit tests for FlowPreference and DownloadConfigMapper-

* Unit tests for day and hour sync tool.

* Complete tests for KeyPackageSyncTool.kt

* klint & detekt <3

* Fix tags and commented out code (the deletion behavior is covered by `getting completed keys`

* Fix text typo.

* Sync tools need to tell what (if) new packages have been downloaded.

* Interval checks for exposure detection.

* Use the ENFClient, avoid direct access to the calculation tracker.

* Remove defensive check "wanted countries"/"available countries" to reduce server hits.

* CalculationTracker's timeout should come from the AppConfig (androidExposureDetectionParameters.overallTimeoutInSeconds)

* Add test to confirm that we delete stale locations.

* Finished key packages test screen.
Introduced some boilerplate code to make async diffutils easier to use (will help us with the homefragment later on).

* Remove ETag fallback behavior, missing ETag is such an edge case that we should throw.

* Check valid max and min config values.

* Fix linting issues.

* Address PR comments.

* EXPECT_NEW_*_PACKAGES should be based on the package time data, not the download creation timestamp.
The downloads creation timestamp can be subject to race conditions.

* Fix overflow due to time addition with Long.MAX_VALUE

* Clean up tests

* Don't crash the task if we can't get today's hour index.
(Makes timetravel testing difficult, and a real abort reason would be a crash on the day index)

* Display errors (don't crash) when running manual keysync from the test menu.

* Fix last day/hour sync result success state not being updated.

* Change Key download task collision mode from ENQUEUE to SKIP_IF_SIBLING_RUNNING.

* Rename CalculationTracker to ExposureDetectionTracker

* Rename `forceSync` to `forceIndexLookup`.

* Remove outdated comments.

* Spell out viewholder.

* Move `maxExposureDetectionsPerUTCDay == 0` into it's own check.

* Change test menu metered connection button behavior. It's now a button that "fakes" the connection status to metered.
It's now also part of the `TestSettings` and the connection state is faked for the `NetworkStateProvider` class which may be reused.

* Change test menu metered connection button behavior. It's now a button that "fakes" the connection status to metered.
It's now also part of the `TestSettings` and the connection state is faked for the `NetworkStateProvider` class which may be reused.

* Dry sync/error logging call.

* Improve parameter naming, it's target locations, not available locations.
The app config alone determines which locations we try to sync.

Co-authored-by: BMItter <46747780+BMItter@users.noreply.github.com>

* Fix task exceptions not being forwarded to our current error UI elements.

* Clean up old transaction names.

* Use simple class name as error tag.

* Prevent error reporting from triggering in unit tests due to static access.

* Move isUnitTest to CWDebug

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>

* Add default app config as last resort fallback.

* Change config type names and improve logging for HotDataFlow.kt

* Run the app config retrieval on the app scope, otherwise UI unsubscription could cancel the app config retrieval.

* Offer "identifier" property that can identify app configs such that subscribers know when it is considered changed.

* Behavior change AppConfigProvider.kt, there is no caching/replay grace period, once the last subscription is gone, it's  last value is cleared.

* Klint ;)

* Add a unit test specifically for checking the default configs checkum and parsability.

* Fix test breaking on CI.

* Don't check the checksum for the config loaded from assets.

Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>