From 299d6fcc2fdab04dc6b970cf9f31c22384a1a222 Mon Sep 17 00:00:00 2001
From: Chris Bobbe <cbobbe@zulipchat.com>
Date: Fri, 21 Feb 2020 13:51:41 -0800
Subject: [PATCH] notifs [nfc]: Ensure tests pass with representative pm_users
 values.

To be reverted in the next commit.

The output of GroupPm.getPmUsersString in our Kotlin code was
actually a string of numbers separated by ', ' instead of ',' as we
expected. Ensure that our tests pass with ', ' before changing the
Kotlin code to use ','.
---
 src/notification/__tests__/notification-test.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/notification/__tests__/notification-test.js b/src/notification/__tests__/notification-test.js
index 2e9acc28828..8daba25744c 100644
--- a/src/notification/__tests__/notification-test.js
+++ b/src/notification/__tests__/notification-test.js
@@ -30,7 +30,7 @@ describe('getNarrowFromNotificationData', () => {
   test('on notification for a group message returns a group narrow', () => {
     const notification = {
       recipient_type: 'private',
-      pm_users: '1,2,4',
+      pm_users: '1, 2, 4',
     };
     const usersById = new Map([
       [1, { email: 'me@example.com' }],
@@ -47,7 +47,7 @@ describe('getNarrowFromNotificationData', () => {
   test('do not throw when users are not found; return null', () => {
     const notification = {
       recipient_type: 'private',
-      pm_users: '1,2,4',
+      pm_users: '1, 2, 4',
     };
     const usersById = new Map();