Repository navigation
Audit and remove unused test fixtures - #294
Conversation
Audit test fixtures across workflows, tests, scripts, and documentation. Remove __tests__/fixtures/pages/node/bun/minimal as it is demonstrably unused.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
🦋 Changeset detectedLatest commit: 64456d3 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Removed test fixtures that are no longer in use to clean up the codebase.
Test Fixture Audit & Cleanup Summary
1. List of Fixture Directories and Files Removed
__tests__/fixtures/pages/node/bun/minimal/__tests__/fixtures/pages/node/bun/minimal/astro.config.mjs__tests__/fixtures/pages/node/bun/minimal/bun.lock__tests__/fixtures/pages/node/bun/minimal/package.json__tests__/fixtures/pages/node/bun/minimal/src/pages/index.astro2. Reason for Removal
__tests__/fixtures/pages/node/bun/minimal: An audit of all active GitHub Actions workflows (.github/workflows/test-actions.yml), unit/integration test suites (__tests__/**/*.test.ts), scripts (.github/scripts/), and documentation confirmed thatpages/node/bun/minimalis never referenced directly or dynamically resolved.3. Audit of Retained Fixtures
The following 25 fixture directories are retained as they are actively required by unit tests or CI workflows:
__tests__/fixtures/consumer-project/and__tests__/fixtures/pages/node/npm/consumer/) were previously migrated in PR refactor: dynamic consumer projects and fixture consolidation #293 to dynamic project generation via.github/scripts/generate-consumer-project.jsand__tests__/helpers/consumer-project.ts.vite-plus/node/pnpm/minimal: Used intest-actions.ymlandresolve-environment.test.ts.pages/node/npm/basic: Used intest-actions.ymlandresolve-environment.test.ts.pages/node/npm/minimal: Used intest-actions.ymlmatrix.pages/node/npm/vite: Used intest-actions.ymlmatrix.pages/node/pnpm/basic: Used intest-actions.ymlmatrix.pages/node/pnpm/minimal: Used intest-actions.ymlmatrix.pages/node/bun/basic: Used intest-actions.ymlmatrix.pages/bun/bun/basic: Used intest-actions.ymlandresolve-environment.test.ts.pages/bun/bun/minimal: Used intest-actions.ymlmatrix.pages/bun/pnpm/basic: Used intest-actions.ymlandresolve-environment.test.ts.autofix/node/npm/basic: Used intest-actions.ymlandautofix.test.ts.autofix/node/npm/minimal: Used intest-actions.ymlmatrix.autofix/node/pnpm/basic: Used intest-actions.ymlandautofix.test.ts.autofix/node/pnpm/minimal: Used intest-actions.ymlmatrix.autofix/node/bun/basic: Used intest-actions.ymlmatrix.autofix/node/bun/minimal: Used intest-actions.ymlmatrix.autofix/bun/bun/basic: Used intest-actions.ymlandautofix.test.ts.autofix/bun/bun/minimal: Used intest-actions.ymlmatrix.autofix/bun/pnpm/basic: Used intest-actions.ymlmatrix.autofix/python/uv: Used intest-actions.ymlmatrix.autofix/python/pip: Used intest-actions.ymlmatrix.autofix/python/conflicting: Used intest-actions.ymlmatrix.autofix/python/none: Used intest-actions.ymlandresolve-environment.test.ts.autofix/python/mixed: Used intest-actions.ymlmatrix.autofix/python/mixed-bun: Used intest-actions.ymlmatrix.4. Commands Executed & Results
npm test(vp test): Passed (9 test files, 448 tests passed).npm run lint(vp lint): Passed (0 warnings, 0 errors).npm run format -- --check(vp fmt --check): Passed (all files correctly formatted).npm run build(vp pack): Passed (successfully compiled dist/*.mjs bundles).5. Limitations & Unrun Checks
PR created automatically by Jules for task 2702519860170047643 started by @torn4dom4n