Skip to content

AVA to Vitest #6

Merged
70-10 merged 3 commits intomainfrom
vitest
Sep 22, 2025
Merged

AVA to Vitest #6
70-10 merged 3 commits intomainfrom
vitest

Conversation

@70-10
Copy link
Copy Markdown
Owner

@70-10 70-10 commented Sep 19, 2025

No description provided.

mti-yoshida-nao and others added 3 commits September 19, 2025 14:04
- Replace AVA with Vitest 3.2.4 for testing framework
- Update test files to use Vitest syntax:
  - Change test.cb to Promise-based tests
  - Replace t.is() with expect().toBe()
  - Replace t.true() with expect().toBe(true)
  - Convert callback-based tests to async/Promise patterns
- Add vitest.config.js for test configuration
- Add @vitest/coverage-v8 for coverage reporting
- Update package.json test scripts to use Vitest

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Remove .circleci/config.yml (Node.js 4-alpine setup)
- Add .github/workflows/ci.yml with modern CI configuration:
  - Matrix build with Node.js 22.x and 24.x
  - Lint, test, and coverage steps
  - Triggers on push/PR to main/master branches
- Update package.json engines requirement to "node": ">=22"

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Update ESLint to latest version with flat config format
- Install and use 'globals' package for Node.js globals instead of manual definition
- Update languageOptions:
  - ecmaVersion: 2017 → 2022
  - sourceType: 'commonjs' for src files, 'module' for test files
  - Use globals.node instead of manual global definitions
- Add specific configuration for test files to support ES modules
- Remove deprecated .eslintrc in favor of eslint.config.js

Fixes CI cosmicconfig compatibility issues with Node.js 22/24.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@70-10 70-10 merged commit cacb10c into main Sep 22, 2025
2 checks passed
@70-10 70-10 deleted the vitest branch September 22, 2025 02:34
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