Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: resolve TypeScript and Jest test issues in transform tests #333

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

amirabbas-gh
Copy link

@amirabbas-gh amirabbas-gh commented Feb 1, 2025

📚 Description

This PR addresses multiple issues related to TypeScript errors, Jest test failures, and incorrect fixture names in transform tests. The main updates include:

  1. Fixed TypeScript errors related to require.requireActual in multiple transform tests
  2. Fixed Jest test failures in transform tests
  3. Updated jscodeshift to the latest version
  4. Fixed incorrect TypeScript test fixture names

These changes ensure that all affected tests run correctly without TypeScript or Jest errors.

🔗 Linked Issue

🧪 Test Plan

  1. Updated jscodeshift and verified that all transforms still function correctly.
  2. Ran the full test suite to confirm all Jest and TypeScript errors are resolved.
  3. Checked that all fixture names are correctly aligned with test cases.

@amirabbas-gh amirabbas-gh marked this pull request as draft February 2, 2025 19:03
@amirabbas-gh amirabbas-gh force-pushed the fix_issue_332 branch 4 times, most recently from ec1ee2b to 9df34d2 Compare February 5, 2025 20:20
@amirabbas-gh amirabbas-gh changed the title fix: resolve require.requireActual TypeScript errors in transform tests fix: resolve TypeScript and Jest test issues in transform tests Apr 2, 2025
@amirabbas-gh amirabbas-gh marked this pull request as ready for review April 2, 2025 14:03
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.

❌ require.requireActual Causes TypeScript Error in some Tests
1 participant