Skip to content

Commit 7bcfb20

Browse files
authored
Update wording for push provider support test. (#4079) (#4553)
1 parent 20ac0f9 commit 7bcfb20

File tree

3 files changed

+6
-8
lines changed

3 files changed

+6
-8
lines changed

libraries/push/impl/src/main/kotlin/io/element/android/libraries/push/impl/troubleshoot/PushProvidersTest.kt

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -38,10 +38,8 @@ class PushProvidersTest @Inject constructor(
3838
val result = sortedPushProvider.isNotEmpty()
3939
if (result) {
4040
delegate.updateState(
41-
description = stringProvider.getQuantityString(
42-
resId = R.plurals.troubleshoot_notifications_test_detect_push_provider_success,
43-
quantity = sortedPushProvider.size,
44-
sortedPushProvider.size,
41+
description = stringProvider.getString(
42+
resId = R.string.troubleshoot_notifications_test_detect_push_provider_success_2,
4543
sortedPushProvider.joinToString { it.name }
4644
),
4745
status = NotificationTroubleshootTestState.Status.Success

libraries/push/impl/src/main/res/values/localazy.xml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,13 +58,14 @@
5858
<string name="troubleshoot_notifications_test_current_push_provider_failure">"No push providers selected."</string>
5959
<string name="troubleshoot_notifications_test_current_push_provider_success">"Current push provider: %1$s."</string>
6060
<string name="troubleshoot_notifications_test_current_push_provider_title">"Current push provider"</string>
61-
<string name="troubleshoot_notifications_test_detect_push_provider_description">"Ensure that the application has at least one push provider."</string>
62-
<string name="troubleshoot_notifications_test_detect_push_provider_failure">"No push providers found."</string>
61+
<string name="troubleshoot_notifications_test_detect_push_provider_description">"Ensure that the application supports at least one push provider."</string>
62+
<string name="troubleshoot_notifications_test_detect_push_provider_failure">"No push provider support found."</string>
6363
<plurals name="troubleshoot_notifications_test_detect_push_provider_success">
6464
<item quantity="one">"Found %1$d push provider: %2$s"</item>
6565
<item quantity="other">"Found %1$d push providers: %2$s"</item>
6666
</plurals>
67-
<string name="troubleshoot_notifications_test_detect_push_provider_title">"Detect push providers"</string>
67+
<string name="troubleshoot_notifications_test_detect_push_provider_success_2">"The application was built with support for: %1$s"</string>
68+
<string name="troubleshoot_notifications_test_detect_push_provider_title">"Push provider support"</string>
6869
<string name="troubleshoot_notifications_test_display_notification_description">"Check that the application can display notification."</string>
6970
<string name="troubleshoot_notifications_test_display_notification_failure">"The notification has not been clicked."</string>
7071
<string name="troubleshoot_notifications_test_display_notification_permission_failure">"Cannot display the notification."</string>

libraries/push/impl/src/test/kotlin/io/element/android/libraries/push/impl/troubleshoot/PushProvidersTestTest.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ class PushProvidersTestTest {
5353
assertThat(awaitItem().status).isEqualTo(NotificationTroubleshootTestState.Status.InProgress)
5454
val lastItem = awaitItem()
5555
assertThat(lastItem.status).isEqualTo(NotificationTroubleshootTestState.Status.Success)
56-
assertThat(lastItem.description).contains("2")
5756
assertThat(lastItem.description).contains("foo")
5857
assertThat(lastItem.description).contains("bar")
5958
}

0 commit comments

Comments
 (0)