test(wtr): remove unnecessary support for .only files @W-19098252 (#5513)

* test(wtr): update tests to use relative path to utils

instead of weird fake module

* test(wtr): revert removing TestUtils

* test(wtr): remove useless describe

There's no setup/teardown needed, it's a single test, and WTR provides per-file encapsulation

* test(wtr): replace IIFE script with module import

* test(wtr): clean up SSR execution script

working toward just importing and executing things, but not quite there yet

* test(wtr): always use DISABLE_SYNTHETIC

two env vars for the same goal is unnecessary

* test(wtr): enable all hydration tests

I think the last one was a concurrency related timeout,
which was previously addressed.

* test(wtr): remove side effect from signals helper

helper files shouldn't have side effects; all setup should be in the setup file

* test(wtr): import directly from file, not from barrel exporter

* test(wtr): change bulk export statement to individual exports

* test(wtr): remove unnecessary aria re-export

* test(wtr): fix a few more ARIA util imports

* chore: move comment for nicer aesthetic

* test(wtr): remove unused option

* test(wtr): fix another ARIA util import

* test(wtr): import directly from hooks file rather than utils

* test(wtr): import directly from signals file rather than utils

* test(wtr): import directly from console helper rather than utils

* test(wtr): import directly from constants helper rather than utils

* test(wtr): only import what is needed from LWC

* test(wtr): split lwc:dynamic load helpers into separate file

* test(wtr): avoid relying on global LWC

* test(wtr): remove unnecessary wrapping of hydration test config

* test(wtr): remove test-utils logic from resolveImport

no longer necessary

* test(wtr): move plugin from shared config to only config that uses it

* test(wtr): remove useless guid

all tests are run in isolation

* test(wtr): remove unnecessary test-utils global

* test(wtr): clean up component definition

* test(wtr): change component in test from IIFE to import

* test(wtr): shift logic out of wrapper into static test runner file

we want to keep the generated wrappers as thin as possible

and use regular js files as much as possible

* test(wtr): make test setup/teardown more idiomatic

* test(wtr): clean up module compilation

* chore(wtr): make dependency on @vitest/spy explicit

* test(wtr): remove console suppression

It was originally implemented to suppress distracting warnings in Karma tests,
but WTR's log output is different / managed differently.

* test(wtr): clean up names and comments

* test(wtr): move and rename file

* test(wtr): remove usage of global TestUtils from hydration tests

* test(wtr): remove LWC import from utils

It breaks hydration tests if there's an import from LWC.

* test(wtr): remove unused external deps

* test(wtr): use imports instead of jasmine globals

* chore(eslint): remove unused rule config

* fix: attribute cte syntax in integration tests

* test(wtr): fix case in import

mismatched case passes locally, but causes issues in CI

* test(wtr): re-enable tests that used to fail in CI

they should all work now

* test(wtr): enable browser logs

* test(wtr): name plugins

helpful when debugging

* test(wtr): rename variables and document functions for clarity

* test(wtr): convert promise chains to async/await

* test(wtr): load code from known url, rather than sniffing script tag

* test(wtr): convert LWC from iife to fake cjs

* test(wtr): remove unnecessary wrapping

* test(wtr): data can be data, not a function

* test(wtr): replace `evaluate` with functions that require less wrapping

* test(wtr): only fetch scripts once

* test(wtr): john was here

* test(wtr): add line breaks and vars for clarity

* test(wtr): rename variable

it's just one code blob now, not multiple scripts

* test(wtr): fix param pass through

* test(wtr): more cleanup

* test(wtr): remove outdated comment

* test(wtr): avoid shadowing global var

ensures we never accidentally use the wrong one

* test(wtr): enable custom element registry tests

* test(wtr): hoist function definition

* chore: use vitest spy instead of custom hack

* test(wtr): use vitest spy instead of custom

* test(wtr): convert test to async/await

* test(wtr): use vitest spy instead of custom

* test(wtr): move util into only file that uses it

* test(wtr): remove .only file support for hydration tests

Web Test Runner lets you specify individual files via CLI, so the workaround is not necessary.

* test(wtr): disable browser logs for more readable output

* Revert "test(wtr): disable browser logs for more readable output"

This reverts commit 56a39c7.

---------

Co-authored-by: John Hefferman <jhefferman@salesforce.com>

Author: wjhsf

packages/@lwc/integration-not-karma/configs/plugins/serve-hydration.js

@@ -1,6 +1,5 @@
 import path from 'node:path';
 import vm from 'node:vm';
-import fs from 'node:fs/promises';
 import { fileURLToPath } from 'node:url';
 import { rollup } from 'rollup';
 import lwcRollupPlugin from '@lwc/rollup-plugin';
@@ -21,16 +20,6 @@ const ROOT_DIR = path.join(import.meta.dirname, '../..');
 const COMPONENT_NAME = 'x-main';
 const COMPONENT_ENTRYPOINT = 'x/main/main.js';
 
-// Like `fs.existsSync` but async
-async function exists(path) {
-    try {
-        await fs.access(path);
-        return true;
-    } catch (_err) {
-        return false;
-    }
-}
-
 async function compileModule(input, targetSSR, format) {
     const modulesDir = path.join(ROOT_DIR, input.slice(0, -COMPONENT_ENTRYPOINT.length));
     const bundle = await rollup({
@@ -106,15 +95,6 @@ async function getSsrMarkup(componentEntrypoint, configPath) {
     return await script.runInContext(vm.createContext({ LWC: lwcSsr }));
 }
 
-async function existsUp(dir, file) {
-    while (true) {
-        if (await exists(path.join(dir, file))) return true;
-        dir = path.join(dir, '..');
-        const basename = path.basename(dir);
-        if (basename === '.') return false;
-    }
-}
-
 /**
  * Hydration test `index.spec.js` files are actually config files, not spec files.
  * This function wraps those configs in the test code to be executed.
@@ -129,8 +109,6 @@ async function wrapHydrationTest(configPath) {
 
         const suiteDir = path.dirname(configPath);
         const componentEntrypoint = path.join(suiteDir, COMPONENT_ENTRYPOINT);
-        // You can add an `.only` file alongside an `index.spec.js` file to make the test focused
-        const onlyFileExists = await existsUp(suiteDir, '.only');
         const ssrOutput = await getSsrMarkup(componentEntrypoint, configPath);
 
         return `
@@ -139,8 +117,7 @@ async function wrapHydrationTest(configPath) {
         runTest(
             '/${configPath}?original=1',
             '/${componentEntrypoint}',
-            ${JSON.stringify(ssrOutput) /* escape quotes */},
-            ${onlyFileExists}
+            ${JSON.stringify(ssrOutput) /* escape quotes */}
         );
         `;
     } finally {

packages/@lwc/integration-not-karma/configs/plugins/test-hydration.js

@@ -45,8 +45,7 @@ function setFeatureFlags(requiredFeatureFlags, value) {
 }
 
 // Must be sync to properly register tests; async behavior can happen in before/after blocks
-export function runTest(configPath, componentPath, ssrRendered, focused) {
-    const test = focused ? it.only : it;
+export function runTest(configPath, componentPath, ssrRendered) {
     const description = new URL(configPath, location.href).pathname;
     let consoleSpy;
     let testConfig;
@@ -70,7 +69,7 @@ export function runTest(configPath, componentPath, ssrRendered, focused) {
         setFeatureFlags(testConfig.requiredFeatureFlags, false);
     });
 
-    test(description, async () => {
+    it(description, async () => {
         const container = appendTestTarget(ssrRendered);
         const selector = container.firstChild.tagName.toLowerCase();
         let target = container.querySelector(selector);