upgrade-helper icon indicating copy to clipboard operation
upgrade-helper copied to clipboard

#92: Techdebt/testing infrastructure improvements

Open mjstelly opened this issue 10 months ago • 3 comments

Summary

This PR addresses the beginning of a solution for issue #92.

Test Plan

The ACTION_PLAN.md file, found in the root directory, itemizes the work plan for this solution.

What are the steps to reproduce?

Nothing to reproduce.

Checklist

What was performed: Fixed Issues:

  • ✅ Jest testMatch pattern syntax error (removed extra >)
  • ✅ Standardized test file naming (.test.tsx → .spec.tsx)
  • ✅ Organized test files in proper tests/ directory structure
  • ✅ Updated import paths and snapshots
  • ✅ Verified all 4 test suites pass (18 tests total)
  • [x] I tested this thoroughly
  • [ ] I added the documentation in README.md (if needed)
  • [ ]

mjstelly avatar Jun 16 '25 12:06 mjstelly

AS mentioned, this is phase 1 of the action plan. If you approve of this PR and the associated plan, I'll proceed with the plan execution.

mjstelly avatar Jun 16 '25 12:06 mjstelly

we basically need tests for the version selection, thats all. https://github.com/react-native-community/upgrade-helper/issues/92#issuecomment-534981592. if you're up for that, do some of this? no need for extra files.

pvinis avatar Jun 16 '25 14:06 pvinis

we basically need tests for the version selection, thats all. #92 (comment). if you're up for that, do some of this? no need for extra files.

This first PR is organizational in nature. It lays the groundwork for further planned improvements and fixes a syntax error in the config file. The files are necessary for this work to continue.

mjstelly avatar Jun 16 '25 14:06 mjstelly