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>

* Update EN SDK to 1.7.2 (EXPOSUREAPP-3662)

* Upgrade Work Manager dependencies

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

* Extend NotificationHelper with functionality to schedule repeating notifications.

* Enable cancelation of recurring notifications

* Only show two positive test reminders

* Implement proper positive test result reminder triggers

* please ktlint

* Reset notification on app reset via settings

* Extract positive test result notification logic in an injectable service

* Add tests

* Fix injection issue of NotificationReceiver

* Reset states of the SubmissionRepository on app reset via settings menu

* Fix ktlint issue

* add more logs

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

* Small change to check length of TAN before checking it's validity

* Unit test adjustment to reflect changes.

* Adjusted comparison to use less than operator. Additional tests.

* Seperated tan valid and tan length check fore better readibility. Adjusted tests

* lint fix

* use new isCorrectLength in isTanValidFormat

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

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

* Deadman notification: strings, onetimeworker, time calculation and unit tests

* Periodic deadman notification worker added

* Deadman notification refactor

* Deadman notification refactor
New tests

* Worker refactored to new DI

* Test boilerplate added

* Additional tests

* Clean up

* Added mock providers for deadman notification workers

* Comments and code clean up

* Test clean up

* Schedule work from CoronaWarnApplication

* Variable naming fix

* Switch to foreground state

* Formatting

* Formatting

* Inject Notification Manager Compat

* Change WorkManager initialization, make it impossible to inject the instance without it being initialized.

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

* converted RetrieveDiagnosisKeysTransaction to Task

* cherry picked and solved access to the new task

* fixed background workers

* fixed background workers

* detekt ktlint unit test

* ktlint

* check cancel

* fixed flow

* fixed flow

* fixed flow

* fixed typo

* Make klint happy.

* Little refactoring to address PR comments (DI instead of static access).

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

* Harmonization of the expired QR-Code

* implmentation and refactoring of the asyncRegisterDevice method

* fix detekt pipeline issue

* test failing pipeline fix

* QRCode Exceotion, observevent and timestamp improvements as per code reviews

* removed deregistration logic from Fragment to viewModel

UI Thread protection as this is important logic and should run on ViewModel

* Code conflict resolution

* ktlint fixes

* changin the method deregisterTestFromDevice() to private

* scanStatus SingleLIveEvent and relevant test cases added

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

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