From 13a0467da28c7ea74ac3782cfb059f740cbbc9ff Mon Sep 17 00:00:00 2001 From: Benoit Marty Date: Tue, 8 Apr 2025 12:01:52 +0200 Subject: [PATCH] Update wording for push provider support test. (#4079) (#4553) --- .../libraries/push/impl/troubleshoot/PushProvidersTest.kt | 6 ++---- libraries/push/impl/src/main/res/values/localazy.xml | 7 ++++--- .../push/impl/troubleshoot/PushProvidersTestTest.kt | 1 - 3 files changed, 6 insertions(+), 8 deletions(-) diff --git a/libraries/push/impl/src/main/kotlin/io/element/android/libraries/push/impl/troubleshoot/PushProvidersTest.kt b/libraries/push/impl/src/main/kotlin/io/element/android/libraries/push/impl/troubleshoot/PushProvidersTest.kt index d75c4f4d59..5a30db6f59 100644 --- a/libraries/push/impl/src/main/kotlin/io/element/android/libraries/push/impl/troubleshoot/PushProvidersTest.kt +++ b/libraries/push/impl/src/main/kotlin/io/element/android/libraries/push/impl/troubleshoot/PushProvidersTest.kt @@ -38,10 +38,8 @@ class PushProvidersTest @Inject constructor( val result = sortedPushProvider.isNotEmpty() if (result) { delegate.updateState( - description = stringProvider.getQuantityString( - resId = R.plurals.troubleshoot_notifications_test_detect_push_provider_success, - quantity = sortedPushProvider.size, - sortedPushProvider.size, + description = stringProvider.getString( + resId = R.string.troubleshoot_notifications_test_detect_push_provider_success_2, sortedPushProvider.joinToString { it.name } ), status = NotificationTroubleshootTestState.Status.Success diff --git a/libraries/push/impl/src/main/res/values/localazy.xml b/libraries/push/impl/src/main/res/values/localazy.xml index 5c6c852f14..9860c8bef7 100644 --- a/libraries/push/impl/src/main/res/values/localazy.xml +++ b/libraries/push/impl/src/main/res/values/localazy.xml @@ -58,13 +58,14 @@ "No push providers selected." "Current push provider: %1$s." "Current push provider" - "Ensure that the application has at least one push provider." - "No push providers found." + "Ensure that the application supports at least one push provider." + "No push provider support found." "Found %1$d push provider: %2$s" "Found %1$d push providers: %2$s" - "Detect push providers" + "The application was built with support for: %1$s" + "Push provider support" "Check that the application can display notification." "The notification has not been clicked." "Cannot display the notification." diff --git a/libraries/push/impl/src/test/kotlin/io/element/android/libraries/push/impl/troubleshoot/PushProvidersTestTest.kt b/libraries/push/impl/src/test/kotlin/io/element/android/libraries/push/impl/troubleshoot/PushProvidersTestTest.kt index cc4df51b55..8947b646a6 100644 --- a/libraries/push/impl/src/test/kotlin/io/element/android/libraries/push/impl/troubleshoot/PushProvidersTestTest.kt +++ b/libraries/push/impl/src/test/kotlin/io/element/android/libraries/push/impl/troubleshoot/PushProvidersTestTest.kt @@ -53,7 +53,6 @@ class PushProvidersTestTest { assertThat(awaitItem().status).isEqualTo(NotificationTroubleshootTestState.Status.InProgress) val lastItem = awaitItem() assertThat(lastItem.status).isEqualTo(NotificationTroubleshootTestState.Status.Success) - assertThat(lastItem.description).contains("2") assertThat(lastItem.description).contains("foo") assertThat(lastItem.description).contains("bar") }