From 7aa61e89982438931a984e00e8bca2ce8bbe740f Mon Sep 17 00:00:00 2001 From: Chloe Herd Date: Tue, 19 Sep 2023 12:50:09 -0400 Subject: [PATCH] Runexperiments --- src/router/index.js | 28 ++++++++++++++++++---------- src/store/index.js | 7 ++++++- src/store/store.spec.js | 10 ++++++++-- 3 files changed, 32 insertions(+), 13 deletions(-) diff --git a/src/router/index.js b/src/router/index.js index e8fbf4d26..b49b38d9f 100644 --- a/src/router/index.js +++ b/src/router/index.js @@ -394,18 +394,26 @@ function arePagePrerequisitesValid(component) { // Run SiteEntry and PageEntry triggers for experiments async function runExperiments(nextPage) { if (!store.getters.applicationUser.triggeredSiteEntry) { - await baseMixin.methods.dispatchStoreAction(storeActions.RUN_EXPERIMENTS_FOR_TRIGGER, { - userId: getDeviceIdValue(), - triggerEvent: experimentTriggers.SITE_ENTRY, - triggerValue: applicationConfig.SITE_ENTRY_TRIGGER_VALUE, - }); + await baseMixin.methods.dispatchStoreActionWithLogging( + storeActions.RUN_EXPERIMENTS_FOR_TRIGGER, + { + userId: getDeviceIdValue(), + triggerEvent: experimentTriggers.SITE_ENTRY, + triggerValue: applicationConfig.SITE_ENTRY_TRIGGER_VALUE, + }, + nextPage + ); } - await baseMixin.methods.dispatchStoreAction(storeActions.RUN_EXPERIMENTS_FOR_TRIGGER, { - userId: getDeviceIdValue(), - triggerEvent: experimentTriggers.PAGE_ENTRY, - triggerValue: nextPage, - }); + await baseMixin.methods.dispatchStoreActionWithLogging( + storeActions.RUN_EXPERIMENTS_FOR_TRIGGER, + { + userId: getDeviceIdValue(), + triggerEvent: experimentTriggers.PAGE_ENTRY, + triggerValue: nextPage, + }, + nextPage + ); } export default router; diff --git a/src/store/index.js b/src/store/index.js index a8cfe77fd..50124bee7 100644 --- a/src/store/index.js +++ b/src/store/index.js @@ -1083,7 +1083,10 @@ export const actions = { }); }, - async runExperimentsForTrigger(context, { userId, triggerEvent, triggerValue }) { + async runExperimentsForTrigger( + context, + { payload: { userId, triggerEvent, triggerValue }, pageNameToLog } + ) { if (triggerEvent == experimentTriggers.SITE_ENTRY) { context.commit(storeMutations.UPDATE_TRIGGERED_SITE_ENTRY, true); } @@ -1100,6 +1103,8 @@ export const actions = { method: endpoints.RunExperimentsForTrigger.method, endpoint: endpoints.RunExperimentsForTrigger.url, payload: payload, + logApiCall: true, + pageNameToLog: pageNameToLog, }); context.commit(storeMutations.UPDATE_EXPERIMENTS, response.data.experiments); diff --git a/src/store/store.spec.js b/src/store/store.spec.js index a0491f47c..611341aa7 100644 --- a/src/store/store.spec.js +++ b/src/store/store.spec.js @@ -1751,7 +1751,10 @@ describe("Actions", () => { // Act await actions.runExperimentsForTrigger(context, { - triggerEvent: experimentTriggers.SITE_ENTRY, + payload: { + triggerEvent: experimentTriggers.SITE_ENTRY, + }, + pageNameToLog: "test", }); // Assert @@ -1785,7 +1788,10 @@ describe("Actions", () => { // Act await actions.runExperimentsForTrigger(context, { - triggerEvent: "NotSiteEntry", + payload: { + triggerEvent: "NotSiteEntry", + }, + pageNameToLog: "test", }); // Assert