fix(infra): move surrogate-pair test to describe scope level

lsr911 · lsr911 · commit f70268b9b0b6 · 2026-07-07T20:31:50.000+08:00
The regression test in push-apns.test.ts was inadvertently nested
inside the previous it() callback, causing CI failures with
'Calling the test function inside another test function is not
allowed'. Move it to the correct scope inside the describe block.
diff --git a/src/infra/push-apns.test.ts b/src/infra/push-apns.test.ts
@@ -836,31 +836,31 @@ describe("push APNs send semantics", () => {
       environment: "production",
       transport: "relay",
     });
+  });
 
-    it("does not split surrogate pairs when truncating non-JSON error bodies", async () => {
-      const { send, registration, auth } = createDirectApnsSendFixture({
-        nodeId: "ios-node-surrogate-reason",
-        environment: "sandbox",
-        sendResult: {
-          status: 400,
-          apnsId: "apns-surrogate-reason-id",
-          body: "x".repeat(199) + "🚀tail",
-        },
-      });
-
-      const result = await sendApnsAlert({
-        registration,
-        nodeId: "ios-node-surrogate-reason",
-        title: "Wake",
-        body: "Ping",
-        auth,
-        requestSender: send,
-      });
+  it("does not split surrogate pairs when truncating non-JSON error bodies", async () => {
+    const { send, registration, auth } = createDirectApnsSendFixture({
+      nodeId: "ios-node-surrogate-reason",
+      environment: "sandbox",
+      sendResult: {
+        status: 400,
+        apnsId: "apns-surrogate-reason-id",
+        body: "x".repeat(199) + "🚀tail",
+      },
+    });
 
-      const record = requireRecord(result, "APNs result");
-      expect(record.reason).not.toContain("�");
-      expect(record.ok).toBe(false);
-      expect(record.status).toBe(400);
+    const result = await sendApnsAlert({
+      registration,
+      nodeId: "ios-node-surrogate-reason",
+      title: "Wake",
+      body: "Ping",
+      auth,
+      requestSender: send,
     });
+
+    const record = requireRecord(result, "APNs result");
+    expect(record.reason).not.toContain("�");
+    expect(record.ok).toBe(false);
+    expect(record.status).toBe(400);
   });
 });
