Skip to content

Commit 6899f08

Browse files
vgeorgekylebarron
andauthored
feat: add pytest-ipywidgets bbox testing, remove playwright (#999)
## Summary Add pytest-ipywidgets for browser-based bbox interaction testing and remove Playwright/Vitest infrastructure. ## Changes - Add `tests/ui/test_bbox_interaction.py` with complete bbox workflow testing - Add pytest-ipywidgets>=1.54.0 dependency - Remove all Playwright tests, configs, and dependencies - Remove Vitest testing infrastructure - Update GitHub Actions workflow to focus on build and lint ## Benefits - Faster widget-level testing vs full E2E - Better debugging with console access - Simplified maintenance with single test file - Same bbox interaction validation maintained ## Related Issues - Fixes #524 - Automated widget testing - Replaces #928 - Playwright CI testing --------- Co-authored-by: Kyle Barron <[email protected]>
1 parent 77bb3a0 commit 6899f08

23 files changed

+2491
-3852
lines changed

.github/workflows/test-js.yml

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: CI
1+
name: JavaScript Build and Lint
22

33
on:
44
push:
@@ -11,7 +11,7 @@ concurrency:
1111
cancel-in-progress: true
1212

1313
jobs:
14-
lint-and-test:
14+
build-and-lint:
1515
runs-on: ubuntu-latest
1616

1717
steps:
@@ -42,6 +42,3 @@ jobs:
4242

4343
- name: Lint check
4444
run: npm run lint
45-
46-
- name: Test
47-
run: npm run test

.github/workflows/test.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,10 @@ jobs:
5959
run: |
6060
npm run build
6161
62+
- name: Install Playwright browsers
63+
run: |
64+
uv run playwright install chromium
65+
6266
- name: Run tests
6367
run: uv run pytest
6468

0 commit comments

Comments
 (0)