Skip to content

Conversation

@danadajian
Copy link
Contributor

@danadajian danadajian commented Feb 3, 2026

📝 Description

  • Nukes Jest and switches to Bun's built-in test runner!

danadajian and others added 15 commits February 2, 2026 11:39
- Remove Jest dependencies (jest, @swc/jest, @types/jest)
- Remove Jest configuration from package.json
- Update test script to use 'bun test'
- Update test type definitions to use Bun's Mock type
- Migrate all 43 test files to Bun test syntax:
  - Replace jest.mock() with mock.module()
  - Replace jest.fn() with mock()
  - Replace jest.spyOn() with spyOn()
  - Convert static imports to dynamic imports (await import())
  - Add process.env.INPUT_GITHUB_TOKEN for test environment
  - Create comprehensive mockOctokit structure
  - Mock octokit module directly for proper test isolation

Current status:
- 87 tests passing (41%)
- 125 tests need mock implementation adjustments
- Some linting errors to be addressed in follow-up
- All files structurally migrated and executable with bun test

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
@danadajian danadajian marked this pull request as ready for review February 3, 2026 18:43
@danadajian danadajian requested a review from a team as a code owner February 3, 2026 18:43
@danadajian danadajian merged commit 97786b8 into main Feb 3, 2026
1 check passed
@danadajian danadajian deleted the migrate-to-bun-test branch February 3, 2026 18:43
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