Refactor LocalData (EXPOSUREAPP-2850) (#2569)
* Shared preferences migration: SETTINGS DATA (EXPOSUREAPP-5378) (#2468) * Migrate Shared Preferences (SETTINGS) * Code refactoring * Fix tests * Migrate Shared Preferences (SETTINGS) * Cleaning code * Cleaning code * LocalData Migration skeleton (EXPOSUREAPP-5295) (#2509) * EncryptedPreferencesMigration skeleton class * SecurityHelper return nullable SharedPreferences not an exception * Remove Onboarding related Properties from LocalData (EXPOSUREAPP-5375) #2470 * Removed Onboarding related Properties from LocalData Added OnboardingData which provides the FlowPreferences that were defined in LocalData beforehand Currently no migration for these values is implemented Signed-off-by:Kolya Opahle <k.opahle@sap.com> * Turns out MockFlowPreference and MockSharedPreferences were not usable in the device test flavour moved them into testShared Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * OnboardingData -> OnboardingSettings OnboardingSettings: moved to get / set instead of FlowPreferences when subscribing is not needed isOnboarded is now based on onboardingCompletedTimestamp != null instead of set seperately Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Removed Onboarding Preference key strings from all languages Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Added Singleton annotation to OnboardingSettings Added migration code for OnboardingSettings to EncryptedPreferencesMigration Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Migrate Interoperability settings * Remove tracing local data (EXPOSUREAPP-5377) (#2568) * migrate LocalData tracing preferences to TracingSettings * update tests * Remove old code * Update tests * Remove submission local data (EXPOSUREAPP-5379) (#2505) * Moved submission related settings from LocalData to SubmissionSettings Made BackgroundNoise injectable Refactored some of the SubmissionSettings Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Replaced isBeforeNow in BackgroundNoisePeriodicWorker with timeStamper.nowUTC to help with testing Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Fixed MainActivityTest by providing SubmissionSettings as a mockk Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Removed all Preference keys removed from LocalData Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Added migration code for SubmissionSettings to EncryptedPreferencesMigration Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Moved Instant conversion to extension function to satisfy detekt Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Removed lazy from dagger injection in BackgroundWorkSchedulerBase * Cleaned up TestResultDonorTest (Removed Instant.now() calls, fixed shouldBe case where Instant.now() would break the calculation) * Moved toInstantOrNull into TimeAndDateExtensions and used it in a few more places * Fixed a unit test that was added by the merge and fixed a merge diff that was resolved wrongly Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * Format on commit got rid of these imports for some reason Signed-off-by:
Kolya Opahle <k.opahle@sap.com> * EncryptedPreferencesMigration refactoring * DataReset implementation * Drop storage of active tracing time * Cleaning code * Remove TODO * Call migration earlier * Update tests * Delete PreferencesModule * Update EncryptedPreferencesMigration * Fix lint issue * Update EncryptedPreferencesHelper Co-authored-by:
Kolya Opahle <k.opahle@sap.com> Co-authored-by:
harambasicluka <64483219+harambasicluka@users.noreply.github.com> Co-authored-by:
Matthias Urhahn <matthias.urhahn@sap.com>
Showing
- Corona-Warn-App/src/androidTest/java/de/rki/coronawarnapp/ui/main/home/HomeFragmentTest.kt 4 additions, 1 deletion...ava/de/rki/coronawarnapp/ui/main/home/HomeFragmentTest.kt
- Corona-Warn-App/src/androidTest/java/de/rki/coronawarnapp/ui/onboarding/OnboardingTracingFragmentTest.kt 4 additions, 1 deletion...onawarnapp/ui/onboarding/OnboardingTracingFragmentTest.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/test/deltaonboarding/ui/DeltaOnboardingFragmentViewModel.kt 2 additions, 3 deletions...st/deltaonboarding/ui/DeltaOnboardingFragmentViewModel.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/test/submission/ui/SubmissionTestFragmentViewModel.kt 9 additions, 10 deletions...app/test/submission/ui/SubmissionTestFragmentViewModel.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/CoronaWarnApplication.kt 13 additions, 3 deletions...c/main/java/de/rki/coronawarnapp/CoronaWarnApplication.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/RegistrationTokenCensor.kt 5 additions, 3 deletions...nawarnapp/bugreporting/censors/RegistrationTokenCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/datadonation/analytics/Analytics.kt 4 additions, 4 deletions.../de/rki/coronawarnapp/datadonation/analytics/Analytics.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/datadonation/analytics/modules/registeredtest/TestResultDonor.kt 7 additions, 7 deletions...ation/analytics/modules/registeredtest/TestResultDonor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/diagnosiskeys/download/DownloadDiagnosisKeysTask.kt 4 additions, 3 deletions...rnapp/diagnosiskeys/download/DownloadDiagnosisKeysTask.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/main/CWASettings.kt 22 additions, 2 deletions...pp/src/main/java/de/rki/coronawarnapp/main/CWASettings.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/nearby/TracingPermissionHelper.kt 3 additions, 6 deletions...va/de/rki/coronawarnapp/nearby/TracingPermissionHelper.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/nearby/modules/tracing/DefaultTracingStatus.kt 5 additions, 3 deletions...onawarnapp/nearby/modules/tracing/DefaultTracingStatus.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/notification/ShareTestResultNotificationService.kt 8 additions, 7 deletions...arnapp/notification/ShareTestResultNotificationService.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/notification/TestResultAvailableNotificationService.kt 4 additions, 3 deletions...pp/notification/TestResultAvailableNotificationService.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/playbook/BackgroundNoise.kt 8 additions, 19 deletions...ain/java/de/rki/coronawarnapp/playbook/BackgroundNoise.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/RiskLevelChangeDetector.kt 8 additions, 4 deletions...java/de/rki/coronawarnapp/risk/RiskLevelChangeDetector.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/RiskLevelTask.kt 3 additions, 2 deletions.../src/main/java/de/rki/coronawarnapp/risk/RiskLevelTask.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/storage/EncryptedPreferences.kt 0 additions, 8 deletions...java/de/rki/coronawarnapp/storage/EncryptedPreferences.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/storage/LocalData.kt 0 additions, 448 deletions...p/src/main/java/de/rki/coronawarnapp/storage/LocalData.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/storage/OnboardingSettings.kt 40 additions, 0 deletions...n/java/de/rki/coronawarnapp/storage/OnboardingSettings.kt
Loading
Please register or sign in to comment