Support arrays of domains for patching settings (#506)

Author: jonathanKingston

src/content-feature.js

@@ -147,6 +147,11 @@ export default class ContentFeature {
         if (!domain) return []
         const domains = this._getFeatureSetting()?.[featureKeyName] || []
         return domains.filter((rule) => {
+            if (Array.isArray(rule.domain)) {
+                return rule.domain.some((domainRule) => {
+                    return matchHostname(domain, domainRule)
+                })
+            }
             return matchHostname(domain, rule.domain)
         })
     }

unit-test/content-feature.js

@@ -8,6 +8,7 @@ describe('Helpers checks', () => {
                 expect(this.getFeatureSetting('test')).toBe('enabled3')
                 expect(this.getFeatureSetting('otherTest')).toBe('enabled')
                 expect(this.getFeatureSetting('otherOtherTest')).toBe('ding')
+                expect(this.getFeatureSetting('arrayTest')).toBe('enabledArray')
                 didRun = true
             }
         }
@@ -21,6 +22,7 @@ describe('Helpers checks', () => {
                     test: 'enabled',
                     otherTest: 'disabled',
                     otherOtherTest: 'ding',
+                    arrayTest: 'enabled',
                     domains: [
                         {
                             domain: 'example.com',
@@ -34,6 +36,12 @@ describe('Helpers checks', () => {
                             patchSettings: [
                                 { op: 'replace', path: '/test', value: 'enabled3' }
                             ]
+                        },
+                        {
+                            domain: ['meep.com', 'example.com'],
+                            patchSettings: [
+                                { op: 'replace', path: '/arrayTest', value: 'enabledArray' }
+                            ]
                         }
                     ]
                 }