Merge pull request #19 from DavidBiesack/issue-18-remove-contentMedia…

…Type strip contentMediaType from schemas; add test
apiture · Mar 13, 2024 · 997ed6f · 997ed6f
2 parents e0dfbdf + a10d09e
commit 997ed6f
Show file tree

Hide file tree

Showing 5 changed files with 50 additions and 3 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@apiture/openapi-down-convert",
-  "version": "0.11.0",
+  "version": "0.12.0",
   "description": "Tool to down convert OpenAPI 3.1 to OpenAPI 3.0",
   "main": "lib/src/index.js",
   "bin": {

diff --git a/src/converter.ts b/src/converter.ts
@@ -251,7 +251,7 @@ export class Converter {
     }
   }
   removeUnsupportedSchemaKeywords() {
-    const keywordsToRemove = ['$id', '$schema', 'unevaluatedProperties'];
+    const keywordsToRemove = ['$id', '$schema', 'unevaluatedProperties', 'contentMediaType'];
     const schemaVisitor: SchemaVisitor = (schema: SchemaObject): SchemaObject => {
       keywordsToRemove.forEach((key) => {
         if (schema.hasOwnProperty(key)) {

diff --git a/test/converter.spec.ts b/test/converter.spec.ts
@@ -355,6 +355,47 @@ describe('resolver test suite', () => {
     done();
   });
 
+  test('Remove contentMediaType keywords', (done) => {
+    const input = {
+      openapi: '3.1.0',
+      components: {
+        schemas: {
+          a: {
+            type: 'object',
+            unevaluatedProperties: false,
+            properties: {
+              b: {
+                type: 'string',
+                contentMediaType: 'application/pdf',
+                maxLength: 5000000
+              },
+            },
+          },
+        },
+      },
+    };
+    const expected = {
+      openapi: '3.0.3',
+      components: {
+        schemas: {
+          a: {
+            type: 'object',
+            properties: {
+              b: {
+                type: 'string',
+                maxLength: 5000000
+              },
+            },
+          },
+        },
+      },
+    };
+    const converter = new Converter(input, { verbose: true });
+    const converted: any = converter.convert();
+    expect(converted).toEqual(expected);
+    done();
+  });
+
 
    test('Remove webhooks object', (done) => {
     const input = {

diff --git a/test/data/openapi.yaml b/test/data/openapi.yaml
@@ -63,6 +63,12 @@ paths:
             application/json:
               schema:
                 $ref: '#/components/schemas/preferences'
+            application/pdf:
+              schema:
+                type: string
+                contentMediaType: application/json
+                contentEncoding: base64
+                maxLength: 5000000
         '400':
           $ref: '#/components/responses/400'
         '401':