Remove any paths that had all their verbs removed.

Signed-off-by: dblock <dblock@amazon.com>

tools/src/merger/OpenApiVersionExtractor.ts

@@ -7,7 +7,7 @@
 * compatible open source license.
 */
 
-import _, { extend } from 'lodash'
+import _, { extend, isEmpty } from 'lodash'
 import { delete_matching_keys, find_refs, write_yaml } from '../helpers'
 import { Logger } from '../Logger'
 import { type OpenAPIV3 } from 'openapi-types'
@@ -98,5 +98,7 @@ export default class OpenApiVersionExtractor {
     delete_matching_keys(this._spec, (obj) =>
       obj.$ref !== undefined && !_.includes(remaining, obj.$ref)
     )
+
+    this._spec.paths = _.omitBy(this._spec.paths, isEmpty)
   }
 }

tools/tests/merger/fixtures/extractor/expected_2.0.yaml

@@ -31,6 +31,15 @@ paths:
           $ref: '#/components/responses/nodes.info@200'
         '201':
           $ref: '#/components/responses/nodes.info@201'
+  /cluster_manager:
+    get:
+      operationId: cluster_manager.0
+      x-version-added: '2.0'
+      parameters: []
+    post:
+      operationId: cluster_manager.0
+      x-version-added: '2.0'
+      parameters: []
   /index:
     get:
       operationId: get.0

tools/tests/merger/fixtures/extractor/expected_default.yaml

@@ -31,6 +31,15 @@ paths:
           $ref: '#/components/responses/nodes.info@200'
         '201':
           $ref: '#/components/responses/nodes.info@201'
+  /cluster_manager:
+    get:
+      operationId: cluster_manager.0
+      x-version-added: '2.0'
+      parameters: []
+    post:
+      operationId: cluster_manager.0
+      x-version-added: '2.0'
+      parameters: []
   /index:
     get:
       operationId: get.0

tools/tests/tester/MergedOpenApiSpec.test.ts

@@ -22,6 +22,7 @@ describe('merged API spec', () => {
     test('paths', () => {
       expect(spec.paths()).toEqual({
         '/_nodes/{id}': ['get', 'post'],
+        '/cluster_manager': ['get', 'post'],
         '/index': ['get'],
         '/nodes': ['get']
       })

tools/tests/tester/ResultLogger.test.ts

@@ -75,7 +75,7 @@ describe('ConsoleResultLogger', () => {
 
       expect(log.mock.calls).toEqual([
         [],
-        ['Tested 1/4 paths.']
+        ['Tested 1/6 paths.']
       ])
     })
   })

tools/tests/tester/TestResults.test.ts

@@ -39,16 +39,16 @@ describe('TestResults', () => {
   })
 
   test('spec_paths_count', () => {
-    expect(test_results.spec_paths_count()).toEqual(4)
+    expect(test_results.spec_paths_count()).toEqual(6)
   })
 
   test('write_coverage', () => {
     const filename = 'coverage.json'
     test_results.write_coverage(filename)
     expect(JSON.parse(fs.readFileSync(filename, 'utf8'))).toEqual({
       evaluated_paths_count: 1,
-      evaluated_paths_pct: 25,
-      paths_count: 4
+      evaluated_paths_pct: 16.67,
+      paths_count: 6
     })
     fs.unlinkSync(filename)
   })

tools/tests/tester/fixtures/specs/complete/namespaces/cluster_manager.yaml

@@ -0,0 +1,17 @@
+openapi: 3.1.0
+info:
+  title: OpenSearch API
+  description: OpenSearch API
+  version: 1.0.0
+paths:
+  /cluster_manager:
+    get:
+      operationId: cluster_manager.0
+      x-version-added: '2.0'
+    post:
+      operationId: cluster_manager.0
+      x-version-added: '2.0'
+components:
+  requestBodies: []
+  parameters: []
+  responses: []