Skip to content

[WIP] Add extensive unit tests for core logic and utilities - #490

Draft
timrogers with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-93d5c23f-a6f6-4344-a298-c4e5a12449f7
Draft

[WIP] Add extensive unit tests for core logic and utilities#490
timrogers with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-93d5c23f-a6f6-4344-a298-c4e5a12449f7

Conversation

Copilot AI commented Jun 15, 2025

Copy link
Copy Markdown
Contributor
  • Explore repository structure and understand codebase
  • Identify functions that need unit testing
  • Verify current build/lint setup works
  • Install and configure Jest with TypeScript support for ES modules
  • Create tests/ directory structure
  • Set up mocking infrastructure for GitHub API interactions
  • Add unit tests for core utilities in src/utils.ts
  • Add unit tests for product/version logic in src/github-products.ts
  • Add unit tests for helpers in src/commands/import.ts
  • Ensure good coverage of edge cases and error handling
  • Add npm script to run unit tests
  • Document how to run tests in CONTRIBUTING.md

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@timrogers

Copy link
Copy Markdown
Owner

@copilot Try again - you got stuck on the test setup last time, so try taking a step back

Copilot AI commented Jun 15, 2025

Copy link
Copy Markdown
Contributor Author

@copilot Try again - you got stuck on the test setup last time, so try taking a step back

Taking a step back and trying a simpler approach. Instead of struggling with Jest + TypeScript + ES modules configuration, I'll use Node.js built-in test runner which works well with ES modules out of the box.

Co-authored-by: timrogers <116134+timrogers@users.noreply.github.com>
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