Improve config-tests.js errors

eslint.config.mjs

@@ -14,6 +14,7 @@ export default [
                 ...globals.commonjs,
                 ...globals.browser,
                 ...globals.jasmine,
+                ...globals.mocha,
                 ...globals.node,
             },
         },

overrides/android-override.json

@@ -14,7 +14,7 @@
                             }
                         ]
                     },
-                    "cohorts": [
+                    "cohort": [
                         {
                             "name": "control",
                             "weight": 0

tests/config-tests.js

@@ -30,6 +30,16 @@ const previousConfigs = platformOutput.map((plat) => {
 describe('Config schema tests', () => {
     for (const config of latestConfigs) {
         describe(`${config.name}`, () => {
+            before(() => {
+                const validate = createValidator(platformSpecificSchemas[config.name] || 'GenericV4Config');
+                const validateResult = validate(config.body);
+                if (!validateResult) {
+                    throw new Error(
+                        `JSON Schema validation failed:\n${formatErrors(validate.errors)} Check ${config.name} output against the schema/`,
+                    );
+                }
+            });
+
             // appTrackerProtection should only be on the Android config since it is a large feature
             const shouldContainAppTP = config.name.split('/')[1] === 'android-config.json';
             it('should contain appTrackerProtection or not', () => {
@@ -39,11 +49,6 @@ describe('Config schema tests', () => {
                 );
             });
 
-            it('should validate against the full configV4 schema', () => {
-                const validate = createValidator(platformSpecificSchemas[config.name] || 'GenericV4Config');
-                expect(validate(config.body)).to.be.equal(true, formatErrors(validate.errors));
-            });
-
             it('all features should be named correctly', () => {
                 const legacyFeatures = [
                     'androidBrowserConfig',
@@ -93,7 +98,7 @@ describe('Config schema tests', () => {
                 }
             });
 
-            it('All patchSettings should also be valid', () => {
+            it('All patchSettings should also be valid', function () {
                 const validate = createValidator(platformSpecificSchemas[config.name] || 'GenericV4Config');
                 function applyPatchAndValidate(featureName, feature, conditionalChange, config) {
                     for (const change of conditionalChange) {
@@ -155,7 +160,7 @@ describe('Config schema tests', () => {
                 );
             });
 
-            it('should validate against the legacy schema', () => {
+            it('should validate against the legacy schema', function () {
                 const validate = createValidator(platformSpecificSchemas[config.name] || 'LegacyConfig');
                 expect(validate(config.body)).to.be.equal(true, formatErrors(validate.errors));
             });