Add test to validate that cohort names do not contain special charact…

…ers. (#2688)
duckduckgo · Jan 29, 2025 · 75d790f · 75d790f
1 parent 5c086ee
commit 75d790f
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/tests/config-tests.js b/tests/config-tests.js
@@ -109,6 +109,22 @@ describe('Config schema tests', () => {
                     }
                 }
             });
+
+            it('All experiment cohorts should be named correctly', () => {
+                const cohortNameRegex = /^[a-zA-Z0-9]+$/;
+                /** @type {Record<string, import('../schema/feature').GenericFeature>} */
+                const features = config.body.features;
+                for (const featureName of Object.keys(config.body.features)) {
+                    for (const subfeatureName of Object.keys(features[featureName].features || {})) {
+                        const subFeature = features[featureName].features[subfeatureName];
+                        if (subFeature.cohorts) {
+                            for (const cohort of subFeature.cohorts) {
+                                expect(cohort.name).to.match(cohortNameRegex);
+                            }
+                        }
+                    }
+                }
+            });
         });
     }