Skip to content

Commit db508ec

Browse files
committed
remove config overrides from fixtures process
1 parent 8909b3c commit db508ec

4 files changed

Lines changed: 0 additions & 239 deletions

File tree

dotcom-rendering/fixtures/config-overrides.js

Lines changed: 0 additions & 25 deletions
This file was deleted.

dotcom-rendering/fixtures/config.js

Lines changed: 0 additions & 167 deletions
This file was deleted.

dotcom-rendering/fixtures/switch-overrides.js

Lines changed: 0 additions & 28 deletions
This file was deleted.

dotcom-rendering/scripts/test-data/gen-fixtures.js

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,6 @@
44
const { execFileSync } = require('node:child_process');
55
const fs = require('node:fs/promises');
66
const { resolve } = require('node:path');
7-
const { config } = require('../../fixtures/config');
8-
const { configOverrides } = require('../../fixtures/config-overrides');
9-
const { switchOverrides } = require('../../fixtures/switch-overrides');
107
const {
118
validateAsCricketMatchPageType,
129
validateAsFEArticle,
@@ -156,22 +153,6 @@ const requests = articles.map((article) => {
156153
return fetch(`${article.url}.json?dcr`)
157154
.then((res) => res.json())
158155
.then((frontendJson) => {
159-
// Override config
160-
frontendJson.config = { ...config, ...configOverrides };
161-
// Override switches
162-
frontendJson.config.switches = {
163-
...frontendJson.config.switches,
164-
...switchOverrides,
165-
};
166-
167-
// Override this config property but only for Labs articles
168-
// TODO: Remove this once we are fully typing the config property
169-
// and no longer need to use a fixed `config.js` object to replace
170-
// the live one
171-
if (frontendJson.format.theme === 'Labs') {
172-
frontendJson.config.isPaidContent = true;
173-
}
174-
175156
// Manual hack for LiveBlog vs DeadBlog
176157
if (
177158
article.name === 'Live' ||

0 commit comments

Comments
 (0)