Skip to content

Commit

Permalink
Add unique id to each configuration.
Browse files Browse the repository at this point in the history
The id is used in the report.json files and before this change
all files have the same id inherited from the common config.

By adding a unique id, we can use the report.json files directly
and send metrics to our time series database.
  • Loading branch information
soulgalore committed Sep 27, 2024
1 parent 4698c94 commit d4686aa
Show file tree
Hide file tree
Showing 8 changed files with 8 additions and 0 deletions.
1 change: 1 addition & 0 deletions config/configCampaignEvents.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ const scenarios = tests.map( ( test ) => {

module.exports = {
...configCommon,
id: 'campaign-events',
onBeforeScript: 'puppet/onBefore.js',
onReadyScript: 'puppet/onReady.js',
viewports: [
Expand Down
1 change: 1 addition & 0 deletions config/configCodex.js
Original file line number Diff line number Diff line change
Expand Up @@ -167,6 +167,7 @@ const scenarios = components.flatMap( ( componentData ) => {

module.exports = {
...configCommon,
id: 'codex',
onReadyScript: 'puppet/onReady-codex.js',
viewports: [
VIEWPORT_PHONE,
Expand Down
1 change: 1 addition & 0 deletions config/configDesktop.js
Original file line number Diff line number Diff line change
Expand Up @@ -272,6 +272,7 @@ const scenarios = tests.map( ( test ) => {

module.exports = {
...configCommon,
id: 'desktop',
onBeforeScript: 'puppet/onBefore.js',
onReadyScript: 'puppet/onReady.js',
viewports: [
Expand Down
1 change: 1 addition & 0 deletions config/configEcho.js
Original file line number Diff line number Diff line change
Expand Up @@ -126,6 +126,7 @@ const makeMobileAndDesktopScenarios = ( tests ) => {
const scenarios = makeMobileAndDesktopScenarios( baseScenarios );

module.exports = Object.assign( {}, configDesktop, {
id: 'echo',
scenarios,
paths: utils.makePaths( 'echo' )
} );
1 change: 1 addition & 0 deletions config/configLogin.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ const scenarios = utils.makeScenariosForSkins( [

module.exports = {
...configCommon,
id: 'login',
onBeforeScript: 'puppet/onBefore.js',
onReadyScript: 'puppet/onReady.js',
viewports: [
Expand Down
1 change: 1 addition & 0 deletions config/configMobile.js
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,7 @@ const scenarios = tests.map( ( test ) => {
} );

module.exports = Object.assign( {}, configDesktop, {
id: 'mobile',
scenarios,
paths: utils.makePaths( 'mobile' )
} );
1 change: 1 addition & 0 deletions config/configWebMaintained.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ const nearbyScenarios = utils.makeScenariosForSkins(
);

module.exports = Object.assign( {}, configDesktop, {
id: 'web-maintained',
scenarios: skinScenarios.concat( quickSurveyScenarios ).concat(
nearbyScenarios
),
Expand Down
1 change: 1 addition & 0 deletions config/configWikiLambda.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ const scenarios = tests.map( ( test ) => {

module.exports = {
...configCommon,
id: 'wikilambda',
onBeforeScript: 'puppet/onBefore.js',
onReadyScript: 'puppet/onReady.js',
viewports: [
Expand Down

0 comments on commit d4686aa

Please sign in to comment.