Skip to content

fix: fix spelling errors in test files#26765

Open
FrankFMY wants to merge 1 commit intooven-sh:mainfrom
FrankFMY:fix-test-spelling-errors
Open

fix: fix spelling errors in test files#26765
FrankFMY wants to merge 1 commit intooven-sh:mainfrom
FrankFMY:fix-test-spelling-errors

Conversation

@FrankFMY
Copy link

@FrankFMY FrankFMY commented Feb 5, 2026

Summary

  • Fix hadnler -> handler in test/js/web/fetch/abort-signal-leak.test.ts
  • Fix self asign -> self-signed in test/js/node/tls/node-tls-connect.test.ts
  • Fix succesfully -> successfully in test/cli/install/bun-add.test.ts
  • Fix occurences -> occurrences in test/cli/install/hosted-git-info/cases.ts

- abort-signal-leak.test.ts: hadnler -> handler
- node-tls-connect.test.ts: self asign -> self-signed
- bun-add.test.ts: succesfully -> successfully
- hosted-git-info/cases.ts: occurences -> occurrences
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Walkthrough

This pull request contains minor typographical corrections across test files, including comment typos and test naming improvements. No functional or behavioral changes are introduced.

Changes

Cohort / File(s) Summary
CLI Tests
test/cli/install/bun-add.test.ts, test/cli/install/hosted-git-info/cases.ts
Fixed comment typos: "succesfully" corrected to "successfully" and "occurrences" corrected in comments.
JS Runtime Tests
test/js/node/tls/node-tls-connect.test.ts, test/js/web/fetch/abort-signal-leak.test.ts
Fixed test naming: "self asign" corrected to "self-signed" and "hadnler" corrected to "handler".
🚥 Pre-merge checks | ✅ 1 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description provides a summary of all spelling fixes but lacks the structured sections required by the template (What does this PR do? and How did you verify your code works?). Expand the description to follow the template structure with dedicated sections for 'What does this PR do?' and 'How did you verify your code works?'.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: fixing spelling errors across multiple test files.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant