preferredRequestBodyType option- Set default to 'first-listed" and remove fallback

Author: thim81

libV2/schemaUtils.js

@@ -1646,7 +1646,7 @@ let QUERYPARAM = 'query',
       rawModeRequestBody;
 
     const { preferredRequestBodyType: optionRequestBodyType } = context.computedOptions,
-      preferredRequestBodyType = optionRequestBodyType || 'x-www-form-urlencoded';
+      preferredRequestBodyType = optionRequestBodyType || 'first-listed';
 
     if (!requestBody) {
       return requestBody;
@@ -1694,22 +1694,9 @@ let QUERYPARAM = 'query',
       else if (preferredRequestBodyType === 'form-data' && formDataRequestBody) {
         return formDataRequestBody;
       }
-      else if (preferredRequestBodyType === 'raw' && rawModeRequestBody) {
-        return rawModeRequestBody;
-      }
     }
 
-    // Legacy fallback
-    // If preferredRequestBodyType is not provided, return the first available request body
-    if (encodedRequestBody) {
-      return encodedRequestBody;
-    }
-    else if (formDataRequestBody) {
-      return formDataRequestBody;
-    }
-    else {
-      return rawModeRequestBody;
-    }
+    return rawModeRequestBody;
   },
 
   resolvePathItemParams = (context, operationParam, pathParam) => {

test/system/structure.test.js

@@ -241,7 +241,7 @@ const optionIds = [
       description: 'When there are multiple content-types defined in the request body of OpenAPI, the conversion ' +
         'selects the preferred option content-type as request body.If "first-listed" is set, the first ' +
         'content-type defined in the OpenAPI spec will be selected.',
-      external: true,
+      external: false,
       usage: ['CONVERSION']
     }
   };

test/unit/schemaUtilsV2.test.js

@@ -86,7 +86,7 @@ const {
 
 describe('resolveRequestBodyForPostmanRequest function', function () {
 
-  it('should return encoded request body when preferredRequestBodyType is not set', function () {
+  it('should return first-listed request body when preferredRequestBodyType is not set', function () {
     const contextTest = {
         concreteUtils,
         schemaCache: {},
@@ -98,10 +98,10 @@ describe('resolveRequestBodyForPostmanRequest function', function () {
       operationItemTest = operationItem.put,
       result = resolveRequestBodyForPostmanRequest(contextTest, operationItemTest);
 
-    expect(result.body.mode).to.equal('urlencoded');
+    expect(result.body.mode).to.equal('raw');
   });
 
-  it('should return encoded request body as fallback when preferredRequestBodyType is not a valid option', function () {
+  it('should return first-listed request body when preferredRequestBodyType is not a valid option', function () {
     const contextTest = {
         concreteUtils,
         schemaCache: {},
@@ -114,7 +114,7 @@ describe('resolveRequestBodyForPostmanRequest function', function () {
       operationItemTest = operationItem.put,
       result = resolveRequestBodyForPostmanRequest(contextTest, operationItemTest);
 
-    expect(result.body.mode).to.equal('urlencoded');
+    expect(result.body.mode).to.equal('raw');
   });
 
   it('should return encoded request body when preferredRequestBodyType is x-www-form-urlencoded', function () {