test: make test-inspector-network-resource sequential

[islandryu]

Refs: #59090
I temporarily moved test-inspector-network-resource.js to sequential, and the issue stopped occurring.

https://ci.nodejs.org/job/node-test-binary-windows-js-suites/35657/
https://ci.nodejs.org/job/node-test-binary-windows-js-suites/35658/
https://ci.nodejs.org/job/node-test-binary-windows-js-suites/35659/

However, since I can’t reproduce it in my local environment, it might take some time to apply a proper fix.
Since the root cause is still unknown, it could happen again if another test is added.

[nodejs-github-bot]

Review requested:

• @nodejs/config
• @nodejs/gyp
• @nodejs/inspector

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.06%. Comparing base (b16a0e7) to head (460d999).
⚠️ Report is 69 commits behind head on main.

Additional details and impacted files

@@ Coverage Diff @@## main #59104 +/- ##
==========================================
+ Coverage 90.05% 90.06% +0.01% 
==========================================
Files 645 648 +3 Lines 189153 191025 +1872 Branches 37091 37452 +361 ==========================================
+ Hits 170348 172056 +1708 - Misses 11528 11603 +75 - Partials 7277 7366 +89 

see 97 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.

[StefanStojanovic]

Seems reasonable as long as we do not close the original issue.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/68239/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/68254/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/68256/

[nodejs-github-bot]

Commit Queue failed

- Loading data for nodejs/node/pull/59104
✔ Done loading data for nodejs/node/pull/59104
----------------------------------- PR info ------------------------------------
Title test: make test-inspector-network-resource sequential (#59104)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch islandryu:fix/test-inspector-network-fetch.js -> nodejs:main
Labels c++, lib / src, needs-ci, commit-queue-rebase
Commits 2
- test: make test-inspector-network-resource sequential
- remove test-inspector-network-fetch from flaky
Committers 1
- islandryu <shimaryuhei@gmail.com>
PR-URL: https://github.com/nodejs/node/pull/59104
Refs: https://github.com/nodejs/node/issues/59090
Reviewed-By: Stefan Stojanovic <stefan.stojanovic@janeasystems.com>
Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/59104
Refs: https://github.com/nodejs/node/issues/59090
Reviewed-By: Stefan Stojanovic <stefan.stojanovic@janeasystems.com>
Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Thu, 17 Jul 2025 16:23:19 GMT
✔ Approvals: 2
✔ - Stefan Stojanovic (@StefanStojanovic): https://github.com/nodejs/node/pull/59104#pullrequestreview-3050717647
✔ - Kohei Ueno (@cola119): https://github.com/nodejs/node/pull/59104#pullrequestreview-3059520385
✔ Last GitHub CI successful
ℹ Last Full PR CI on 2025-07-28T02:43:45Z: https://ci.nodejs.org/job/node-test-pull-request/68256/
- Querying data for job/node-test-pull-request/68256/
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ No git cherry-pick in progress
✔ No git am in progress
✔ No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
* branch main -> FETCH_HEAD
✔ origin/main is now up-to-date
- Downloading patch for 59104
From https://github.com/nodejs/node
* branch refs/pull/59104/merge -> FETCH_HEAD
✔ Fetched commits as 0ba6e0d7ce8c..460d99992a0c
--------------------------------------------------------------------------------
[main ce98ceb183] test: make test-inspector-network-resource sequential
Author: islandryu <shimaryuhei@gmail.com>
Date: Thu Jul 17 18:22:12 2025 +0900
1 file changed, 0 insertions(+), 0 deletions(-)
rename test/{parallel => sequential}/test-inspector-network-resource.js (100%)
Auto-merging test/parallel/parallel.status
[main f5f64e9c95] remove test-inspector-network-fetch from flaky
Author: islandryu <shimaryuhei@gmail.com>
Date: Thu Jul 24 15:36:55 2025 +0900
1 file changed, 2 deletions(-)
✔ Patches applied
There are 2 commits in the PR. Attempting autorebase.
Rebasing (2/4)
Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
test: make test-inspector-network-resource sequential
PR-URL: #59104

Refs: #59090

Reviewed-By: Stefan Stojanovic <stefan.stojanovic@janeasystems.com>

Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
[detached HEAD 29499d42fe] test: make test-inspector-network-resource sequential

Author: islandryu <shimaryuhei@gmail.com>

Date: Thu Jul 17 18:22:12 2025 +0900

1 file changed, 0 insertions(+), 0 deletions(-)

rename test/{parallel => sequential}/test-inspector-network-resource.js (100%)

Rebasing (3/4)

Rebasing (4/4)

Executing: git node land --amend --yes

--------------------------------- New Message ----------------------------------

remove test-inspector-network-fetch from flaky
PR-URL: #59104

Refs: #59090

Reviewed-By: Stefan Stojanovic <stefan.stojanovic@janeasystems.com>

Reviewed-By: Kohei Ueno <kohei.ueno119@gmail.com>
[detached HEAD a78cfecd10] remove test-inspector-network-fetch from flaky

Author: islandryu <shimaryuhei@gmail.com>

Date: Thu Jul 24 15:36:55 2025 +0900

1 file changed, 2 deletions(-)

Successfully rebased and updated refs/heads/main.
✔ 29499d42fe7465da882006cfb97c880c1e580030

✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer

✔ 0:0 skipping fixes-url fixes-url

✔ 0:0 blank line after title line-after-title

✔ 0:0 line-lengths are valid line-length

✔ 0:0 metadata is at end of message metadata-end

✔ 1:8 PR-URL is valid. pr-url

✔ 0:0 reviewers are valid reviewers

✔ 0:0 valid subsystems subsystem

✔ 0:0 Title is formatted correctly. title-format

⚠ 0:50 Title should be <= 50 columns. title-length

✖ a78cfecd10f681076d1aa69c16b144dc0e94a1c4

✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer

✔ 0:0 skipping fixes-url fixes-url

✔ 0:0 blank line after title line-after-title

✔ 0:0 line-lengths are valid line-length

✔ 0:0 metadata is at end of message metadata-end

✔ 1:8 PR-URL is valid. pr-url

✔ 0:0 reviewers are valid reviewers

✖ 0:0 Missing subsystem. subsystem

✔ 0:0 Title is formatted correctly. title-format

✔ 0:0 Title is <= 50 columns. title-length
ℹ Please fix the commit message and try again.

Please manually ammend the commit message, by running
git commit --amend

Once commit message is fixed, finish the landing command running
git node land --continue

https://github.com/nodejs/node/actions/runs/16569856429

[islandryu]

Oh, I accidentally did a rebase.

[nodejs-github-bot]

Landed in 04c5a18

[joyeecheung]

This is still failing in the CI: https://github.com/nodejs/reliability/blob/main/reports/2025-08-26.md