Skip to content

chore(ci): add pack install mode to the E2E testapp - #115

Merged
bpapillon merged 1 commit into
mainfrom
sdk-e2e-pack
Sep 14, 2026
Merged

bpapillon merged 1 commit into
mainfrom
sdk-e2e-pack

Conversation

@bpapillon

Copy link
Copy Markdown
Collaborator

Adds scripts/e2e-pack.sh for the SDK E2E harness's pack install mode: it builds the release artifact (sdist + wheel) from the working tree, fails if the rules engine WASM is missing from the wheel, and installs that wheel with the datastream extra plus the testapp deps.

pip install . (the local mode) does not exercise the artifact we actually publish, so a packaging break — a dropped WASM, a bad extra — only shows up after release.

testapp/app.py only prepends src/ to sys.path when SDK_SOURCE=local. It used to do so unconditionally, which meant published mode was silently testing the working tree rather than the installed package. Startup now logs which copy of the SDK was imported.

Setup for pack: pip install --upgrade pip && ./scripts/e2e-pack.sh; start is unchanged.

@schematic-bot

Copy link
Copy Markdown
Contributor

✅ SDK E2E success for sdk-e2e-pack at 60b04a4 — run

mode source result failed step
datastream pack ✅ passed
default pack ✅ passed

The sdk-e2e status on this PR reflects this run. A new push resets it to pending; re-run sdk_e2e.yml with sdk-ref=sdk-e2e-pack to report on the new head.

@bpapillon
bpapillon marked this pull request as ready for review September 14, 2026 22:46
@bpapillon
bpapillon requested a review from a team as a code owner September 14, 2026 22:46
@bpapillon
bpapillon merged commit 753f2b7 into main Sep 14, 2026
6 checks passed
@bpapillon
bpapillon deleted the sdk-e2e-pack branch September 14, 2026 22:46
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.

2 participants