fix(mockapi): dotnet compatibility failure (#769)

* fix(mockapi): dotnet compatibility failure dotnet uses different but compatible datetime format, so the comparison should use `coercedBodyEquals` --------- Co-authored-by: Mingzhe Huang (from Dev Box) <[email protected]>
Azure · Nov 11, 2024 · bb6a2dd · bb6a2dd
1 parent 778bacc
commit bb6a2dd
Show file tree

Hide file tree

Showing 6 changed files with 40 additions and 1 deletion.
diff --git a/.changeset/modern-dodos-act.md b/.changeset/modern-dodos-act.md
@@ -0,0 +1,5 @@
+---
+"@azure-tools/cadl-ranch-specs": patch
+---
+
+Fix dotnet compatibility issues in MockApi
diff --git a/packages/cadl-ranch-specs/http/type/dictionary/mockapi.ts b/packages/cadl-ranch-specs/http/type/dictionary/mockapi.ts
@@ -23,6 +23,12 @@ function createServerTests(uri: string, data: any) {
       response: {
         status: 204,
       },
+      handler: (req) => {
+        req.expect.coercedBodyEquals(data);
+        return {
+          status: 204,
+        };
+      },
       kind: "MockApiDefinition",
     }),
   };

diff --git a/packages/cadl-ranch-specs/http/type/property/additional-properties/mockapi.ts b/packages/cadl-ranch-specs/http/type/property/additional-properties/mockapi.ts
@@ -1,4 +1,4 @@
-import { passOnSuccess, ScenarioMockApi, json } from "@azure-tools/cadl-ranch-api";
+import { passOnSuccess, ScenarioMockApi, json, MockRequest } from "@azure-tools/cadl-ranch-api";
 
 export const Scenarios: Record<string, ScenarioMockApi> = {};
 
@@ -119,6 +119,13 @@ function createServerTests(url: string, value: any) {
       response: {
         status: 204,
       },
+      handler: (req: MockRequest) => {
+        const expectedBody = JSON.parse(JSON.stringify(value));
+        req.expect.coercedBodyEquals(expectedBody);
+        return {
+          status: 204,
+        };
+      },
       kind: "MockApiDefinition",
     }),
   };

diff --git a/packages/cadl-ranch-specs/http/type/property/nullable/mockapi.ts b/packages/cadl-ranch-specs/http/type/property/nullable/mockapi.ts
@@ -29,6 +29,15 @@ function createServerTests(url: string, value: unknown, patchNullableProperty?:
       response: {
         status: 204,
       },
+      handler: (req) => {
+        req.expect.coercedBodyEquals({
+          requiredProperty: "foo",
+          nullableProperty: patchNullableProperty || null,
+        });
+        return {
+          status: 204,
+        };
+      },
       kind: "MockApiDefinition",
     }),
   };

diff --git a/packages/cadl-ranch-specs/http/type/property/optionality/mockapi.ts b/packages/cadl-ranch-specs/http/type/property/optionality/mockapi.ts
@@ -23,6 +23,12 @@ function createServerTests(url: string, value: unknown) {
       response: {
         status: 204,
       },
+      handler: (req) => {
+        req.expect.coercedBodyEquals(value);
+        return {
+          status: 204,
+        };
+      },
       kind: "MockApiDefinition",
     }),
   };

diff --git a/packages/cadl-ranch-specs/http/type/property/value-types/mockapi.ts b/packages/cadl-ranch-specs/http/type/property/value-types/mockapi.ts
@@ -23,6 +23,12 @@ function createServerTests(url: string, data: unknown, convertedToFn?: (_: any)
       response: {
         status: 204,
       },
+      handler: (req) => {
+        req.expect.coercedBodyEquals(data);
+        return {
+          status: 204,
+        };
+      },
       kind: "MockApiDefinition",
     }),
   };