fix(msw): do not use spread objects if nullable object schema in `o…

…neOf` (#1626)

* fix(msw): not use unnecessary spread object when nullable object schema

* chore: add test case

packages/mock/src/faker/resolvers/value.ts

@@ -134,7 +134,13 @@ export const resolveMockValue = ({
         const func = `export const ${funcName} = (${args}): ${newSchema.name} => ({...${value}, ...${overrideVarName}});`;
         splitMockImplementations?.push(func);
       }
-      scalar.value = `{...${funcName}()}`;
+
+      if (newSchema.nullable) {
+        scalar.value = `${funcName}()`;
+      } else {
+        scalar.value = `{...${funcName}()}`;
+      }
+
       scalar.imports.push({
         name: newSchema.name,
         specKey: isRootKey(specKey, context.target) ? undefined : specKey,

tests/specifications/one-of.yaml

@@ -0,0 +1,52 @@
+openapi: 3.0.0
+info:
+  version: 1.0.0
+  title: AnyOf Schema
+  license:
+    name: MIT
+
+paths:
+  /one-of-with-nullable-object:
+    get:
+      operationId: getOneOfWithNullableObject
+      tags:
+        - pets
+      description: |-
+        oneOf with nullable object.
+      responses:
+        '200':
+          description: Pet
+          content:
+            application/json:
+              schema:
+                $ref: '#/components/schemas/Pet'
+
+components:
+  schemas:
+    Pet:
+      oneOf:
+        - $ref: '#/components/schemas/Dog'
+        - $ref: '#/components/schemas/Cat'
+    Dog:
+      type: object
+      nullable: true
+      required:
+        - id
+        - name
+      properties:
+        id:
+          type: integer
+          format: int64
+        name:
+          type: string
+    Cat:
+      type: object
+      required:
+        - id
+        - category
+      properties:
+        id:
+          type: integer
+          format: int64
+        category:
+          type: string