playwright test files: smarter organization of folders and files, imports cleanup

tobiu · tobiu · commit fbb84ccb1f4e · 2025-11-04T17:31:32.000+01:00
diff --git a/test/playwright/unit/button/Base.spec.mjs b/test/playwright/unit/button/Base.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 const appName = 'ClassicButtonTest';
 
@@ -13,13 +13,12 @@ setup({
     }
 });
 
-import { test, expect } from '@playwright/test';
-
-import Neo                from '../../../src/Neo.mjs';
-import * as core          from '../../../src/core/_export.mjs';
-import Button             from '../../../src/button/Base.mjs';
-import DomApiVnodeCreator from '../../../src/vdom/util/DomApiVnodeCreator.mjs';
-import VdomHelper         from '../../../src/vdom/Helper.mjs';
+import {test, expect}     from '@playwright/test';
+import Neo                from '../../../../src/Neo.mjs';
+import * as core          from '../../../../src/core/_export.mjs';
+import Button             from '../../../../src/button/Base.mjs';
+import DomApiVnodeCreator from '../../../../src/vdom/util/DomApiVnodeCreator.mjs';
+import VdomHelper         from '../../../../src/vdom/Helper.mjs';
 
 test.describe('Neo.button.Base VDOM (Node.js)', () => {
 
diff --git a/test/playwright/unit/collection/Base.spec.mjs b/test/playwright/unit/collection/Base.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 const appName = 'CollectionBaseTest';
 
@@ -8,8 +8,7 @@ setup({
     }
 });
 
-import { test, expect } from '@playwright/test';
-
+import {test, expect}  from '@playwright/test';
 import Neo             from '../../../../src/Neo.mjs';
 import * as core       from '../../../../src/core/_export.mjs';
 import Collection      from '../../../../src/collection/Base.mjs';
diff --git a/test/playwright/unit/config/AfterSetConfig.spec.mjs b/test/playwright/unit/config/AfterSetConfig.spec.mjs
@@ -1,11 +1,10 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup();
 
-import { test, expect } from '@playwright/test';
-
-import Neo   from '../../../../src/Neo.mjs';
-import Base  from '../../../../src/core/Base.mjs';
+import {test, expect} from '@playwright/test';
+import Neo            from '../../../../src/Neo.mjs';
+import Base           from '../../../../src/core/Base.mjs';
 
 class TestComponent extends Base {
     static config = {
@@ -109,4 +108,4 @@ test.describe('Neo.core.Base#afterSetConfig', () => {
         expect(instance.nonReactive).toBe('changedAgainNonReactive');
         expect(instance.classField).toBe('changedAgainClassField');
     });
-});
+});
diff --git a/test/playwright/unit/config/Basic.spec.mjs b/test/playwright/unit/config/Basic.spec.mjs
@@ -1,9 +1,8 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup();
 
-import { test, expect } from '@playwright/test';
-
+import {test, expect} from '@playwright/test';
 import Neo            from '../../../../src/Neo.mjs';
 import Base           from '../../../../src/core/Base.mjs';
 import {isDescriptor} from '../../../../src/core/ConfigSymbols.mjs';
diff --git a/test/playwright/unit/config/CircularDependencies.spec.mjs b/test/playwright/unit/config/CircularDependencies.spec.mjs
@@ -1,11 +1,10 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup();
 
-import { test, expect } from '@playwright/test';
-
-import Neo  from '../../../../src/Neo.mjs';
-import Base from '../../../../src/core/Base.mjs';
+import {test, expect} from '@playwright/test';
+import Neo            from '../../../../src/Neo.mjs';
+import Base           from '../../../../src/core/Base.mjs';
 
 class TestComponent extends Base {
     static config = {
diff --git a/test/playwright/unit/config/CustomFunctions.spec.mjs b/test/playwright/unit/config/CustomFunctions.spec.mjs
@@ -1,9 +1,8 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup();
 
-import { test, expect } from '@playwright/test';
-
+import {test, expect} from '@playwright/test';
 import Neo            from '../../../../src/Neo.mjs';
 import Base           from '../../../../src/core/Base.mjs';
 import {isDescriptor} from '../../../../src/core/ConfigSymbols.mjs';
diff --git a/test/playwright/unit/config/Hierarchy.spec.mjs b/test/playwright/unit/config/Hierarchy.spec.mjs
@@ -1,9 +1,8 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup();
 
-import { test, expect } from '@playwright/test';
-
+import {test, expect} from '@playwright/test';
 import Neo            from '../../../../src/Neo.mjs';
 import Base           from '../../../../src/core/Base.mjs';
 import {isDescriptor} from '../../../../src/core/ConfigSymbols.mjs';
diff --git a/test/playwright/unit/config/MemoryLeak.spec.mjs b/test/playwright/unit/config/MemoryLeak.spec.mjs
@@ -1,11 +1,10 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup();
 
-import { test, expect } from '@playwright/test';
-
-import Neo  from '../../../../src/Neo.mjs';
-import Base from '../../../../src/core/Base.mjs';
+import {test, expect} from '@playwright/test';
+import Neo            from '../../../../src/Neo.mjs';
+import Base           from '../../../../src/core/Base.mjs';
 
 class PublisherComponent extends Base {
     static config = {
diff --git a/test/playwright/unit/config/MultiLevelHierarchy.spec.mjs b/test/playwright/unit/config/MultiLevelHierarchy.spec.mjs
@@ -1,9 +1,8 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup();
 
-import { test, expect } from '@playwright/test';
-
+import {test, expect} from '@playwright/test';
 import Neo            from '../../../../src/Neo.mjs';
 import Base           from '../../../../src/core/Base.mjs';
 import {isDescriptor} from '../../../../src/core/ConfigSymbols.mjs';
diff --git a/test/playwright/unit/core/ClassConfigsAndFields.spec.mjs b/test/playwright/unit/core/ClassConfigsAndFields.spec.mjs
@@ -1,11 +1,10 @@
-import { setup } from '../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup();
 
-import { test, expect } from '@playwright/test';
-
-import Neo       from '../../../src/Neo.mjs';
-import * as core from '../../../src/core/_export.mjs';
+import {test, expect} from '@playwright/test';
+import Neo            from '../../../../src/Neo.mjs';
+import * as core      from '../../../../src/core/_export.mjs';
 
 class TestClass extends core.Base {
     fieldA = 1;
diff --git a/test/playwright/unit/core/ClassSystem.spec.mjs b/test/playwright/unit/core/ClassSystem.spec.mjs
@@ -1,11 +1,10 @@
-import { setup } from '../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup();
 
-import { test, expect } from '@playwright/test';
-
-import Neo       from '../../../src/Neo.mjs';
-import * as core from '../../../src/core/_export.mjs';
+import {test, expect} from '@playwright/test';
+import Neo            from '../../../../src/Neo.mjs';
+import * as core      from '../../../../src/core/_export.mjs';
 
 /**
  * @summary Verifies the behavior of config-related hooks (`beforeSet*`, `afterSet*`).
diff --git a/test/playwright/unit/core/Effect.spec.mjs b/test/playwright/unit/core/Effect.spec.mjs
@@ -1,18 +1,17 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup({
     appConfig: {
         name: 'CoreEffectTest'
     }
 });
 
-import { test, expect } from '@playwright/test';
-
-import Neo           from '../../../../src/Neo.mjs';
-import * as core     from '../../../../src/core/_export.mjs';
-import Effect        from '../../../../src/core/Effect.mjs';
-import EffectManager from '../../../../src/core/EffectManager.mjs';
-import Config        from '../../../../src/core/Config.mjs';
+import {test, expect} from '@playwright/test';
+import Neo            from '../../../../src/Neo.mjs';
+import * as core      from '../../../../src/core/_export.mjs';
+import Effect         from '../../../../src/core/Effect.mjs';
+import EffectManager  from '../../../../src/core/EffectManager.mjs';
+import Config         from '../../../../src/core/Config.mjs';
 
 test.describe('Neo.core.Effect', () => {
     test('EffectManager should manage active effects', () => {
diff --git a/test/playwright/unit/core/EffectBatching.spec.mjs b/test/playwright/unit/core/EffectBatching.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 const appName = 'EffectBatchingTest';
 
@@ -8,12 +8,11 @@ setup({
     }
 });
 
-import { test, expect } from '@playwright/test';
-
-import Neo           from '../../../../src/Neo.mjs';
-import * as core     from '../../../../src/core/_export.mjs';
-import Effect        from '../../../../src/core/Effect.mjs';
-import EffectManager from '../../../../src/core/EffectManager.mjs';
+import {test, expect} from '@playwright/test';
+import Neo            from '../../../../src/Neo.mjs';
+import * as core      from '../../../../src/core/_export.mjs';
+import Effect         from '../../../../src/core/Effect.mjs';
+import EffectManager  from '../../../../src/core/EffectManager.mjs';
 
 /**
  * @summary This test suite verifies the batching behavior of effects within Neo.mjs.
@@ -321,4 +320,4 @@ test.describe('Neo.core.EffectManager & Neo.core.Base', () => {
         effect.destroy();
         instance.destroy();
     });
-});
+});
diff --git a/test/playwright/unit/form/field/AfterSetValueSequence.spec.mjs b/test/playwright/unit/form/field/AfterSetValueSequence.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../../../setup.mjs';
+import {setup} from '../../../setup.mjs';
 
 const appName = 'AfterSetValueSequence';
 
@@ -8,8 +8,7 @@ setup({
     }
 });
 
-import { test, expect } from '@playwright/test';
-
+import {test, expect}  from '@playwright/test';
 import Neo             from '../../../../../src/Neo.mjs';
 import * as core       from '../../../../../src/core/_export.mjs';
 import InstanceManager from '../../../../../src/manager/Instance.mjs';
diff --git a/test/playwright/unit/functional/Button.spec.mjs b/test/playwright/unit/functional/Button.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup({
     appConfig: {
@@ -10,8 +10,7 @@ setup({
     }
 });
 
-import { test, expect } from '@playwright/test';
-
+import {test, expect}     from '@playwright/test';
 import Neo                from '../../../../src/Neo.mjs';
 import * as core          from '../../../../src/core/_export.mjs';
 import Button             from '../../../../src/functional/button/Base.mjs';
diff --git a/test/playwright/unit/functional/HtmlTemplateComponent.spec.mjs b/test/playwright/unit/functional/HtmlTemplateComponent.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup({
     appConfig: {
diff --git a/test/playwright/unit/manager/Instance.spec.mjs b/test/playwright/unit/manager/Instance.spec.mjs
@@ -1,12 +1,12 @@
-import { setup } from '../setup.mjs';
+import { setup } from '../../setup.mjs';
 
 setup();
 
 import { test, expect } from '@playwright/test';
 
-import Neo             from '../../../src/Neo.mjs';
-import * as core       from '../../../src/core/_export.mjs';
-import InstanceManager from '../../../src/manager/Instance.mjs';
+import Neo             from '../../../../src/Neo.mjs';
+import * as core       from '../../../../src/core/_export.mjs';
+import InstanceManager from '../../../../src/manager/Instance.mjs';
 
 /**
  * @summary Tests the InstanceManager functionality for tracking Neo.mjs instances.
diff --git a/test/playwright/unit/neo/MixinStaticConfig.spec.mjs b/test/playwright/unit/neo/MixinStaticConfig.spec.mjs
@@ -1,6 +1,6 @@
-import { test, expect } from '@playwright/test';
-import Neo              from '../../../../src/Neo.mjs';
-import Base             from '../../../../src/core/Base.mjs';
+import {test, expect} from '@playwright/test';
+import Neo            from '../../../../src/Neo.mjs';
+import Base           from '../../../../src/core/Base.mjs';
 
 test.describe('neo/MixinStaticConfig', () => {
     test('should merge static configs from mixins', () => {
@@ -134,4 +134,4 @@ test.describe('neo/MixinStaticConfig', () => {
         expect(instance.methodA()).toBe('A');
         expect(instance.methodB()).toBe('B');
     });
-});
+});
diff --git a/test/playwright/unit/state/FeedbackLoop.spec.mjs b/test/playwright/unit/state/FeedbackLoop.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 const appName = 'FeedbackLoopTest';
 
@@ -12,8 +12,7 @@ setup({
     }
 });
 
-import { test, expect } from '@playwright/test';
-
+import {test, expect}  from '@playwright/test';
 import Neo             from '../../../../src/Neo.mjs';
 import * as core       from '../../../../src/core/_export.mjs';
 import InstanceManager from '../../../../src/manager/Instance.mjs';
diff --git a/test/playwright/unit/state/Provider.spec.mjs b/test/playwright/unit/state/Provider.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 const appName = 'StateProviderTest';
 
@@ -12,13 +12,12 @@ setup({
     }
 });
 
-import { test, expect } from '@playwright/test';
-
-import Neo           from '../../../../src/Neo.mjs';
-import Component     from '../../../../src/component/Base.mjs';
+import {test, expect}  from '@playwright/test';
+import Neo             from '../../../../src/Neo.mjs';
+import Component       from '../../../../src/component/Base.mjs';
 import InstanceManager from '../../../../src/manager/Instance.mjs';
-import StateProvider from '../../../../src/state/Provider.mjs';
-import Store         from '../../../../src/data/Store.mjs';
+import StateProvider   from '../../../../src/state/Provider.mjs';
+import Store           from '../../../../src/data/Store.mjs';
 
 // Mock Component for testing purposes
 class MockComponent extends Component {
diff --git a/test/playwright/unit/state/ProviderNestedDataConfigs.spec.mjs b/test/playwright/unit/state/ProviderNestedDataConfigs.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 setup({
     appConfig: {
diff --git a/test/playwright/unit/util/Rectangle.spec.mjs b/test/playwright/unit/util/Rectangle.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../setup.mjs';
+import { setup } from '../../setup.mjs';
 
 const appName = 'RectangleTest';
 
@@ -9,9 +9,9 @@ setup({
 });
 
 import {test, expect} from '@playwright/test';
-import Neo            from '../../../src/Neo.mjs';
-import * as core      from '../../../src/core/_export.mjs';
-import Rectangle      from '../../../src/util/Rectangle.mjs';
+import Neo            from '../../../../src/Neo.mjs';
+import * as core      from '../../../../src/core/_export.mjs';
+import Rectangle      from '../../../../src/util/Rectangle.mjs';
 
 test.describe('Rectangle', () => {
     test('contains', () => {
diff --git a/test/playwright/unit/vdom/Advanced.spec.mjs b/test/playwright/unit/vdom/Advanced.spec.mjs
@@ -1,4 +1,4 @@
-import { setup } from '../../setup.mjs';
+import {setup} from '../../setup.mjs';
 
 // tests are designed for this rendering mode
 setup({
@@ -7,11 +7,10 @@ setup({
     }
 });
 
-import { test, expect } from '@playwright/test';
-
-import Neo                from '../../../../src/Neo.mjs';
-import StringFromVnode    from '../../../../src/vdom/util/StringFromVnode.mjs';
-import VdomHelper         from '../../../../src/vdom/Helper.mjs';
+import {test, expect}  from '@playwright/test';
+import Neo             from '../../../../src/Neo.mjs';
+import StringFromVnode from '../../../../src/vdom/util/StringFromVnode.mjs';
+import VdomHelper      from '../../../../src/vdom/Helper.mjs';
 
 let deltas, output, vdom, vnode;
 
diff --git a/test/playwright/unit/vdom/AsymmetricUpdates.spec.mjs b/test/playwright/unit/vdom/AsymmetricUpdates.spec.mjs
diff --git a/test/playwright/unit/vdom/Calendar.spec.mjs b/test/playwright/unit/vdom/Calendar.spec.mjs
diff --git a/test/playwright/unit/vdom/Helper.spec.mjs b/test/playwright/unit/vdom/Helper.spec.mjs
diff --git a/test/playwright/unit/vdom/RealWorldUpdates.spec.mjs b/test/playwright/unit/vdom/RealWorldUpdates.spec.mjs
diff --git a/test/playwright/unit/vdom/layout/Cube.spec.mjs b/test/playwright/unit/vdom/layout/Cube.spec.mjs
diff --git a/test/playwright/unit/vdom/table/Container.spec.mjs b/test/playwright/unit/vdom/table/Container.spec.mjs
