Skip to content

chore: update Jest configuration for improved caching#5

Merged
xwartz merged 2 commits into
mainfrom
dev
Jun 19, 2025
Merged

chore: update Jest configuration for improved caching#5
xwartz merged 2 commits into
mainfrom
dev

Conversation

@xwartz

@xwartz xwartz commented Jun 19, 2025

Copy link
Copy Markdown
Owner

PR-Codex overview

This PR focuses on enhancing the test suite for a Cursor API, improving error handling, and refining timer management in Jest tests. It also includes updates to the configuration and README for better clarity and functionality.

Detailed summary

  • Added Jest cleanup functions to manage timers.
  • Improved global test configurations for Jest.
  • Updated package.json to include --detectOpenHandles for tests.
  • Enhanced error handling in APIClient methods.
  • Added comprehensive tests for CursorError and TimeoutError.
  • Updated README with new model support and usage examples.
  • Added integration tests for Cursor client functionality.
  • Enhanced streaming tests to cover edge cases and error handling.
  • Improved handling of malformed responses and JSON parsing in tests.

The following files were skipped due to too many changes: tests/core/streaming.test.ts

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@github-actions github-actions Bot added 🎯 api API related 🐛 bug Something isn't working 📚 docs Improvements or additions to documentation 📚 large Large change (> 200 lines) 📦 dependencies Dependencies updates 🔧 config Configuration related 🧪 test Testing related 🧹 chore Maintenance tasks labels Jun 19, 2025
@github-actions

Copy link
Copy Markdown
Contributor

🤖 Auto-labeled this PR with: 🐛 bug, 📚 docs, 🧹 chore, 🧪 test, 📦 dependencies, 🎯 api, 🔧 config, 📚 large

This was done automatically based on the PR title, description, and commits.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 78.54%. Comparing base (f0a05c4) to head (960245c).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/core/api.ts 90.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
+ Coverage   76.51%   78.54%   +2.02%     
==========================================
  Files           8        8              
  Lines         954      960       +6     
  Branches      298      300       +2     
==========================================
+ Hits          730      754      +24     
+ Misses        186      168      -18     
  Partials       38       38              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xwartz xwartz merged commit bee1661 into main Jun 19, 2025
11 checks passed
@xwartz xwartz deleted the dev branch June 19, 2025 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎯 api API related 🐛 bug Something isn't working 🧹 chore Maintenance tasks 🔧 config Configuration related 📦 dependencies Dependencies updates 📚 docs Improvements or additions to documentation 📚 large Large change (> 200 lines) 🧪 test Testing related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants