Skip to content

Commit 0b572b9

Browse files
balzssclaude
andcommitted
fix(ci): integrate regression-test into pnpm workspace and fix Cypress
Major fixes for CI/CD test failures: 1. **Add regression-test to pnpm workspace**: - Update pnpm-workspace.yaml to include 'regression-test' - Now managing 99 workspace packages (was 98) 2. **Fix regression-test dependencies**: - Remove file: protocol dependency ("instructure-ui": "file:../packages") - Add proper workspace dependencies: - @instructure/emotion: workspace:* - @instructure/ui-a11y-content: workspace:* - Fixes module resolution errors in visual regression tests 3. **Fix Cypress binary installation**: - Add "Install Cypress binary" step in pr-validation.yml - Run npx cypress install after pnpm install - Fixes "Cypress binary is missing" error 4. **Update lockfile**: - pnpm install regenerates lockfile with workspace changes - Cypress binary auto-installs via postinstall hook Fixes: - Visual regression: "Module not found: Can't resolve '@instructure/emotion'" - Cypress component tests: "The Cypress binary is missing" All workspace packages now properly linked and accessible. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent b349501 commit 0b572b9

File tree

4 files changed

+2570
-173
lines changed

4 files changed

+2570
-173
lines changed

.github/workflows/pr-validation.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,8 @@ jobs:
5555
cache: 'pnpm'
5656
- name: Install dependencies
5757
run: pnpm install --frozen-lockfile
58+
- name: Install Cypress binary
59+
run: npx cypress install
5860
- name: Bootstrap project
5961
run: pnpm run bootstrap
6062
- name: Run Cypress components tests

0 commit comments

Comments
 (0)