* 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.

* 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.

* Regex updated and new test cases added

* test cases refactored, regex improvement, obsolete tests removed

* Fixed lived data in VM being filtered resulting in issue where it wouldn't be updated in the future

* Removed unused imports

* Delete Project.xml

Removed unnecessary file.

* Forgot to add if statement in observer

* Revert Project.xml

Change-Id: Icbc05a9bc4c2d638335e918ece24bf772769b4c0

* Added logic for showing notification on risk level change

* Got rid of magic number

* Fixed ktlint issue

* Notification only shows up when risk card goes from red to green

* Showing dialog onResume

* Added static string for preference key and renamed method to better reflect usage

* Whether not weather

* Showing dialog is handled by the VM

* Fixed ktlint error

* Fixed an issue where the dialog would be shown only after navigating back to the home fragment.

* Move decision logic for showing the lowered risk into the viewmodel.

* Make the lowered risk level score event observable.

* Fix wrong init value used for testing.

* Update strings.xml

* Fix test regressions due to static LocalData access.

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

* Refactored DataRetentionHelper class to use injection

* Added ViewModel to SettingsReset

* Add a clear cache method to AppConfigProvider.kt

* Fixed repository live data containing stale data after app reset.

* Moved logic from fragment to view model.

* Adjusted tests to accommodate AppConfigProvider changes

* Reverted back last 2 commits and downgraded ktlint plugin

* Fixed DI issue

* Put the scope in the hands of the VM

* Deleted unused imports

* Removed unused imports from VM

* Use injected dispatcher provider to simplify testing.

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

Retry the reset for v1.6.0+ as the failed reset attempt in 1.5.0 may have consumed the reset attempt already. (#1492)

* Cleaner log and remove superfluos removal, if we replace the entry anyways, we don't have to remove it.

* Only propagate changes if the data actually changed.

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

Change-Id: Ie90db444d051c7d37eadfbe1f1e1e5205804eab1

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

Homescreen Notifications, even though keys were submitted (EXPOSUREAPP-3181, EXPOSUREAPP-3182) (#1418)

* Added additional checks to notification of exposure/test-result to check for finished submission

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

* Small changes to code style

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

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

* Two new progress states for the risk card+details.

* Add unit tests for new card states.

* Flatten UI of risk card to improve layouting performance on slow devices.

* Remove stale strings from other translations.

* UX wants the ... ¯\_(ツ)_/¯

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

* ProtoBuf Update at Wed Oct 21 13:15:13 UTC 2020 #1448

* Switch to centralized ProtoBuf format (EXPOSUREAPP-3404) (#1455)

* Switch to centralized ProtoBuf format and fix imports (EXPOSUREAPP-3404)

* Fix ktlint issues

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

* Add a `CalcluationTracker` class that can tell us about the ENF's calcluations.

* Introduces new reactive data provider "HotData" that allows lazy init and safe updates.

* Add calculation time out enforcer (60min), check every 5min.

* Show `isRefreshing` when we are either downloading keys or the ENF is still calculating.

* Rename "token" to "identifier", we'll likely keep using this class for ENF Window Mode,
which has no more "tokens".

* Remove "state", we can only know whether it's running and that can be based on the `finishedAt` timestamp.
Also improve test readability a bit and added more edge cases when checking timeouts.

* Additional test cases for ENF calculation edge cases

* We are fine with tracking the last 5 calculations.

* Exclude timeouts from `latestFinishedCalculation`

* HotData should be named HotDataFlow

* Clean up coroutine/flow packages.

* Remove additional combineTransform instances, to prevent accidental casting errors in the future.

* We should only forward calls to the provider and the calculation tracker if the list of key files is non-empty.

* Add unit test to check for GSON data class restoration behavior with transient fields.
GSON sets it to false, and does not eval the properties to set them.

* Lints, Lints, Lints

* Only check the calculation status of the newest submission.
The chance for overlapping calculations is rare with batched key submission,
and if gives us a lower chance of actually being affected by timeouts.

* Reduce timeout for tracked calculations to 15 minutes after discussion with Maximilian.
In worst case scenarios no calculation exceeded 7 minutes.

* Reduce timeout check interval to 3 minutes, due to lowered overall timeout limit.

* Add additional test case that checks that a late result, past timeout, overwrites the timeout.

* Create `BaseGson` instance within the DI graph use it.
Allows use to later set global settings (pretty print for testers?),
or hook up custom serializers app-wide.

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

* Added new card to show redeemed tests on the home screen

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

* Fixed linting and unit tests

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

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

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

Typo.

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

* Deadman notification text for translation

* Fixed typo

* Removed background priority settings from the settings ui

* Revert "Removed background priority settings from the settings ui"

This reverts commit c0447a1a52da6a4bc33aac01ceb28afce049cf98.

* Removed background priority switch from settings ui and replaced with intent to open settings,
also replaced in notification on first launch

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

* Card is now always visible and changed text

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

* Revert "Card is now always visible and changed text"

This reverts commit cae7c0fcc4575c67fb1a3d12b8a304a2416543f0.

* Added new german strings to status card and made it permanently visible

* Removed unnecessary constraint layout from xml file

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

* Removed additional unneeded constraint layout

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

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

* Fix text issues for low risk with 0/1+ encounters (EXPOSUREAPP-3193)

* revert deletion of default string

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

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

* move strings

* added default strings

* fixed comment indention

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

* Build.gradle signing fix

* Test config.yml

* Clean up

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

Version bump

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

* Increase transaction timeout to 8 minutes.

* Fix timeout variable test.

* Adjust code comment to mention the 10min limit.

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

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

Fix automatic data reset on for users affected by encryption issues (EXPOSUREAPP-3313, EXPOSUREAPP-3335) (#1433)

* I overlooked that `KeyException` is a subclass of `GeneralSecurityException`.
This causes us to continue with `KeyException` which then fails the message matching test.
As the underlying Tink code throws the GeneralSecurityException first, we check the type of the last exception.

* Add additional specific test cases for nested exceptions.

* The dialog shouldn't automatically dismiss when the user clicks details.
They may not have read the dialog message completely or want to re-read it.

* Added conditions for apk signing

* Update testers flavour

* Config fix

* Disable assemble job skip

* Create separate workflow

* Fix config.yml

* Clean up

* Introduced `TracingCardState` and `TracingDetailsState`.
Removed all direct viewmodel/livedata access from all nested layouts for `TracingViewModel`.
Removed `FormatterRiskHelper`.
Introduced logic to reactively update the UI state if bluetooth, location or tracing state changes.

TODO:
* Finish tests in TracingCardStateTest.kt and TracingDetailsStateTest.kt
* Finish TracingDetailsViewModel.kt and TracingCardViewModel.kt
* Remove obsolete dependencies on TracingViewModel.kt and singletons RiskLevelRepository.kt, TracingRepository.kt

* Breakthrough, cards update again.

* Further cleanup of TracingViewModel.kt, TracingRepository.kt
Still not quite there yet, too many calls going into every direction, back and forth, instead of just top->down, but we are getting there.
We want data flow from business logic code to the UI, keep the UI KISS.

* Update TracingCardState tests to account for changes

* More refactoring, TracingViewModel.kt is now finally gone.

* Fix risk details card, we reuse the cardviewmodel but need to overwrite the `showDetails` flag depending from where it's used.

* Refactored several properties (notification related) out of SettingsViewModel/Repository

* Fix injection

* Refactoring submission related databinding code on the home fragment

* Make TracingRepository.kt injectable and change viewmodel scoping for refresh calls to app scope.

* Remove additional formatter calls from submission cards on home screen.

* Refactored additional SubmissionViewModel logic into SubmissionRepository on the Homescreen.
First UI Test is now passing.

Simplified ViewModel instantiation related code.
Prepared HomeFragment for smoother UI state changes, less reloading.

* Improved home,tracing-details and tracing-settings loading behavior, less flickering.
Fixed LiveData value assignment on background-thread.

* New reactive provider for Bluetooth and Location status.
Cleaned up ConnectivityHelper.kt partially.

* Implement remaining tests, batch 1.

* Implement remaining tests, batch 2.

* Fix lint issues.

* Removed a few codesmells and code duplication.
Introduced Flow.shareLatest(...)

* Removed unused parameters.

* LINT fixes.

* Fix unit test failing due to missing mocks.

* Address PR comments.

* Improve UI loading states, remove flickering.

* Remove log spam due to polling values, add extra abort condition that checks for cancellation.

Co-authored-by: ralfgehrer <mail@ralfgehrer.com>

* converted SubmissionTanFragment to assisted inject

* removed unused import

* Refactored submission dispatcher

* lint fixes

* removed unnecessary companion  object

* Submission Test Result refactored

* Submission Contact refactored

* Submission Done Refactored and lint fixes

* more lint fixes

* Submission Intro refactored

* interface fix

* Submission QR Codde Scan refactored

* SubmissionResultPositiveOtherWarning refactored

* Adapted Navigation style to use SubmissionNavigationEvent - aligns with Onboarding refactoring.

* Submission Symptom Introduction refactored and fragment restructure

* SymptomCalendar refactor and ktlint

* ktlint

* deleted unnecessary extra line

* ktlint

* removed directions declaration that was causing issues

* navigation fix

* Move companion object to end of class

* Migrated submission modules to SubmissionActivityModule, and included in MainActivityModule

* Use nav util instead of findNavController + ktlint fixes

* more lint fixes

* refactor name of SubmissionActivityModule to SubmissionFragmentModule

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