wip

Author: silesky

packages/signals/signals-integration-tests/src/helpers/base-page-object.ts

@@ -64,7 +64,7 @@ export class BasePage {
     await this.page.goto(url, { waitUntil: 'domcontentloaded' })
     if (!options.skipSignalsPluginInit) {
       void this.invokeAnalyticsLoad({
-        sandboxStrategy: envConfig.SANDBOX_STRATEGY ?? 'iframe',
+        sandboxStrategy: envConfig.SANDBOX_STRATEGY,
         flushInterval: 500,
         ...signalSettings,
       })

packages/signals/signals-integration-tests/src/helpers/env-config.ts

@@ -2,9 +2,8 @@ import { SignalsSettingsConfig } from '@segment/analytics-signals/dist/types/cor
 
 // This is for testing with the global sandbox strategy with an npm script, that executes processSignal in the global scope
 // If we change this to be the default, this can be rejiggered
-const SANDBOX_STRATEGY = process.env.SANDBOX_STRATEGY as
-  | SignalsSettingsConfig['sandboxStrategy']
-  | undefined
+const SANDBOX_STRATEGY = (process.env.SANDBOX_STRATEGY ??
+  'iframe') as SignalsSettingsConfig['sandboxStrategy']
 
 export const envConfig = {
   SANDBOX_STRATEGY,