Fix bugs in sync test

This test wrongly asserted that `initialSyncLimit` would be used to make a filter It is used only for the initial sync inline filter, and not in POST /filter
matrix-org · Sep 1, 2022 · e808e0f · e808e0f
1 parent faa8186
commit e808e0f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/spec/unit/matrix-client.spec.ts b/spec/unit/matrix-client.spec.ts
@@ -427,7 +427,7 @@ describe("MatrixClient", function() {
                 }
             });
         });
-        await client.startClient();
+        await client.startClient({ filter });
         await syncPromise;
     });
 

diff --git a/src/sync.ts b/src/sync.ts
@@ -102,7 +102,7 @@ const MSC2716_ROOM_VERSIONS = [
 function getFilterName(userId: string, suffix?: string): string {
     // scope this on the user ID because people may login on many accounts
     // and they all need to be stored!
-    return `FILTER_SYNC_${userId}` + suffix ? "_" + suffix : "";
+    return `FILTER_SYNC_${userId}` + (suffix ? "_" + suffix : "");
 }
 
 function debuglog(...params) {
-Original file line number
+Diff line change
@@ Expand Up / @@ -427,7 +427,7 @@ describe("MatrixClient", function() { @@
                     }
                 });
             });
-            await client.startClient();
+            await client.startClient({ filter });
             await syncPromise;
         });
@@ Expand Down @@