Skip to content

test: add ESM script build tests to publish/download suites#1798

Merged
kpal81xd merged 6 commits intomainfrom
worktree-test/esm-scripts
Feb 26, 2026
Merged

test: add ESM script build tests to publish/download suites#1798
kpal81xd merged 6 commits intomainfrom
worktree-test/esm-scripts

Conversation

@kpal81xd
Copy link
Copy Markdown
Contributor

@kpal81xd kpal81xd commented Feb 25, 2026

Fixes #1788

Summary

  • Merge separate publish/download and publish/download (contains ESM scripts) test suites into a single block using loop-based test generation over ['classic', 'esm'] script types
  • Add createEsmScript helper to common.ts that waits for the server-side upload pipeline (file:set event) instead of relying on networkidle
  • Add EsmBuildOptions interface and optional ESM options to downloadApp/publishApp helpers

Test plan

  • Run api/basic.test.ts — verify download app (scripts: classic), publish app (scripts: classic), create ESM script, download app (scripts: esm), publish app (scripts: esm) all pass
  • Run ui/basic.test.ts — verify the same test matrix passes via UI flow
  • Confirm no regressions in other test suites (create/delete, export/import, navigation)

🤖 Generated with Claude Code

@kpal81xd kpal81xd added run: test-suite Run the test suite and removed run: test-suite Run the test suite labels Feb 25, 2026
@kpal81xd kpal81xd merged commit 4cb3cd7 into main Feb 26, 2026
5 checks passed
@kpal81xd kpal81xd deleted the worktree-test/esm-scripts branch February 26, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run: test-suite Run the test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Test Suite] Check building MJS projects (uses different system in pipeline job)

1 participant