From fbe162a1bc6779ea3c281a6126eeed8ae004ebbe Mon Sep 17 00:00:00 2001 From: Ivan Enderlin Date: Thu, 6 Apr 2023 11:29:10 +0200 Subject: [PATCH] test(sdk): Fix an integration test in Sliding Sync. The test does the following: 1. Create 2 (identical) lists, 2. Do a sync. 3. Assert that the 1st and 2nd lists are receiving an update, 4. Add a 3rd (identical) list, 5. Do a new sync, 6. Assert that 3rd list is receiving an update. This last step is wrong. All lists should receive an update as they are identical. --- testing/sliding-sync-integration-test/src/lib.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/testing/sliding-sync-integration-test/src/lib.rs b/testing/sliding-sync-integration-test/src/lib.rs index 78628a45f..10ab983e1 100644 --- a/testing/sliding-sync-integration-test/src/lib.rs +++ b/testing/sliding-sync-integration-test/src/lib.rs @@ -360,7 +360,7 @@ async fn adding_list_later() -> anyhow::Result<()> { // we only heard about the ones we had asked for if !summary.lists.is_empty() { // only if we saw an update come through - assert_eq!(summary.lists, [list_name_3]); + assert_eq!(summary.lists, [list_name_1, list_name_2, list_name_3]); // we didn't update the other lists, so only no 2 should se an update saw_update = true; break;