Merge tag '1.9.3' into 1.10-maintenance

Fedify 1.9.3

Author: dahlia

CHANGES.md

@@ -8,6 +8,13 @@ Version 1.10.1
 
 To be released.
 
+### @fedify/testing
+
+ -  Fixed `TestContext.getActor()` and `TestContext.getObject()` returning
+    `null` instead of calling registered dispatchers.  The methods now properly
+    invoke actor and object dispatchers when they are registered via
+    `setActorDispatcher()` and `setObjectDispatcher()`.  [[#530]]
+
 
 Version 1.10.0
 --------------
@@ -106,6 +113,21 @@ Released on December 24, 2025.
  -  Implemented `list()` method in `WorkersKvStore`.  [[#498], [#500]]
 
 
+Version 1.9.3
+-------------
+
+Released on January 22, 2026.
+
+### @fedify/testing
+
+ -  Fixed `TestContext.getActor()` and `TestContext.getObject()` returning
+    `null` instead of calling registered dispatchers.  The methods now properly
+    invoke actor and object dispatchers when they are registered via
+    `setActorDispatcher()` and `setObjectDispatcher()`.  [[#530]]
+
+[#530]: https://github.com/fedify-dev/fedify/issues/530
+
+
 Version 1.9.2
 -------------
 

packages/testing/src/mock.test.ts

@@ -357,3 +357,86 @@ test("MockFederation without queue sends all activities immediately", async () =
   assertEquals(mockFederation.sentActivities[0].queue, undefined);
   assertEquals(mockFederation.sentActivities[0].sentOrder, 1);
 });
+
+test("MockContext.getActor() calls registered actor dispatcher", async () => {
+  const mockFederation = createFederation<void>();
+
+  // Register actor dispatcher
+  mockFederation.setActorDispatcher(
+    "/users/{identifier}",
+    (ctx, identifier) => {
+      return new Person({
+        id: ctx.getActorUri(identifier),
+        preferredUsername: identifier,
+        name: `Test User ${identifier}`,
+      });
+    },
+  );
+
+  const context = mockFederation.createContext(
+    new URL("https://example.com"),
+    undefined,
+  );
+
+  const actor = await context.getActor("alice");
+
+  assertEquals(actor instanceof Person, true);
+  assertEquals(actor?.preferredUsername, "alice");
+  assertEquals(actor?.name, "Test User alice");
+  assertEquals(actor?.id?.href, "https://example.com/users/alice");
+});
+
+test("MockContext.getObject() calls registered object dispatcher", async () => {
+  const mockFederation = createFederation<void>();
+
+  // Register object dispatcher
+  mockFederation.setObjectDispatcher(
+    Note,
+    "/users/{identifier}/posts/{postId}",
+    (ctx, values) => {
+      return new Note({
+        id: ctx.getObjectUri(Note, values),
+        content: `Post ${values.postId} by ${values.identifier}`,
+      });
+    },
+  );
+
+  const context = mockFederation.createContext(
+    new URL("https://example.com"),
+    undefined,
+  );
+
+  const note = await context.getObject(Note, {
+    identifier: "alice",
+    postId: "123",
+  });
+
+  assertEquals(note instanceof Note, true);
+  assertEquals(note?.content, "Post 123 by alice");
+  assertEquals(note?.id?.href, "https://example.com/users/alice/posts/123");
+});
+
+test("MockContext.getActor() returns null when no dispatcher registered", async () => {
+  const mockFederation = createFederation<void>();
+  const context = mockFederation.createContext(
+    new URL("https://example.com"),
+    undefined,
+  );
+
+  const actor = await context.getActor("alice");
+  assertEquals(actor, null);
+});
+
+test("MockContext.getObject() returns null when no dispatcher registered", async () => {
+  const mockFederation = createFederation<void>();
+  const context = mockFederation.createContext(
+    new URL("https://example.com"),
+    undefined,
+  );
+
+  const note = await context.getObject(Note, {
+    identifier: "alice",
+    postId: "123",
+  });
+  assertEquals(note, null);
+});

packages/testing/src/mock.ts

@@ -178,7 +178,7 @@ class MockFederation<TContextData> implements Federation<TContextData> {
   // which causes JSR type analyzer to hang when combined with @opentelemetry/api
   // types present in Context.tracerProvider (issue #468).
   private webFingerDispatcher?: any;
-  private actorDispatchers: Map<string, any> = new Map();
+  public actorDispatchers: Map<string, any> = new Map();
   public actorPath?: string;
   public inboxPath?: string;
   public outboxPath?: string;
@@ -190,7 +190,7 @@ class MockFederation<TContextData> implements Federation<TContextData> {
   public nodeInfoPath?: string;
   public sharedInboxPath?: string;
   public objectPaths: Map<string, string> = new Map();
-  private objectDispatchers: Map<string, any> = new Map();
+  public objectDispatchers: Map<string, any> = new Map();
   private inboxDispatcher?: any;
   private outboxDispatcher?: any;
   private followingDispatcher?: any;
@@ -622,12 +622,34 @@ class MockContext<TContextData> implements Context<TContextData> {
     this.tracerProvider = options.tracerProvider ?? noopTracerProvider;
   }
 
-  getActor(_handle: string): Promise<any> {
-    return Promise.resolve(null);
+  async getActor(handle: string): Promise<any> {
+    if (
+      this.federation instanceof MockFederation && this.federation.actorPath
+    ) {
+      const dispatcher = this.federation.actorDispatchers.get(
+        this.federation.actorPath,
+      );
+      if (dispatcher) {
+        return await dispatcher(this, handle);
+      }
+    }
+    return null;
   }
 
-  getObject(_cls: any, _values: any): Promise<any> {
-    return Promise.resolve(null);
+  async getObject<TObject extends Object>(
+    cls: (new (...args: any[]) => TObject) & { typeId: URL },
+    values: Record<string, string>,
+  ): Promise<TObject | null> {
+    if (this.federation instanceof MockFederation) {
+      const path = this.federation.objectPaths.get(cls.typeId.href);
+      if (path) {
+        const dispatcher = this.federation.objectDispatchers.get(path);
+        if (dispatcher) {
+          return await dispatcher(this, values);
+        }
+      }
+    }
+    return null;
   }
 
   getSignedKey(): Promise<any> {