fix: fix issue of vendor-id being required parameter

lib/commands/smapi/cli-customization-processor.js

@@ -32,6 +32,10 @@ class CliCustomizationProcessor {
      * @param {Map} definitions Map with all Swagger definitions.
      */
     processParameter(parameter, parentOperation, definitions) {
+        const customization = customizationMap.get(parameter.name) || {};
+        parameter.skip = customization.skip || false;
+        if (parameter.skip) return;
+
         if (parameter.in === 'body') {
             this._processBodyParameter(parameter, parentOperation, definitions);
         } else {

lib/commands/smapi/customizations/hook-functions/append-vendor-id.js

@@ -8,9 +8,9 @@ const appendVendorId = (requestParameters, profile) => {
     const { createSkillRequest, createInSkillProductRequest } = requestParameters;
     const body = createSkillRequest || createInSkillProductRequest;
     const vendorId = AppConfig.getInstance().getVendorId(profile);
-    if (body && !body.vendorId) {
+    if (body) {
         body.vendorId = vendorId;
-    } else if (!requestParameters.vendorId) {
+    } else {
         requestParameters.vendorId = vendorId;
     }
 };

lib/commands/smapi/customizations/parameters.json

@@ -2,6 +2,9 @@
     "stageV2": {
         "name": "stage"
     },
+    "vendorId": {
+        "skip": true
+    },
     "createSkillRequest": {
         "skipUnwrap": true,
         "name": "manifest"

test/unit/commands/smapi/cli-customization-processor-test.js

@@ -96,6 +96,17 @@ describe('Smapi test - CliCustomizationProcessor class', () => {
             expect(flatParamsMap.get(key)).eql(expected);
         });
 
+        it('| should skip processing of parameter', () => {
+            const parameter = makeParameter();
+            const skip = true;
+
+            sinon.stub(customizationMap, 'get').returns({ skip });
+
+            processor.processParameter(parameter, parentOperation, definitions);
+
+            expect(parameter.skip).eql(true);
+        });
+
 
         it('| should add simple array of non body paramter', () => {
             const parameter = makeParameter({ type: 'array' });

test/unit/commands/smapi/hook-functions/append-vendor-id-test.js

@@ -38,15 +38,6 @@ describe('Smapi hook functions test - appendVendorId tests', () => {
         expect(requestParameters.vendorId).eql(vendorId);
     });
 
-    it('| should not append vendor id to requestParameters if already exists', () => {
-        const existingVendorId = 'some existing vendor id';
-        const requestParameters = { vendorId: existingVendorId };
-
-        appendVendorId(requestParameters, profile);
-
-        expect(requestParameters.vendorId).eql(existingVendorId);
-    });
-
     afterEach(() => {
         sinon.restore();
     });