Skip to content

Conversation

@Pugma
Copy link
Collaborator

@Pugma Pugma commented Mar 16, 2025

User description

テストコードにおいて、 Vitest が導入されているにも関わらず jest からの移行がされていなかったのでここでしておきました
確認をお願いします


PR Type

  • Tests
  • Enhancement

Description

  • テストコードをVitestに移行

  • 不要なJest依存(@types/jest)を削除

  • テストランナーの統一と依存整理


Changes walkthrough 📝

Relevant files
Tests
validate.test.ts
Vitestテストインポートの追加                                                                               

src/lib/validate.test.ts

  • Vitest用テスト関数(test, describe, expect)を追加
  • Jest依存インポートを置換・整理
+1/-0     
Dependencies
package.json
package.jsonにおけるJest依存削除                                                                 

package.json

  • @types/jestの依存定義を削除
  • テスト関連の型定義クリーンアップ実施
+0/-1     

Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @github-actions
    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 PR contains tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Vitest導入

    テストファイルにVitestのインポートが追加されました。Jestからの完全な移行となっているか、他のテストコードでJest特有の記述が残っていないか確認してください。

    import { test, describe, expect } from 'vitest'

    @github-actions
    Copy link
    Contributor

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Copy link
    Contributor

    @sh0go07 sh0go07 left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    遅くなりました…!
    よさそうです!

    @Pugma Pugma merged commit e3025eb into main Mar 20, 2025
    11 checks passed
    @Pugma Pugma deleted the dev/uninstallJest branch March 20, 2025 03:46
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    3 participants