feat: Added tests and review

Author: makeev-pavel

src/apitypes/rest/rest.operation.ts

@@ -295,7 +295,7 @@ const createSingleOperationSpec = (
     return {
       ...specBase,
       paths: {
-        [path]: { ...pathData },
+        [path]: pathData,
       },
       components: {
         ...specBase.components,

test/document-group.test.ts

@@ -47,8 +47,18 @@ describe('Document Group test', () => {
     await runMergeOperationsCase('case1')
   })
 
-  test('should have properly merged documents with pathItems', async () => {
-    await runMergeOperationsCase('case2')
+  describe('PathItems tests', () => {
+    test('should have properly merged documents', async () => {
+      await runMergeOperationsCase('case2')
+    })
+
+    test('should have properly merged documents and delete unused operations', async () => {
+      await runMergeOperationsCase('case3')
+    })
+
+    test('should have properly merged documents mixed formats (operation + pathItems operation)', async () => {
+      await runMergeOperationsCase('case4')
+    })
   })
 
   async function runMergeOperationsCase(caseName: string): Promise<void> {

test/projects/merge-operations/case3/1.yaml

@@ -0,0 +1,14 @@
+openapi: "3.1.0"
+info:
+  title: test
+  version: 0.1.0
+paths:
+  /path1:
+    $ref: '#/components/pathItems/pathItem1'
+components:
+  pathItems:
+    pathItem1:
+      get:
+        responses:
+          '200':
+            description: response description main

test/projects/merge-operations/case3/2.yaml

@@ -0,0 +1,19 @@
+openapi: "3.1.0"
+info:
+  title: test
+  version: 0.1.0
+paths:
+  /path2:
+    $ref: '#/components/pathItems/pathItem2'
+components:
+  pathItems:
+    pathItem2:
+      post:
+        responses:
+          '200':
+            description: response description main
+    pathItem3:
+      post:
+        responses:
+          '200':
+            description: response description unused

test/projects/merge-operations/case3/config.json

@@ -0,0 +1,19 @@
+{
+  "packageId": "55",
+  "apiType": "rest",
+  "version": "v1",
+  "files": [
+    {
+      "fileId": "1.yaml",
+      "publish": true,
+      "labels": [],
+      "commitId": "6c778b1f44200bd19944a6a8eac10a4e5a21a1cd"
+    },
+    {
+      "fileId": "2.yaml",
+      "publish": true,
+      "labels": [],
+      "commitId": "6c778b1f44200bd19944a6a8eac10a4e5a21a3cd"
+    }
+  ]
+}

test/projects/merge-operations/case3/result.yaml

@@ -0,0 +1,21 @@
+openapi: 3.1.0
+info:
+  title: manualGroup
+  version: v1
+paths:
+  /path1:
+    $ref: '#/components/pathItems/pathItem1'
+  /path2:
+    $ref: '#/components/pathItems/pathItem2'
+components:
+  pathItems:
+    pathItem1:
+      get:
+        responses:
+          '200':
+            description: response description main
+    pathItem2:
+      post:
+        responses:
+          '200':
+            description: response description main

test/projects/merge-operations/case4/1.yaml

@@ -0,0 +1,14 @@
+openapi: "3.1.0"
+info:
+  title: test
+  version: 0.1.0
+paths:
+  /path1:
+    $ref: '#/components/pathItems/pathItem1'
+components:
+  pathItems:
+    pathItem1:
+      get:
+        responses:
+          '200':
+            description: response description main

test/projects/merge-operations/case4/2.yaml

@@ -0,0 +1,10 @@
+openapi: "3.1.0"
+info:
+  title: test
+  version: 0.1.0
+paths:
+  /path2:
+    post:
+      responses:
+        '200':
+          description: response description main

test/projects/merge-operations/case4/config.json

@@ -0,0 +1,19 @@
+{
+  "packageId": "55",
+  "apiType": "rest",
+  "version": "v1",
+  "files": [
+    {
+      "fileId": "1.yaml",
+      "publish": true,
+      "labels": [],
+      "commitId": "6c778b1f44200bd19944a6a8eac10a4e5a21a1cd"
+    },
+    {
+      "fileId": "2.yaml",
+      "publish": true,
+      "labels": [],
+      "commitId": "6c778b1f44200bd19944a6a8eac10a4e5a21a3cd"
+    }
+  ]
+}

test/projects/merge-operations/case4/result.yaml

@@ -0,0 +1,19 @@
+openapi: 3.1.0
+info:
+  title: manualGroup
+  version: v1
+paths:
+  /path1:
+    $ref: '#/components/pathItems/pathItem1'
+  /path2:
+    post:
+      responses:
+        '200':
+          description: response description main
+components:
+  pathItems:
+    pathItem1:
+      get:
+        responses:
+          '200':
+            description: response description main