test(user): expect the user store as the resolver's second argument

Author: nperez0111

packages/core/src/user/createUserStore.test.ts

@@ -25,7 +25,9 @@ describe("createUserStore", () => {
     await userStore.loadUsers(["1", "2"]);
 
     expect(resolveUsers).toHaveBeenCalledTimes(1);
-    expect(resolveUsers).toHaveBeenCalledWith(["1", "2"]);
+    // The store itself is passed as the second argument, so resolvers can
+    // return synchronously and fill in more info via `setUser` later.
+    expect(resolveUsers).toHaveBeenCalledWith(["1", "2"], userStore);
     expect(userStore.getUser("1")).toEqual(makeUser("1"));
     expect(userStore.getUser("2")).toEqual(makeUser("2"));
     expect(userStore.store.state.size).toBe(2);
@@ -40,7 +42,7 @@ describe("createUserStore", () => {
 
     // Only the missing "3" should be requested the second time.
     expect(resolveUsers).toHaveBeenCalledTimes(2);
-    expect(resolveUsers).toHaveBeenNthCalledWith(2, ["3"]);
+    expect(resolveUsers).toHaveBeenNthCalledWith(2, ["3"], userStore);
 
     await userStore.loadUsers(["1", "2", "3"]);
     // Everything cached now → no further requests.

packages/core/src/y/extensions/AttributionExtension.test.ts

@@ -58,7 +58,8 @@ describe("AttributionExtension user loading", () => {
     addInsertMark(editor, ["alice"]);
 
     expect(resolveUsers).toHaveBeenCalledTimes(1);
-    expect(resolveUsers).toHaveBeenCalledWith(["alice"]);
+    // The extension's user store passes itself as the resolver's second arg.
+    expect(resolveUsers).toHaveBeenCalledWith(["alice"], expect.anything());
   });
 
   it("does not load users for changes without attribution marks", () => {