* - Removed Kotlin Android Extensions plugin
- Replaced usages of Kotlin Synthetics by Jetpack view binding
- Added Kotlin Parcelize standalone plugin

(#2022)

* Fix detekt issue (#2022)

* Use view binding for TestResultSectionView (EXPOSUREAPP-4495) (#2022)

* Use bind() instead of inflate() for view binding in TestResultSectionView (EXPOSUREAPP-4495) (#2022)

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

* Change primary key for risk result database to a monotonic ID.
This prevents time travel from breaking "get latest" as we will get the latest by monotonic ID, instead of "calculatedAt".

* We don't need `fallbackToDestructiveMigrationFrom(3)` until we actually upgrade the schema to 3.

* Fix database test (missing monotonicId)

* Catch exceptions during migrations and recreate the table.

* Add check for result size after recreating+insertion.

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

* Update work manager to 2.5.0-beta02 to fix potential reliability issues.

* Update work manager test dependency to 2.5.0-beta02.

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

* Extract native libs set to true for both builds (DEV) (#1689)

* Fix sporadic worker crash on older android versions (EXPOSUREAPP-3987) (#1727)

* Fix notification issues by introducing IDs (EXPOSUREAPP-3971) (#1732)

* Reduce redundant code in notifications and introduce notification IDs

* Clean up unused constants

* Cancel risk score level notification once new test result is available

* Add more logging

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

* Versionbump for 1.7.1/RC1 (DEV) (#1733)

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

* Update to latest stable coroutines.

* Fix flaky test.
Depending how how fast items emitted, some intermediate values might never be emitted.

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

* Better log output.

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

* Test run

* Test run

* Test run

* Clean up

* Property fix

* Quotes fix

* merge conflict solved

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

* moved getHash call to DeviceForTesters Flavor to satisfy CircleCI

* added variant prefix

* Initial refactoring on the Submission flow.
Trimming down SubmissionViewModel.kt

* Refactored submission tan, and began instrumentation tests. Intro, Dispatcher and Tan complete

* Adapted instrumentation tests after merge - added ToDos

* Completed boilerplate submission instrumentation tests. Verification and screenshots to follow

* Fix lints

* More lint issues.

* Fix instrumentation test dependency missmatch.

* Move screenshot related permissions into a debug AndroidManifest.xml,
such that they don't get included in production builds.

* Added missing new line to manifest

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

* Allow WorkerManager's workers to be injected.

* Remove static access for Playbook

* Add mock for playbook so dagger can create the worker factory map.

Test pipeline config

* FileLoggerPlus - RX added - preparation

* Buffer config as constants

* avoid empty lines

* Reworked logging to file - wip

* Collecting errors to dao - wip

* Added crash report to app db and created module

* do it

* inject crashReportTree - wip

* use dagger injection

* retrieve short hash via gradle

* removed todo

* Updated database to version 2 for new ReportEntity - old base

* deprecated old engagement - RX not allowed anymore

* use global scope for coroutines in crash report tree - ktlint clean

* added menu item - error reporter (replace later to new structure)

* basic recylcerview and fragment added to menu - wip

* deprecated data class

* Delete CrashModel.kt

* crash report list item to use data binding

* separated Reporteradapter - klint clean

* use separated Adapter in ReporterFragment - cleanup

* Added ErrorReporterViewModel - connected data to list

* dependency injection for crash report, better structure, todo cleanup

* some refactoring, cleanup, ktlint

* adjustments for 1.6.x

* temp action to open crash report fragment

* removed testcode

* updated layout - wip

* connected view & data

* satisfy circle ci checks

* plant reportingTree, testbutton, klint clean

* updated layout for reporter - wip

* better format and layout

* better message handling

* missing icons for errors on debug

* Adjusted error report index

* Removed ugly show button, report clickable

* Added SettingsCrashReportDetailsFragment & corresponding files

* fixed merge inconsistence

* fixed injection overall

* use livedata

* details added to report - wip

* beauty

* share & layout root preparation

* make error report shareable

* removed Dispatcher.IO for example

* Introduce skeleton classes for bugreporting

* Refactoring, adjustments to be able to build again

* Created BugEventEntity, Changed id to Long for more efficiency,  Removed unused events

* refactored ui to deviceForTesters, testmenu etc

* adjusted interfaces & related, better structure

* uses own db, added converter, adjusted modules

* adjusted bugrepo, defaultbugrepo

* shared report, better visualisation

* changed crash db name

* Implemented DefaultBugRepository

* Added device info to BugEvent, BugReporter collects Tag, Added default impl of BugEvent etc

* corrected bug in RollingLogHistory

* removed deprecated crash package

* quickfix - wip

* cleanup

* cleanUp, adjusted simulation button to new structure

* moved bug related layouts to deviceForTesters package

* removed old RX dependenies and related context

* fix build for deviceDebug

* lint clean

* current set of relevant exceptions as before

* attach log to output

* improved output

* Delete obsolet ic_bug in main

* Delete ic_coffee.xml

* changed shortID to shortCommitHash for better transparency

* companion to bottom

* removed string encapsulation

* cleanup unused import and blank line - PR Comment

* typo fixed

* rm empty line

* removed relict module

* layout fix

* adjusted unrelated method call

* fixed package inkonsistence

* Use flow instead of livedata in non-ui

* better fun naming

* Replaced interface with inline listener

* Improved SettingsCrashReportDetailsFragment and SettingsCrashReportViewModel, formatting adjusted

* cleaned converter

* Delete Project.xml

* added log statement to DefaultBugRepository

* adjusted magical number

* Use CommonConverters instead of unique Converters in BugDatabase

* Extension function for the resuce -defaultBugRepository

* removed FormatterCrashReport completly

* changed long to UUID

* satisfy buggy ktlint checks

* Update SettingsCrashReportViewModel.kt

use postvalue

* avoid passing viewmodel

* store appVersionCode as Long for beeing futuresave

* Update SettingsCrashReportViewModel.kt

removed lazy init

* Changed BugDatabase to factory pattern

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

Co-authored-by: harambasicluka <64483219+harambasicluka@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

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

This reverts commit 86406397

* Remove URLs from gradle.properties they are now read via `prod_environment.json`

* Calculate app versionName and versionCode from gradle major/minor/patch/build variables.

* tasks wip

* Additional work on task controller skeleton code.

* Further refactoring iterations on the new task system interfaces.

* Further refactoring iterations on the new task system interfaces.

* Remove enum key type and replace with TaskRequest class

* Move the config interface into the factory show make clear what the relation here is.

* * Remove the task ID from the public interface, there is currently no need to expose it. Interested parties use the request's ID to find their task data.
* Throw a defined exception (early) if we submit a task for which there is no factory hooked up.

* Finished unit tests for TaskController
* Small adjustments to the public TaskController interface
* Task typing should not be based on the request types, this allows TaskRequests to be subclassed without shooting yourself in the foot by unexpectedly breaking the type checks.

* Lints

* Verify factory call order.

* Lints

* Add test menu entry for the TaskController

* Address PR comments.

* Fix flaky tests.

* Address PR comments.

* Fix dagger graph in deviceRelease mode, wrong file name ending.

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

Version bump

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

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

* Use signingConfig if locally available.
Make it easier for devs to test release builds.

* Remove empty signing scope

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

Version bump

* CircleCI new workflow

Co-authored-by: Ralf Gehrer <ralfgehrer@users.noreply.github.com>
Co-authored-by: Matthias Urhahn <matthias.urhahn@sap.com>

Introduce VM injection for home screen and enable screen testing (EXPOSUREAPP-2946, EXPOSUREAPP-2948) (#1320)

* Rename "MainFragment" to "HomeFragment"
Add VM injection.
Move initial logic into VM.

TODO: Move more code, write more tests.

* Additional refactoring, moving logic out of the Fragment.
Moving code into their own encapsulated routines/components.
Use composition to reuse TracingViewModel, SettingsViewModel and SubmissionViewModel

* Introduce @AppContext annotation to prevent anyone mistaking it for something else.

* Add VM to RiskDetailsFragment

* Simplify nested viewmodels, here: TracingViewModel.kt

* Add skeletons instrumentation test skeleton for single fragments with injection and mocking.

* Address PR comments.

* Introduce sealed events class, + some refactoring.

* Make the linter happy.

1.5.0 version code bump to 44

1.5.0 version code and name bump

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

* Remove code duplication between different flavors.
Remove extra MainFragment and NavGraph.
Introduce TestMenu indirection
Add logic for splitting test code up into extra fragments.

* Style list items a bit :)

* fix lints

* Detekt *hug*

* We can only show the menu entry if we are actually on the right flavor,
otherwise, in deviceDebug, we would not have access to the right resources.

* Address PR comments

* Fix wrong XML namespace.

* Define environment for each flavor and build type.

* Add smart LiveData class that does async initialization (not on the main thread).

* View extensions setGone/setInvisible

* Refactor test for api fragment.

* Replace ALT environment with a list to select from.

* klint

* Fix tests

* Fix tests

* Further flaky test fixes.

* Move the debug check for isLast3HourModeEnabled to the settings.
The KeyFileDownloader.kt shouldn't be aware of that detail.
The settings are what is affected by the build flavors/modes.

* Code cleanup and UI improvement of fragment_test_for_a_p_i.xml

* Code that retries flaky tests until we found a better solution.
Not great, Not terrible...

Co-authored-by: Mert Safter <mert.safter@sap.com>

* Move AppConfig related code into it's own package.
* Use "files" instead of "cache" for storing our AppConfigApiTest.
* Add "files" to "cache" migration.
* Refactor CDN related code into an independent "environment module".

TODO: UnitTests, Fallback behavior even if cached AppConfig is stale.

* Further structure refactoring and unit test for app config cache migration.

* Added config fallback behavior for all cases + additional tests.

* Improve code readability.

* First POC for multi-environment support.

* logic for ACME test Countries toggle button in Dev menu

* ACME Test Countries dev menu

* Additional clean up and refactoring on build constants.

* Cleaned up the test fragment a bit.

* Fixed detekt issue

* Fix flaky test that depends on build variant.

* Let the test fragment show us the actual environment we toggled to.

* Changes upon review

* Address PR comments.

Co-authored-by: Rituraj Sambherao <git.rituraj.sambherao@gmail.com>