Add Playwright tests that execute every playground cell, fix merge_ordered#259
Add Playwright tests that execute every playground cell, fix merge_ordered#259
Conversation
Agent-Logs-Url: https://github.com/githubnext/tsessebe/sessions/5c9060df-3524-4889-907b-4502985236a9 Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
….html Agent-Logs-Url: https://github.com/githubnext/tsessebe/sessions/5c9060df-3524-4889-907b-4502985236a9 Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
|
Warning The Evergreen: Merge Conflict Resolved ✅Merged Conflict details: Three code cells had conflicting versions — the PR's HEAD had self-contained cells (with imports and variable definitions, needed for Playwright test execution), while
|
|
Warning The 🌿 Evergreen fix: resolved merge conflict in The conflict was in 3 code cells (inner join, different key names, and suffix examples). I kept the PR's version which has self-contained examples with imports and
|
|
@copilot resolve the merge conflicts in this pull request |
Agent-Logs-Url: https://github.com/githubnext/tsessebe/sessions/f9757419-b539-49c1-8f3d-f4bdcc589a01 Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Resolved in e0823ea. The merge conflict was in While merging I also regenerated |
|
@copilot fix tests |
Agent-Logs-Url: https://github.com/githubnext/tsessebe/sessions/0c7c6af4-c8ae-4328-ae2f-aa1ddf7531b2 Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Fixed in 51ec9da. The unit-test CI job's "Test" step was failing because the |
|
Warning The Evergreen Fix 🌿Root cause: The CI test command Fix: Changed A prior Copilot SWE agent (commit Warning Firewall blocked 2 domainsThe following domains were blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "releaseassets.githubusercontent.com"
- "resultsreceiver.actions.githubusercontent.com"See Network Configuration for more information.
|
|
Warning The Evergreen fix: CI test pathRoot cause: The Fix: Changed the CI test command from Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "releaseassets.githubusercontent.com"See Network Configuration for more information.
|
|
Warning The Evergreen fix: The "Test & Lint" job was failing because Fixed by changing the CI command to
|
|
Warning The Evergreen fix applied 🌿 The "Test & Lint" job was failing with 1 unnamed test error because Fix: Changed to: The Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
network:
allowed:
- defaults
- "releaseassets.githubusercontent.com"See Network Configuration for more information.
|
Run: https://github.com/githubnext/tsessebe/actions/runs/25139337654 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…s_object Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…pe, format - Use local variable 'h' instead of reassigning 'hash' parameter in fnvString and fnvScalar - Replace non-null assertion bytes[i]! with null-coalescing bytes[i] ?? 0 - Auto-fix: sort imports, make DataFrame import type-only, format test file Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Warning The Evergreen fix applied ✅ Resolved a merge conflict in
Kept the PR's
|
|
Warning The Evergreen 🌲 — Merge conflict resolved. Merged
|
|
Warning The Evergreen: Resolved merge conflict in The conflict was in two code blocks where the PR used The merge with
|
|
Warning The Evergreen 🌿 — merge conflict resolved. The conflict was in
|
|
Warning The Evergreen: Merge conflict resolved ✅ Merged
|
|
Warning The 🌿 Evergreen: Merge Conflict ResolvedMerged Conflict: Both branches added
Kept
|
|
Warning The Evergreen fix: Resolved merge conflict in The conflict was an Changes pushed to branch.
|
|
Warning The Evergreen 🌲 — Merge conflict resolved. Merged
|
|
Warning The Evergreen: Resolved merge conflict in The conflict was an
|
|
Warning The Evergreen fix: Resolved merge conflict in The conflict was in two
|
|
Warning The Evergreen: Resolved merge conflict in The conflict was an
|
|
Warning The Evergreen: Merge conflict resolved ✅ The PR had a single conflict in I kept the PR's version (
|
|
Warning The Evergreen: Resolved merge conflict in The conflict was an add/add conflict where
|
|
Warning The Evergreen fix: Resolved merge conflict in The conflict was between Merged
|
|
Warning The Evergreen 🌲: Resolved merge conflict with Conflict:
|
|
Warning The Evergreen 🌲 — merged Resolved one merge conflict in The branch is now up to date with
|
|
Warning The 🌿 Evergreen — Merge conflict resolvedMerged Conflict: Both branches added the same file. Resolved by keeping CI checks are now running on the new HEAD. No code logic was changed — only the playground HTML was updated.
|
|
Warning The Evergreen 🌲 — Resolved merge conflict PR had a single conflict in
|
|
Warning The 🌿 Evergreen: Merge Conflict ResolvedMerged Conflict resolution: Two versions of the file added New HEAD:
|
|
Warning The 🌿 Evergreen: Merge Conflict ResolvedI resolved a merge conflict in Conflict: Two versions of The branch is now up to date with
|
|
Warning The Evergreen: Resolved merge conflict with The conflict was in Pushed a clean merge commit:
|
|
Warning The 🌿 Evergreen: Merge conflict resolvedThis PR had a merge conflict in I resolved the conflict in favor of the PR's The merge commit has been pushed to this branch.
|
|
Warning The 🌿 Evergreen: Merge conflict resolvedMerged Conflict: Both sides added the same file ( CI checks are now running on the updated branch.
|
|
@copilot resolve the merge conflicts in this pull request |
…t-tests-for-playground # Conflicts: # playground/hash_pandas_object.html Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
Resolved in 823c55a. The conflict was again in |
The hosted playground pages had widespread breakage (cells with no output, undefined imports, Python pasted into TypeScript editors), and there was no test catching it. A scan of all 130 pages found 78 with at least one broken cell — 436 of 795 cells fail when actually run.
Test infrastructure
tests-e2e/playground-cells.test.ts: launches headless Chromium, navigates to everyplayground/*.html, clicks ▶ Run on every.playground-block, and asserts each cell produces non-empty output with no TypeScript or runtime error. ~42s for the full 130-page sweep at concurrency 6.bun run test:e2escript andplayground-e2eCI job (Chromium cached viaactions/cache). The unit-test script is now scoped totests/so e2e doesn't run twice.Known-failures allowlist
tests-e2e/known-failures.jsonenumerates the currently-broken(file, cell)pairs so CI is green today while still catching:Fix
playground/merge_ordered.htmlDemonstrates the canonical pattern and is removed from the allowlist. Each cell is now self-contained — the playground runtime executes every cell in its own
new Function()scope, so nothing leaks between cells:Docs
docs/playground.mddocuments the e2e test, the allowlist workflow, and the per-cell authoring rule (own imports, own data, explicitconsole.log).Out of scope
Fixing the other 78 pages is mechanical but voluminous; they remain on the allowlist for follow-up.
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
dl.google.com/usr/lib/apt/methods/https /usr/lib/apt/methods/https(dns block)If you need me to access, download, or install something from one of these locations, you can either: