Skip to content

test: apply correction to comment#64524

Open
Trott wants to merge 1 commit into
nodejs:mainfrom
Trott:comment-correction
Open

test: apply correction to comment#64524
Trott wants to merge 1 commit into
nodejs:mainfrom
Trott:comment-correction

Conversation

@Trott

@Trott Trott commented Jul 15, 2026

Copy link
Copy Markdown
Member

The subsequent test checks for an invalid groupAddress, not sourceAddress as the comment say. Adjust the comment to correctly reflect what the test does.

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Jul 15, 2026
The subsequent test checks for an invalid groupAddress, not
sourceAddress as the comment say. Adjust the comment to correctly
reflect what the test does.

Signed-off-by: Rich Trott <rtrott@gmail.com>
@Trott Trott force-pushed the comment-correction branch from 1740f70 to 12d33c9 Compare July 15, 2026 23:26
@Trott Trott added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Jul 16, 2026
@codecov

codecov Bot commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.24%. Comparing base (34c28d5) to head (12d33c9).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #64524      +/-   ##
==========================================
+ Coverage   90.23%   90.24%   +0.01%     
==========================================
  Files         739      739              
  Lines      241639   241639              
  Branches    45547    45543       -4     
==========================================
+ Hits       218031   218057      +26     
+ Misses      15152    15117      -35     
- Partials     8456     8465       +9     

see 35 files with indirect coverage changes

🚀 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants