LearnNewsExamplesServices
Frontmatter
id7293
titleConvert vdom/layout/Cube.mjs Test from Siesta to Playwright
stateClosed
labels
enhancementhelp wantedgood first issuehacktoberfest
assigneeskart-u
createdAtSep 27, 2025, 4:07 PM
updatedAtOct 4, 2025, 8:47 PM
githubUrlhttps://github.com/neomjs/neo/issues/7293
authortobiu
commentsCount6
parentIssue7262
subIssues[]
subIssuesCompleted0
subIssuesTotal0
contentTrust
projected
quarantined0
signals[]
blockedBy[]
blocking[]
closedAtOct 4, 2025, 7:58 PM

Convert vdom/layout/Cube.mjs Test from Siesta to Playwright

Closed v11.0.0/archive-v11-0-0-chunk-1 enhancementhelp wantedgood first issuehacktoberfest
tobiu
tobiu commented on Sep 27, 2025, 4:07 PM

This task is to migrate the unit test for vdom/layout/Cube.mjs from the Siesta test harness to the Playwright test runner.

Acceptance Criteria

  1. Create a new test file at test/playwright/unit/vdom/layout/Cube.spec.mjs.
  2. Translate all assertions from the original file (test/siesta/tests/vdom/layout/Cube.mjs) to the Playwright/Jest expect syntax.
  3. Ensure the new test runs successfully via npm test.
  4. The new test must cover all the functionality of the original Siesta test.
tobiu added the enhancement label on Sep 27, 2025, 4:07 PM
tobiu added parent issue #7262 on Sep 27, 2025, 4:07 PM
tobiu
tobiu Oct 3, 2025, 6:24 PM

Ha, these 2 comments came in almost at the same time. Assuming you are not the same person, I would stick to "first comment wins". I hope this is fine, since both of you already have other tickets assigned.

No worries, I can easily create more new tickets if needed.

  • 2025-10-04T10:56:55Z @kart-u cross-referenced by PR #7352
  • 2025-10-04T12:23:03Z @kart-u cross-referenced by #7286
tobiu
tobiu Oct 4, 2025, 8:31 PM

reopening, since something is off with the imports / top-level setup. only happens when running all tests combined.

@kart-u - 2025-10-04T18:39:35Z

please check again made some changes asked Gemini on issue it state modified state of Neo.config.useDomApiRenderer might be messing with other tests

tobiu
tobiu Oct 4, 2025, 8:47 PM

the PR was already merged. somehow the combination of the cube layout and vdomrealword tests clashed. now all 114 tests are passing.