Improved app config handling (Identifier, local fallback) (DEV) (#1553)
* 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>
Showing
- Corona-Warn-App/build.gradle 2 additions, 0 deletionsCorona-Warn-App/build.gradle
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/test/appconfig/ui/AppConfigTestFragment.kt 1 addition, 1 deletion.../coronawarnapp/test/appconfig/ui/AppConfigTestFragment.kt
- Corona-Warn-App/src/main/assets/default_app_config.bin 0 additions, 0 deletionsCorona-Warn-App/src/main/assets/default_app_config.bin
- Corona-Warn-App/src/main/assets/default_app_config.sha256 1 addition, 0 deletionsCorona-Warn-App/src/main/assets/default_app_config.sha256
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/AppConfigProvider.kt 24 additions, 18 deletions.../java/de/rki/coronawarnapp/appconfig/AppConfigProvider.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/AppConfigSource.kt 16 additions, 4 deletions...in/java/de/rki/coronawarnapp/appconfig/AppConfigSource.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/ConfigData.kt 26 additions, 3 deletions...rc/main/java/de/rki/coronawarnapp/appconfig/ConfigData.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/DefaultConfigData.kt 2 additions, 1 deletion.../java/de/rki/coronawarnapp/appconfig/DefaultConfigData.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/download/AppConfigServer.kt 9 additions, 3 deletions...e/rki/coronawarnapp/appconfig/download/AppConfigServer.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/download/AppConfigStorage.kt 2 additions, 1 deletion.../rki/coronawarnapp/appconfig/download/AppConfigStorage.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/download/ConfigDownload.kt 1 addition, 0 deletions...de/rki/coronawarnapp/appconfig/download/ConfigDownload.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/appconfig/download/DefaultAppConfigSource.kt 16 additions, 0 deletions...oronawarnapp/appconfig/download/DefaultAppConfigSource.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/util/flow/HotDataFlow.kt 1 addition, 8 deletions...c/main/java/de/rki/coronawarnapp/util/flow/HotDataFlow.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/util/retrofit/RetrofitExtensions.kt 5 additions, 0 deletions.../de/rki/coronawarnapp/util/retrofit/RetrofitExtensions.kt
- Corona-Warn-App/src/test/java/de/rki/coronawarnapp/appconfig/AppConfigProviderTest.kt 48 additions, 18 deletions...a/de/rki/coronawarnapp/appconfig/AppConfigProviderTest.kt
- Corona-Warn-App/src/test/java/de/rki/coronawarnapp/appconfig/AppConfigSourceTest.kt 33 additions, 3 deletions...ava/de/rki/coronawarnapp/appconfig/AppConfigSourceTest.kt
- Corona-Warn-App/src/test/java/de/rki/coronawarnapp/appconfig/download/AppConfigServerTest.kt 37 additions, 8 deletions...i/coronawarnapp/appconfig/download/AppConfigServerTest.kt
- Corona-Warn-App/src/test/java/de/rki/coronawarnapp/appconfig/download/AppConfigStorageTest.kt 6 additions, 2 deletions.../coronawarnapp/appconfig/download/AppConfigStorageTest.kt
- Corona-Warn-App/src/test/java/de/rki/coronawarnapp/appconfig/download/DefaultAppConfigSanityCheck.kt 57 additions, 0 deletions...warnapp/appconfig/download/DefaultAppConfigSanityCheck.kt
- Corona-Warn-App/src/test/java/de/rki/coronawarnapp/appconfig/download/DefaultAppConfigSourceTest.kt 55 additions, 0 deletions...awarnapp/appconfig/download/DefaultAppConfigSourceTest.kt
Loading
Please register or sign in to comment