Compare commits

...
Author SHA1 Message Date
abeatrix 290798ef0f Posthog: add test env config
- Remove redundant "none" case and error logging in TelemetryProviderFactory
- Add test environment configuration for PostHog with disabled API keys
- Enhance environment detection logic to support test and local environments
- Simplify default case handling to always return NoOpTelemetryProvider
2025-09-04 15:05:01 -07:00
2 changed files with 11 additions and 5 deletions
@@ -33,10 +33,7 @@ export class TelemetryProviderFactory {
return new PostHogTelemetryProvider(sharedClient)
}
return new NoOpTelemetryProvider()
case "none":
return new NoOpTelemetryProvider()
default:
console.error(`Unsupported telemetry provider type: ${config.type}`)
return new NoOpTelemetryProvider()
}
}
+11 -2
View File
@@ -26,5 +26,14 @@ const posthogDevEnvConfig = {
uiHost: "https://us.i.posthog.com",
} satisfies PostHogClientConfig
// NOTE: Ensure that dev environment is used when process.env.IS_DEV is "true"
export const posthogConfig = process.env.IS_DEV === "true" ? posthogDevEnvConfig : posthogProdConfig
const posthogTestEnvConfig = {
...posthogDevEnvConfig,
apiKey: undefined,
errorTrackingApiKey: undefined,
} satisfies PostHogClientConfig
const isTestEnv = process.env.IS_TEST === "true" || process.env.E2E_TEST
const isDevEnv = process.env.IS_DEV === "true" || process.env.CLINE_ENVIRONMENT === "local"
// NOTE: Ensure the correct config is used based on environment
export const posthogConfig = isTestEnv ? posthogTestEnvConfig : isDevEnv ? posthogDevEnvConfig : posthogProdConfig