test(wtr): get another pile of tests passing (#5425)

* refactor(wtr): move NODE_ENV_FOR_TEST default into options file

* chore(wtr): move hooks util to separate file

* chore(wtr): move aria utils to separate file

* chore(wtr): move constants to separate file

* chore(wtr): clean up custom rollup plugin

* test(wtr): get hydration tests kinda running

* test(wtr): kinda start moving to ESM instead of IIFE

* fix(shared): make sanitizeHtmlContent work

* chore(ci): run hydration tests in ci

* test(wtr): remove unused script

* test(wtr): implement missing jasmine assertions

* test(wtr): clean up test to make assertions more clear

* test(wtr): implement custom console matchers

* test(wtr): use simpler console spy

* test(wtr): remove outdated, unnecessary spies

* test(wtr): move signals util to own file

* test(wtr): implement custom error matchers

everything passed first try, which is kinda sus

* chore: split matchers into separate files

* chore: ugh tui go away

* chore: remove unused file

code is now in helpers/matchers/index.mjs

* test(wtr): conditionally import polyfills

reduces test failures from 291 to 220

* test(wtr): use one setup file instead of two

* chore: add comment explaining file

* chore(wtr): rename file

* chore(wtr): move test configs into directory

* chore(wtr): extract shared parts of config into base config object

* chore(wtr): rename config files

who needs convention?

* chore(wtr): move WTR plugins from helpers dir to config dir

* chore: move full plugins to plugin files

instead of just inner logic

* test(wtr): not all env vars are boolean

fixes ~50 test failures

* test(wtr): make tests async so they clean up properly

I'm not sure why the async matters, but it does.

* test(wtr): fix resolution of wire-service

* test(wtr): skip flaky tests

* test(wtr): add more jasmine spy adapter methods

* test(wtr): enable passing tests

turns out they just had browser log warnings, not failures

* test(wtr): make spy props non-enumerable

* test(wtr): remove unused matcher

* test(wtr): fix assertion to match error message

* test(wtr): implement jasmine.any

* test(wtr): mark test as flaky timeout

* update comment

* update comment

* test(wtr): move scripts to head instead of body

tests manipulate the body, so we use head for scripts to keep the body clean

* Update packages/@lwc/integration-not-karma/configs/base.mjs

* Update packages/@lwc/integration-not-karma/configs/base.mjs

* test(wtr): clean DOM after each test

* test(wtr): implement callFake jasmine adapter

* test(wtr): add missing argsFor jasmine adapter

* test(wtr): add missing toHaveBeenCalledOnceWith assertion

* test(wtr): categorize failures

* test(wtr): fix .toMatch and categorize failing files

* test(wtr): using detached objectContaining does not work

it breaks the required `this` reference

* test(wtr): add pre-test assertion checking for window focus

* test(wtr): eliminate concurrency

it messes with focus and breaks tests

* chore(deps): bump @web/test-runner to latest

* test(wtr): reset logged messages after each test

* chore(wtr): link potential workaround for concurrency issue

* test(wtr): reset logs between all tests

* test(wtr): fix more toMatch failures

* chore(deps): bump chai

Author: wjhsf

packages/@lwc/integration-karma/test/component/LightningElement.attachInternals/api/index.spec.js

@@ -39,7 +39,7 @@ const attachInternalsSanityTest = (tagName, ctor) => {
         });
 
         afterAll(() => {
-            document.body.removeChild(elm);
+            elm.remove();
         });
 
         it('should be able to create ElementInternals object', () => {

packages/@lwc/integration-karma/test/events/focus-event-related-target/index.spec.js

@@ -6,6 +6,7 @@ import Container from 'x/container';
 // So this test may "flap" during local development if your browser window isn't focused (e.g. you're focused on
 // the DevTools instead).
 it('should retarget relatedTarget', function () {
+    expect(document.hasFocus(), 'Browser window must be focused for test to work').toBe(true);
     const elm = createElement('x-container', { is: Container });
     document.body.appendChild(elm);
 

packages/@lwc/integration-karma/test/light-dom/slot-fowarding/scoped-slots/index.spec.js

@@ -10,7 +10,7 @@ describe('scoped slots, slot forwarding', () => {
     });
 
     afterAll(() => {
-        document.body.removeChild(lightContainer);
+        lightContainer.remove();
     });
 
     // TODO [#3889]: This test should be updated once a fix is ready.

packages/@lwc/integration-karma/test/light-dom/slot-fowarding/slots/forwarding/index.spec.js

@@ -57,7 +57,7 @@ describe('slot forwarding', () => {
     });
 
     afterAll(() => {
-        document.body.removeChild(lightContainer);
+        lightContainer.remove();
     });
 
     it('should correctly forward slot assignments - light > light slot', () => {

packages/@lwc/integration-karma/test/light-dom/slot-fowarding/slots/reactivity/index.spec.js

@@ -7,14 +7,14 @@ describe('light DOM slot forwarding reactivity', () => {
     let nodes;
     let lightContainer;
 
-    beforeAll(() => {
+    beforeEach(() => {
         lightContainer = createElement('x-light-container', { is: LightContainer });
         document.body.appendChild(lightContainer);
         nodes = extractDataIds(lightContainer);
     });
 
-    afterAll(() => {
-        document.body.removeChild(lightContainer);
+    afterEach(() => {
+        lightContainer.remove();
     });
 
     const verifySlotContent = (leaf, expected) => {

packages/@lwc/integration-karma/test/profiler/mutation-logging/index.spec.js

@@ -4,7 +4,8 @@ import Child from 'x/child';
 import GetterThrows from 'x/getterThrows';
 
 const arr = jasmine.arrayWithExactContents;
-const obj = jasmine.objectContaining;
+// `jasmine.objectContaining` is long, but the method can't be detached/aliased, ergo wrapper
+const obj = (val) => jasmine.objectContaining(val);
 
 let entries = [];
 

packages/@lwc/integration-karma/test/regression/invalid-key/index.spec.js

@@ -15,13 +15,12 @@ it('W-15885661 - renders list when key is invalid (preserve backwards compat)',
     elm.items = [{ value: 1 }];
     await Promise.resolve();
 
-    const {
-        calls: { error },
-    } = spy;
-    expect(error.length).toBe(process.env.NODE_ENV === 'production' ? 0 : 2);
-    error.forEach((error) =>
-        expect(error).toMatch(/(Invalid key value.*|Invalid "key" attribute.*)/)
-    );
+    const errorCalls = spy.calls.error;
+    expect(errorCalls.length).toBe(process.env.NODE_ENV === 'production' ? 0 : 2);
+    errorCalls.forEach(([error]) => {
+        expect(error).toBeInstanceOf(Error);
+        expect(error.message).toMatch(/(Invalid key value.*|Invalid "key" attribute.*)/);
+    });
 
     spy.reset();
 

packages/@lwc/integration-karma/test/rendering/sanitize-stylesheet-token/index.spec.js

@@ -89,7 +89,8 @@ props.forEach((prop) => {
                         } else {
                             // dev mode
                             expect(logger).toHaveBeenCalledTimes(1);
-                            expect(logger.calls.allArgs()[0]).toMatch(
+                            const loggedError = logger.calls.allArgs()[0][0];
+                            expect(loggedError.message).toMatch(
                                 new RegExp(
                                     `Mutating the "${prop}" property on a template is deprecated`
                                 )

packages/@lwc/integration-karma/test/static-content/index.spec.js

@@ -819,7 +819,7 @@ describe('text containing comments', () => {
         describe(`preserveComments ${preserveComments}`, () => {
             let elm;
             let nodes;
-            beforeAll(async () => {
+            beforeEach(async () => {
                 elm = createElement(tagName, {
                     is: ctor,
                 });
@@ -829,7 +829,7 @@ describe('text containing comments', () => {
             });
 
             afterAll(() => {
-                document.body.removeChild(elm);
+                elm.remove();
             });
 
             const assertChildNodesInnerHTMLMatches = (actual, expected) => {

packages/@lwc/integration-karma/test/template/directive-for-each/index.spec.js

@@ -96,8 +96,10 @@ describe('null/undefined values', () => {
                 expect(spy.calls.error.length).toBe(0);
             } else {
                 expect(spy.calls.error.length).toBe(1);
+                const err = spy.calls.error[0][0]; // first arg of first call
+                expect(err).toBeInstanceOf(Error);
                 // TODO [#1283]: Improve this error message. The vm should not be exposed and the message is not helpful.
-                expect(spy.calls.error[0]).toMatch(/It must be an array-like object/);
+                expect(err.message).toMatch(/It must be an array-like object/);
             }
         });
     });