Don't test all new combinations

We already had 24 combinations before.
Testing each with and without seems overkill compared to the increase in test duration.

Author: eps1lon

test/integration/create-next-app/templates/matrix.test.ts

@@ -33,26 +33,30 @@ describe.each(['app', 'pages'] as const)(
       // eslint: ['--eslint', '--no-linter'],
       eslint: ['--eslint'],
 
-      srcDir: ['--src-dir', '--no-src-dir'],
+      // Trading test perf for robustness:
+      // srcDir and reactCompiler don't interact so we're testing them together
+      // instead of all permutations.
+      srcDirAndCompiler: [
+        '--src-dir --react-compiler',
+        '--no-src-dir --no-react-compiler',
+      ],
       tailwind: ['--tailwind', '--no-tailwind'],
 
       // shouldn't affect if the app builds or not
       // packageManager: ['--use-npm', '--use-pnpm', '--use-yarn', '--use-bun'],
-
-      reactCompiler: ['--react-compiler', '--no-react-compiler'],
     }
 
-    const getPermutations = <T>(items: T[][]): T[][] => {
+    const getCombinations = (items: string[][]): string[][] => {
       if (!items.length) return [[]]
       const [first, ...rest] = items
-      const children = getPermutations(rest)
+      const children = getCombinations(rest)
       return first.flatMap((value) =>
-        children.map((child) => [value, ...child])
+        children.map((child) => [...value.split(' '), ...child])
       )
     }
 
-    const flagPermutations = getPermutations(Object.values(allFlagValues))
-    const testCases = flagPermutations.map((flags) => ({
+    const flagCombinations = getCombinations(Object.values(allFlagValues))
+    const testCases = flagCombinations.map((flags) => ({
       name: flags.join(' '),
       flags,
     }))