Make sure mimeType has the type from the operation

mia-platform · Nov 13, 2017 · 918dfb7 · 918dfb7
1 parent bd27813
commit 918dfb7
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 20 deletions.
diff --git a/packages/api-explorer-ui/__tests__/lib/oas-to-har.test.js b/packages/api-explorer-ui/__tests__/lib/oas-to-har.test.js
@@ -288,34 +288,51 @@ describe('header values', () => {
   });
 });
 
-describe('body values', () => {
-  const pathOperation = {
-    path: '/body',
-    method: 'get',
-    requestBody: {
-      content: {
-        'application/json': {
-          schema: {
-            type: 'object',
-            properties: {
-              a: {
-                type: 'string',
-              },
+const pathOperation = {
+  path: '/body',
+  method: 'get',
+  requestBody: {
+    content: {
+      'application/json': {
+        schema: {
+          type: 'object',
+          properties: {
+            a: {
+              type: 'string',
             },
           },
         },
       },
     },
-  };
-  it('should not add on empty unrequired values', () => {
-    expect(oasToHar({}, pathOperation).log.entries[0].request.postData.text).toEqual(undefined);
-  });
-  it('should pass in mimeType', () => {
+  },
+};
+
+describe('mimeType', () => {
+  it('should default to application/json', () => {
     expect(oasToHar({}, pathOperation).log.entries[0].request.postData.mimeType).toEqual(
       'application/json',
     );
   });
 
+  it('should fetch mimeType from the operation', () => {
+    expect(
+      oasToHar(
+        {},
+        Object.assign({}, pathOperation, {
+          requestBody: {
+            content: { 'application/xml': pathOperation.requestBody.content['application/json'] },
+          },
+        }),
+      ).log.entries[0].request.postData.mimeType,
+    ).toEqual('application/xml');
+  });
+});
+
+describe('body values', () => {
+  it('should not add on empty unrequired values', () => {
+    expect(oasToHar({}, pathOperation).log.entries[0].request.postData.text).toEqual(undefined);
+  });
+
   // TODO extensions[SEND_DEFAULTS]
   it.skip('should set defaults if no value provided but is required', () => {
     expect(

diff --git a/packages/api-explorer-ui/src/lib/oas-to-har.js b/packages/api-explorer-ui/src/lib/oas-to-har.js
@@ -133,10 +133,11 @@ module.exports = (
   // Add content-type header if there are any body values setup above ^^
   // or if there is a schema defined
   if (har.postData.text || Object.keys(schema.schema).length) {
-    har.postData.mimeType = getContentType();
+    const type = getContentType(pathOperation);
+    har.postData.mimeType = type;
     har.headers.push({
       name: 'Content-Type',
-      value: getContentType(pathOperation),
+      value: type,
     });
   }