fix: default flag to true for testing

Skn0tt · Skn0tt · commit bb4d4cf519e0 · 2023-05-03T12:04:57.000+02:00
diff --git a/packages/runtime/src/helpers/flags.ts b/packages/runtime/src/helpers/flags.ts
@@ -13,5 +13,6 @@ export const splitApiRoutes = (featureFlags: Record<string, unknown>): boolean =
   if (process.env.NEXT_SPLIT_API_ROUTES) {
     return process.env.NEXT_SPLIT_API_ROUTES === 'true'
   }
-  return Boolean(featureFlags.next_split_api_routes)
+  // default to true during testing, swap to false before merging
+  return typeof featureFlags.next_split_api_routes === 'boolean' ? featureFlags.next_split_api_routes : true
 }

Original file line number	Diff line number	Diff line change
`@@ -13,5 +13,6 @@ export const splitApiRoutes = (featureFlags: Record<string, unknown>): boolean =`
`13`	`13`	`if (process.env.NEXT_SPLIT_API_ROUTES) {`
`14`	`14`	`return process.env.NEXT_SPLIT_API_ROUTES === 'true'`
`15`	`15`	`}`
`16`		`- return Boolean(featureFlags.next_split_api_routes)`
	`16`	`+ // default to true during testing, swap to false before merging`
	`17`	`+ return typeof featureFlags.next_split_api_routes === 'boolean' ? featureFlags.next_split_api_routes : true`
`17`	`18`	`}`