From bb1a9f911a2d7ccaec8964107dac9641757b196d Mon Sep 17 00:00:00 2001
From: Matthias Urhahn <matthias.urhahn@sap.com>
Date: Mon, 12 Apr 2021 17:40:20 +0200
Subject: [PATCH] Improve logging to support debugging efforts
(EXPOSUREAPP-6365) (#2813)
Co-authored-by: harambasicluka <64483219+harambasicluka@users.noreply.github.com>
---
.../TestResultAvailableNotificationService.kt | 8 +++++++-
.../worker/DiagnosisTestResultRetrievalPeriodicWorker.kt | 2 ++
2 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/notification/TestResultAvailableNotificationService.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/notification/TestResultAvailableNotificationService.kt
index 2c17160e7..a6d942933 100644
--- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/notification/TestResultAvailableNotificationService.kt
+++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/notification/TestResultAvailableNotificationService.kt
@@ -23,7 +23,12 @@ class TestResultAvailableNotificationService @Inject constructor(
) {
suspend fun showTestResultAvailableNotification(testResult: TestResult) {
- if (foregroundState.isInForeground.first()) return
+ Timber.d("showTestResultAvailableNotification(testResult=%s)", testResult)
+
+ if (foregroundState.isInForeground.first()) {
+ Timber.d("App in foreground, skipping notification.")
+ return
+ }
if (!cwaSettings.isNotificationsTestEnabled.value) {
Timber.i("Don't show test result available notification because user doesn't want to be informed")
@@ -42,6 +47,7 @@ class TestResultAvailableNotificationService @Inject constructor(
setContentIntent(pendingIntent)
}.build()
+ Timber.i("Showing TestResultAvailable notification!")
notificationHelper.sendNotification(
notificationId = NotificationConstants.TEST_RESULT_AVAILABLE_NOTIFICATION_ID,
notification = notification,
diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisTestResultRetrievalPeriodicWorker.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisTestResultRetrievalPeriodicWorker.kt
index 348777db1..0b719ddd8 100644
--- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisTestResultRetrievalPeriodicWorker.kt
+++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisTestResultRetrievalPeriodicWorker.kt
@@ -95,6 +95,8 @@ class DiagnosisTestResultRetrievalPeriodicWorker @AssistedInject constructor(
tracingSettings.initialPollingForTestResultTimeStamp,
currentMillis
)
+ Timber.tag(TAG).d("Calculated days: %d", calculateDays)
+
if (calculateDays >= BackgroundConstants.POLLING_VALIDITY_MAX_DAYS) {
Timber.tag(TAG)
.d(" $id Maximum of ${BackgroundConstants.POLLING_VALIDITY_MAX_DAYS} days for polling exceeded.")
--
GitLab