Skip to content

mint-components: fix CI unit test failures (Chrome sandbox + Puppeteer 22 compat) - #529

Draft
Coleton (Locrian24) with Copilot wants to merge 1 commit into
masterfrom
copilot/fix-mint-components-unit-tests
Draft

Coleton (Locrian24) with Copilot wants to merge 1 commit into
masterfrom
copilot/fix-mint-components-unit-tests

Conversation

Copilot AI commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

The Unit Tests CI job fails on build (mint-components): Puppeteer's Chrome crashes on launch with No usable sandbox!, so no tests run. Underneath that, the Stencil 2.4 e2e harness is incompatible with the lockfile's puppeteer@22.10.0, and one stale e2e assertion surfaced once tests could execute.

Changes

  • package.json — pass --ci to the test script. Stencil only adds --no-sandbox, --disable-setuid-sandbox, and --disable-dev-shm-usage to Chrome's launch args in CI mode; current runner kernels deny the SUID sandbox without them.

    - "test": "stencil test --spec --e2e --verbose --maxWorkers=1",
    + "test": "stencil test --spec --e2e --verbose --maxWorkers=1 --ci",
  • patches/@stencil+core+2.4.0.patch — extend the existing patch-package patch to bridge the Puppeteer 22 API break: ElementHandle.executionContext() was removed, and evaluate* no longer unwraps arrays of element handles. Replaced with handle.realm (verified to accept element-handle args) and Array.from(...) around the property-map iteration in findAll. Applied automatically via the existing postinstall hook.

  • sqm-big-stat.e2e.tshasInvalidStat now expects "-" instead of "!!!". The component's invalid-stat sentinel changed at some point; the negative-path tests were never updated (confirmed against the live DOM).

Suite result before: browser launch crash, 0 tests executed. After: 6/6 suites, 103/103 tests pass.

…r 22 compat)

- Add --ci to test script so Stencil launches Chrome with --no-sandbox
  (fixes "No usable sandbox" failure on current GitHub runners)
- Patch @stencil/core 2.4.0 testing module for puppeteer 22 compat:
  ElementHandle.executionContext() -> .realm, Array.from for property map
- Update sqm-big-stat e2e: invalid stat-type renders "-" (not "!!!")

Co-authored-by: Locrian24 <46334079+Locrian24@users.noreply.github.com>
Copilot AI changed the title mint-components: fix unit test CI failures (Chrome sandbox + Puppeteer 22 compat) mint-components: fix CI unit test failures (Chrome sandbox + Puppeteer 22 compat) Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants