Skip to content

Commit 36afa77

Browse files
Ensure getFeatureSettingEnabled behaves as expected
1 parent 847694d commit 36afa77

File tree

1 file changed

+24
-0
lines changed

1 file changed

+24
-0
lines changed

injected/unit-test/content-feature.js

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,15 @@ describe('ContentFeature class', () => {
99
expect(this.getFeatureSetting('otherTest')).toBe('enabled');
1010
expect(this.getFeatureSetting('otherOtherTest')).toBe('ding');
1111
expect(this.getFeatureSetting('arrayTest')).toBe('enabledArray');
12+
// Following key doesn't exist so it should return false
13+
expect(this.getFeatureSettingEnabled('someNonExistantKey')).toBe(false);
14+
expect(this.getFeatureSettingEnabled('disabledStatus')).toBe(false);
15+
expect(this.getFeatureSettingEnabled('internalStatus')).toBe(false);
16+
expect(this.getFeatureSettingEnabled('enabledStatus')).toBe(true);
17+
expect(this.getFeatureSettingEnabled('overridenStatus')).toBe(false);
18+
expect(this.getFeatureSettingEnabled('disabledOverridenStatus')).toBe(true);
19+
expect(this.getFeatureSettingEnabled('statusObject')).toBe(true);
20+
expect(this.getFeatureSettingEnabled('statusDisabledObject')).toBe(false);
1221
didRun = true;
1322
}
1423
}
@@ -23,12 +32,27 @@ describe('ContentFeature class', () => {
2332
otherTest: 'disabled',
2433
otherOtherTest: 'ding',
2534
arrayTest: 'enabled',
35+
disabledStatus: 'disabled',
36+
internalStatus: 'internal', // not currently supported
37+
enabledStatus: 'enabled',
38+
overridenStatus: 'enabled',
39+
disabledOverridenStatus: 'disabled',
40+
statusObject: {
41+
state: 'enabled',
42+
bloop: true,
43+
},
44+
statusDisabledObject: {
45+
state: 'disabled',
46+
bloop2: true,
47+
},
2648
domains: [
2749
{
2850
domain: 'example.com',
2951
patchSettings: [
3052
{ op: 'replace', path: '/test', value: 'enabled2' },
3153
{ op: 'replace', path: '/otherTest', value: 'enabled' },
54+
{ op: 'replace', path: '/overridenStatus', value: 'disabled' },
55+
{ op: 'replace', path: '/disabledOverridenStatus', value: 'enabled' },
3256
],
3357
},
3458
{

0 commit comments

Comments
 (0)