test(ssr): remove SSR tests env var

nolanlawson · nolanlawson · commit 3ed796dc4da4 · 2025-01-17T09:41:06.000-08:00
diff --git a/.github/workflows/unit.yml b/.github/workflows/unit.yml
@@ -66,9 +66,6 @@ jobs:
               run: yarn test:types
             - name: Run unit tests
               run: yarn test:ci
-            # TODO [#4815]: enable all SSR v2 tests
-            - name: Run experimental SSR fixture tests
-              run: TEST_SSR_COMPILER=1 yarn test packages/@lwc/ssr-compiler/src/__tests__/fixtures.spec.ts
             - name: Upload unit test coverage report
               uses: actions/upload-artifact@v4
               with:
diff --git a/packages/@lwc/ssr-compiler/src/__tests__/fixtures.spec.ts b/packages/@lwc/ssr-compiler/src/__tests__/fixtures.spec.ts
@@ -77,9 +77,7 @@ async function compileFixture({ input, dirname }: { input: string; dirname: stri
     return outputFile;
 }
 
-// We will enable this for realsies once all the tests are passing, but for now having the env var avoids
-// running these tests in CI while still allowing for local testing.
-describe.runIf(process.env.TEST_SSR_COMPILER).concurrent('fixtures', () => {
+describe.concurrent('fixtures', () => {
     testFixtureDir(
         {
             root: path.resolve(__dirname, '../../../engine-server/src/__tests__/fixtures'),

Original file line number	Diff line number	Diff line change
`@@ -77,9 +77,7 @@ async function compileFixture({ input, dirname }: { input: string; dirname: stri`
`77`	`77`	`return outputFile;`
`78`	`78`	`}`
`79`	`79`
`80`		`-// We will enable this for realsies once all the tests are passing, but for now having the env var avoids`
`81`		`-// running these tests in CI while still allowing for local testing.`
`82`		`-describe.runIf(process.env.TEST_SSR_COMPILER).concurrent('fixtures', () => {`
	`80`	`+describe.concurrent('fixtures', () => {`
`83`	`81`	`testFixtureDir(`
`84`	`82`	`{`
`85`	`83`	`root: path.resolve(__dirname, '../../../engine-server/src/__tests__/fixtures'),`