feat: add Spector integration tests for Payload_Head and Payload_JsonMergePatch

packages/typespec-ts/test/azureModularIntegration/generated/payload/head/.gitignore

@@ -0,0 +1,6 @@
+/**
+!/src
+/src/**
+!/src/index.d.ts
+!/.gitignore
+!/tspconfig.yaml

packages/typespec-ts/test/azureModularIntegration/generated/payload/head/src/index.d.ts

@@ -0,0 +1,24 @@
+import { ClientOptions } from '@azure-rest/core-client';
+import { isRestError } from '@azure/core-rest-pipeline';
+import { OperationOptions } from '@azure-rest/core-client';
+import { Pipeline } from '@azure/core-rest-pipeline';
+import { RestError } from '@azure/core-rest-pipeline';
+
+export declare interface ContentTypeHeaderInResponseOptionalParams extends OperationOptions {
+}
+
+export declare class HeadClient {
+    private _client;
+    readonly pipeline: Pipeline;
+    constructor(options?: HeadClientOptionalParams);
+    contentTypeHeaderInResponse(options?: ContentTypeHeaderInResponseOptionalParams): Promise<void>;
+}
+
+export declare interface HeadClientOptionalParams extends ClientOptions {
+}
+
+export { isRestError }
+
+export { RestError }
+
+export { }

packages/typespec-ts/test/azureModularIntegration/generated/payload/head/tspconfig.yaml

@@ -0,0 +1,13 @@
+emit:
+  - "@azure-tools/typespec-ts"
+options:
+  "@azure-tools/typespec-ts":
+    emitter-output-dir: "{project-root}"
+    generate-test: false
+    add-credentials: false
+    flavor: azure
+    azure-sdk-for-js: false
+    is-typespec-test: true
+    package-details:
+      name: "@msinternal/payload-head"
+      description: "Payload Head Test Service"

packages/typespec-ts/test/azureModularIntegration/generated/payload/json-merge-patch/src/index.d.ts

@@ -1,6 +1,8 @@
 import { ClientOptions } from '@azure-rest/core-client';
+import { isRestError } from '@azure/core-rest-pipeline';
 import { OperationOptions } from '@azure-rest/core-client';
 import { Pipeline } from '@azure/core-rest-pipeline';
+import { RestError } from '@azure/core-rest-pipeline';
 
 export declare interface CreateResourceOptionalParams extends OperationOptions {
 }
@@ -10,6 +12,8 @@ export declare interface InnerModel {
     description?: string;
 }
 
+export { isRestError }
+
 export declare class JsonMergePatchClient {
     private _client;
     readonly pipeline: Pipeline;
@@ -43,6 +47,8 @@ export declare interface ResourcePatch {
     intArray?: number[];
 }
 
+export { RestError }
+
 export declare interface UpdateOptionalResourceOptionalParams extends OperationOptions {
     body?: ResourcePatch;
 }

packages/typespec-ts/test/azureModularIntegration/payloadHead.spec.ts

@@ -0,0 +1,24 @@
+import { describe, it, beforeEach } from "vitest";
+import { HeadClient } from "./generated/payload/head/src/index.js";
+
+describe("Payload Head Client", () => {
+  let client: HeadClient;
+
+  beforeEach(() => {
+    client = new HeadClient({
+      endpoint: "http://localhost:3002",
+      allowInsecureConnection: true,
+      retryOptions: {
+        maxRetries: 0
+      }
+    });
+  });
+
+  it("should call contentTypeHeaderInResponse", async () => {
+    // The generated client returns void for HEAD operations; response headers
+    // (Content-Type and x-ms-meta) are validated by the mock server (returns 200
+    // only when the request is correct). No header assertions can be made here
+    // because the emitter does not expose response headers in the return type.
+    await client.contentTypeHeaderInResponse();
+  });
+});

packages/typespec-ts/test/azureModularIntegration/payloadJsonMergePatch.spec.ts

@@ -0,0 +1,91 @@
+import { assert, describe, it, beforeEach } from "vitest";
+import { JsonMergePatchClient } from "./generated/payload/json-merge-patch/src/index.js";
+
+describe("Payload JsonMergePatch Client", () => {
+  let client: JsonMergePatchClient;
+
+  beforeEach(() => {
+    client = new JsonMergePatchClient({
+      endpoint: "http://localhost:3002",
+      allowInsecureConnection: true,
+      retryOptions: {
+        maxRetries: 0
+      }
+    });
+  });
+
+  it("should createResource", async () => {
+    const body = {
+      name: "Madge",
+      description: "desc",
+      map: {
+        key: {
+          name: "InnerMadge",
+          description: "innerDesc"
+        }
+      },
+      array: [{ name: "InnerMadge", description: "innerDesc" }],
+      intValue: 1,
+      floatValue: 1.25,
+      innerModel: { name: "InnerMadge", description: "innerDesc" },
+      intArray: [1, 2, 3]
+    };
+    const result = await client.createResource(body);
+    assert.strictEqual(result.name, "Madge");
+    assert.strictEqual(result.description, "desc");
+    assert.strictEqual(result.map?.["key"]?.name, "InnerMadge");
+    assert.strictEqual(result.map?.["key"]?.description, "innerDesc");
+    assert.deepEqual(result.array, [
+      { name: "InnerMadge", description: "innerDesc" }
+    ]);
+    assert.strictEqual(result.intValue, 1);
+    assert.strictEqual(result.floatValue, 1.25);
+    assert.strictEqual(result.innerModel?.name, "InnerMadge");
+    assert.strictEqual(result.innerModel?.description, "innerDesc");
+    assert.deepEqual(result.intArray, [1, 2, 3]);
+  });
+
+  it("should updateResource", async () => {
+    // null values are intentional: JSON merge-patch semantics require null to
+    // signal field removal. The generated ResourcePatch type uses optional (?)
+    // fields without null, so 'as any' is needed to pass the null values while
+    // still exercising the correct serialization path.
+    const body = {
+      description: null,
+      map: {
+        key: { description: null },
+        key2: null
+      },
+      array: null,
+      intValue: null,
+      floatValue: null,
+      innerModel: null,
+      intArray: null
+    };
+    const result = await client.updateResource(body as any);
+    assert.strictEqual(result.name, "Madge");
+    assert.strictEqual(result.map?.["key"]?.name, "InnerMadge");
+  });
+
+  it("should updateOptionalResource", async () => {
+    // null values are intentional: JSON merge-patch semantics require null to
+    // signal field removal. The generated ResourcePatch type uses optional (?)
+    // fields without null, so 'as any' is needed to pass the null values while
+    // still exercising the correct serialization path.
+    const body = {
+      description: null,
+      map: {
+        key: { description: null },
+        key2: null
+      },
+      array: null,
+      intValue: null,
+      floatValue: null,
+      innerModel: null,
+      intArray: null
+    };
+    const result = await client.updateOptionalResource({ body: body as any });
+    assert.strictEqual(result.name, "Madge");
+    assert.strictEqual(result.map?.["key"]?.name, "InnerMadge");
+  });
+});

packages/typespec-ts/test/commands/spector-list.js

@@ -776,6 +776,14 @@ export const azureModularTsps = [
     outputPath: "payload/xml",
     inputPath: "payload/xml"
   },
+  {
+    outputPath: "payload/head",
+    inputPath: "payload/head"
+  },
+  {
+    outputPath: "payload/json-merge-patch",
+    inputPath: "payload/json-merge-patch"
+  },
   {
     outputPath: "server/versions/versioned",
     inputPath: "server/versions/versioned"