* Remove "lastTimeDiagnosisKeysFromServerFetch" and replace it with less missleading data.
While it was called "lastTimeDiagnosisKeysFromServerFetch" it was actually "last time we submitted keys to google".

* To decide whether to refresh in "onResume", we now use "has there been any submission to the ENF?"
* To display a timestamp on the risk card, we take the last successful submission to the ENF as the risk card displays the calculation results based on the latest submission. While we could use the last calculated risk level result timestamp, we currently also trigger risk calculations if there are no new submissions to the ENF, which would mean the timestamp is updated even though the result is not based on new data.

I've also fixed the test fragment button behavior and added descriptions, the "Reset risk level" button surfaced the initial issue because it behaved like a "do a 75% data reset" button.

* Tests and LINTs

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 displayed details in tracing inactive states.

* Simplified UI states.
Replaced RiskLevel and RiskLevelConstants with RiskState { LOW; HIGH, FAILED }.

* Fix low risk card missing the encounters if there have been more than 0.

* Fix missing card divider/border on error cards in light mode.

* Fix merge regression.

* LINTs

* Fix progress card colors on white card background color.

* Fix progress card colors on white card background color.

* Don't allow negative active tracing days.

* Revert sync behavior (extra PR)

* Address PR comments.

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

* Use lastRiskResult from DB

* use correct format for windows and timestamp

* klint

* Reduce RiskLevelResult Storage API and fill exposure windows in risk result if available.

* Default exposure windows to null while mapping

* Don't add empty exposure windows list to risk results

* fix broken tests

* Added tests for exposure windows

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

* Converted ExposureWindows to their json wrapper counterparts before exporting to give the json fields proper names instead of the obfuscated enf ones

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

* Added timestamp to exposure window export

* Switched to a more file name compatible DateTimeFormat

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

* ktlint says these are redundant, i don't like it but the overlords forced me to do it.

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

* changed filename suffix to json

Co-authored-by: BMItter <berndus@gmx.de>

* Removed traces of grey risk card.

* Removed UNKNOWN_RISK_INITIAL constant.

* Removed unused method.

* Removed old comments.

* Introduce risklevel calculation failure state to get consistent card states.

* Cleaned up the risk card states.
Refactored "RiskLevel" to be calculated based on either result or failure reasons.
Added the new "no internet" error card.

* Updated color states.

* Fixed card color issues.

* Fixed detekt issues.

* Remove test for deleted file.

* Fix test regressions.

* Fixed tests and LINTs.

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

Risk Calculation does not work because DiagnosisKeysDataMapping is not applied (EXPOSUREAPP-3895) (#1690)

* We now check for the DiagnosisKeyDataMapping and apply it to the ENFClient if changed

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

* Moved diagnosisKeysDataMapping to windowProvider

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

* Renaming and linting

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

* Moved app config fetch to key provide task, added error handling, added logging to config replace

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

* Fixed old unit tests and implemented tests for config comparison,
TestForAPIFragment is currently broken

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

* Linting

* Added more unit testing for DefaultDiagnosisKeysDataMapper and fixed TestForAPIFragment

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

* Update default app config to latest version

This includes a change for the default for reportTypeWhenMissing from
0 to 1, as 0 is not accepted by setDiagnosisKeysDataMapping.

https://github.com/corona-warn-app/cwa-app-tech-spec/pull/10/commits/090801921f4a8581543a9f716fd0572c12901c71
https://github.com/corona-warn-app/cwa-protocol-buffers/commit/38a641e2a808c41d225e9fd09d27ec7959551412



* Fixed Unittests

Co-authored-by: Maximilian Lenkeit <maximilian.lenkeit@sap.com>
Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>
Co-authored-by: Ralf Gehrer <ralfgehrer@users.noreply.github.com>

* preparation share exposureWindows via email etc. - wip

* Exposure Windows can be shared if data is availabe

* Manifest and provider path deviceForTesters only

* fixed strange conflicts

* klint

* Pretty exposure window export

* removed testlog

* removed relicts

Persist ExposureWindow based risk level calculation results and update the UI (EXPOSUREAPP-3910,EXPOSUREAPP-3855) (#1705)

* First draft to refactor `RiskRepository` away and persist risk level results produced by ExposureWindow calculations.

TODO: Tests, Legacy data migration, Persist ExposureWindow's in tester builds.

* Unit tests for refactored classes.

* LINTs

* First draft for legacy risk data migration.

TODO: Tests.

* Store exposure windows on device for tester builds and perform clean on them too after risk result deletions.
Split RiskResultStorage such that the exposure window storage code is only available in deviceForTesters flavor builds.

* simplification

* no preference should return no value

* Remove TODOs, singletons are refactored away ;)

* Simplify RiskLevelTask interface

* Split risk level storage implementations to let production (device flavor) have a NOOP for storing/deleting exposure windows.

* Remove unused storage function.
Add test skeletons.

* unit test

* unit test

* Update risk card to show new window mode based information.

* Fix test regressions.

* Address PR comments.

* LINTs

* Everybody get's LINT for XMAS.

* Finish unit tests and remove unused classes.

Co-authored-by: chris-cwa <chris.cwa.sap@gmail.com>

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

* updated test api fragment to display exposure windows

* do not show dummy text on device

* removed it

* Show aggregated risk result in testmenu riskCalculationFragment

* Use extention fun to build string for aggregated result

* Show exposure window count and exposure windows as json

+ fixed a layoutbug with cut off

* satisfy lint

* Use real values

* Connected ExposureResultStore

* ENF v2 Calculation adjusted Testmenu Entry for v2

* Adjusted to ExposureResultStore changes

* Added logs

* Removed and adjusted some old stuff

* Made backend parameters a little bit better readable

* its better this way

* Created additional risk calc info

* Refactoring

* sourcecheck clean

* inject enf client and exposure result store

* Fix klint.

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

Fix RiskLevelCalculationTask not being executed if no new match was found (EXPOSUREAPP-3849) (#1657)

* Execute risk calculation even if no match.

* Pass an "originTag" with each started task to support debugging efforts.
Fix unit test regression.

* Re-add missing download task arguments.

* Fix misleading log statement.

* activate WindowExposure mode

* extirpated tokens!

* detekt, ktlint

* fixed tests

* - v1 methods

* - get v1 exposure summary

* - fixme

* updated exposure summary repo

* Adding mock dependencies that are required to instantiate the worker factories in our binding test.

* Added numberOfDaysWithHighRisk and numberOfDaysWithLowRisk

according to new tech spec

* Transformed minimumDistinctEncountersForRisk and mostRecentDateForRisk into extension fun

* make sure list of windows and aggregated result belong together

* sending values for current UI

* fixed ktlint

* renamed ExposureResultStore

* fixed tests

Co-authored-by: Matthias Urhahn <matthias.urhahn@sap.com>
Co-authored-by: BMItter <berndus@gmx.de>

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

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

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

* 1. Replaced redundant uiStateState during test result fetch with a NetworkRequestWrapper that is able to represent multiple states. This allows for better request state handling throughout the app
2. Completely removed the SubmissionViewModel
3. Failure to fetch the test should now always display the invalid test card on the home screen

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

* Keep displaying fetching card on server errors as this usually does not indicate any problems with the test itself

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

* Fixing unit tests (mostly just syntax, one test case changed due to new card)

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

* Promoted unknown host exception to server exception

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

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

* Enable logfile for tester builds by default.

* Fix test regression.

* Initial switch to new app config version (no tests/lint)

* Fixed some unit tests, not all are working yet

* Fixed ConfigParserTest

* ktlint fixes

* ktlint

* ExposureWindows to map is now beautiful

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

Co-authored-by: BMItter <Berndus@gmx.de>

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

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

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

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

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

* Split and hide the protobuf config behind interfaces with individual mappers responsible for creating the desired formats.

* Merge branch 'release/1.7.x' into feature/3455-more-frequent-riskscore-updates-configs

# Conflicts:
#	Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/test/risklevel/ui/TestRiskLevelCalculationFragmentCWAViewModel.kt
#	Corona-Warn-App/src/main/java/de/rki/coronawarnapp/transaction/RiskLevelTransaction.kt
#	Corona-Warn-App/src/test/java/de/rki/coronawarnapp/transaction/RiskLevelTransactionTest.kt

* Make the AppConfig observable.
Provide the server time offset.
Offer a lastUpdatedAt timestamp.
Add an app config specific test screen.
Clean up test screens a bit and move debug options out of API test options.

* Fix test regression due to refactoring (moved code around).

* Store the server timestamp and offset at retrieval.
Switch to config storage via json to be able to store additional meta data fields (i.e. time).

* KLint and Me have a hate relationship based on both mutual admiration.

* Fix time offset parsing being locale dependent.

* Fix broken unit tests.

* Improve offset accuracy, move before unzipping.

* Fix overly long livedata subscription to results (viewmodel scope vs observer scope)

* Add mapping for the new protobuf configs + tests.

* For cached (retrofit) response, we need to check the cacheResponse and its timestamps
to determine an accurate time offset.

* Exposure a boolean property to tell us when a fallback config is being used.

* Hide the observable flow<ConfigData> behind a method that can automatically triggers refreshes.

* Use a common mapper interface.

* Address PR comments and KLints.

* Fix refactoring regression.

* Improve config unzipping code.

* Add flag to forward exception thrown during HotDataFlow.kt initialization.

* Don't specify a default context via singleton.

* Move download and fallback logic into it's own class just responsible for sourcing the config: "AppConfigSource".
"AppConfigProvider" is now only responsible for making it available.

* Simplify current concepts for making the app config observable until we have a default configuration.

* Improve app config test screen, delete options, better feedback.
Show toast instead of crash on errors.

* Fixed GSON serialization not encoding/decoding the byte array correctly.
Added specific type adapters for instant and duration to get cleaner json.

* Remove type adapters from base gson due to conflict with CalculationTrackerStorage.

* We want to default to forced serialization of instant by our converters, instead of using the default serialization which will differ
between Java8.Instant and JodaTime.Instant, to prevent future headaches there, register explicit converters by default,
and overwrite them if necessary (currently only needed for CalculationTrackerStorage.kt).

* Improve AppConfigServer code readability by moving code into extensions.

* Fix merge conflicts

* Throw a more specific exception if gson type decoding fails due to malformed base64 encoding.

* Add unit test for Gson ByteArrayAdapter.

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

* converted risk level transaction to task

* simplified

* fixed config: timeout is now duration

* less is more

* arrangement + empty lines

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

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

* submit risk level task requests

* - unreachable catch blocks

* housekeeping

* satisfied ktlint + detekt

* fixed flow: isRefreshing

* fixed flavour for testers

* refactored benchmark

* fixed package declaration

* unit tests

* fixed injection

* fixed tests

* comments from PR

* fixed injection

* fixed lint issues

* fixed another merge conflict

* - unused class

* map task factory

* fulfilled comment style wishes

* fixed tests suspended by changes from develop branch

* Update Corona-Warn-App/src/main/java/de/rki/coronawarnapp/risk/DefaultRiskLevels.kt

https://github.com/corona-warn-app/cwa-app-android/pull/1412#discussion_r514325776



Co-authored-by: Kolya Opahle <k.opahle@sap.com>

* changes from PR

* fixed a test

* show logs, ktlint issues

* made log tag non-nullable

* improvements from reviewers

* reduced static access

* Saint Pipeline, please give me the green light

* toString()

* ktlint + di

* fixed test

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

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

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

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

* Switch 3 to 24 hour mode

* Fix incorrect toast message.

* Create `TestSettings` class.

* Adjust test to handle 24 hour mode.

* Adjust test to check for day rollover in hourly keypkg mode.

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