fix: allow rules with flexible second options

* fix: customize ignore names for inconsistent property

* chore: update comment

Co-authored-by: Dustin Popp <[email protected]>

Author: csWen

src/cli-validator/utils/processConfiguration.js

@@ -363,6 +363,10 @@ const validateConfigOption = function(userOption, defaultOption) {
       optionType = option;
     }
   });
+  // if optionType doesn't match, there are no predefined options for this rule
+  if (!optionType) {
+    return result;
+  }
   // verify the given option is valid
   const validOptions = configOptions[optionType];
   if (!validOptions.includes(userOption)) {

test/plugins/validation/2and3/schema-ibm.test.js

@@ -1800,4 +1800,51 @@ describe('validation plugin - semantic - schema-ibm - OpenAPI 3', () => {
     expect(res.warnings.length).toEqual(0);
     expect(res.errors.length).toEqual(0);
   });
+
+  it('should not produce a warning for properties with duplicate custom names', () => {
+    const customConfig = {
+      schemas: {
+        inconsistent_property_type: ['warning', ['year']]
+      }
+    };
+
+    const spec = {
+      components: {
+        schemas: {
+          person: {
+            description: 'Produce warnings',
+            properties: {
+              year: {
+                description: 'type integer',
+                type: 'integer'
+              }
+            }
+          },
+          adult: {
+            description: 'Causes first warnings',
+            properties: {
+              year: {
+                description: 'different type',
+                type: 'number'
+              }
+            }
+          },
+          kid: {
+            description: 'Causes second warning',
+            properties: {
+              year: {
+                type: 'string',
+                description: 'differnt type'
+              }
+            }
+          }
+        }
+      }
+    };
+
+    const res = validate({ jsSpec: spec }, customConfig);
+
+    expect(res.warnings.length).toEqual(0);
+    expect(res.errors.length).toEqual(0);
+  });
 });