Skip to content

test(e2e): Install test-utils from a tarball - #25396

Merged
mydea merged 0 commit into
ci/playwright-background-installsfrom
test/e2e-test-utils-tarball
Oct 15, 2026
Merged

mydea merged 0 commit into
ci/playwright-background-installsfrom
test/e2e-test-utils-tarball

Conversation

@mydea

@mydea mydea commented Oct 15, 2026 •

Copy link
Copy Markdown
Member

Test apps depended on @sentry-internal/test-utils via link:../../../test-utils. Its imports (express, ws) therefore resolved from the monorepo's root node_modules, so E2E jobs needed that restored even though the apps install everything else themselves.

test-utils is now packed like the SDK packages, and its tarball gets its own artifact so it can't end up among the release tarballs. Apps depend on it directly with file:../../packed/sentry-internal-test-utils-packed.tgz, so pnpm installs its dependencies into each app. It is also pinned through the pnpm overrides like the SDK packages, which is redundant but harmless.

Locally, changes to test-utils now need yarn build:tarball, as for the SDK packages.

Stacked on #25384.

🤖 Generated with Claude Code

@mydea
mydea added this pull request to stack #25397 October 15, 2026 11:58
@mydea
mydea force-pushed the test/e2e-test-utils-tarball branch from e6b2a19 to 5f86924 Compare October 15, 2026 12:51
@mydea
mydea merged commit 5f86924 into develop Oct 15, 2026
3 checks passed
@mydea
mydea deleted the test/e2e-test-utils-tarball branch October 15, 2026 12:51
@mydea
mydea restored the test/e2e-test-utils-tarball branch October 15, 2026 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant