Skip to content

test: mark timeouts & flaky test as flaky on IBM i #58583

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 6, 2025

Conversation

abmusse
Copy link
Contributor

@abmusse abmusse commented Jun 5, 2025

Underlying issues still need to be investigated
but this will prevent these failures from failing
the daily builds.

Timeouts:

  • parallel.test-http-proxy-fetch
  • parallel.test-https-proxy-fetch
  • benchmark.test-benchmark-websocket
  • parallel.test-inspector-network-fetch
  • es-module.test-wasm-web-api
  • parallel.test-fetch
  • parallel.test-without-async-context-frame
  • report.test-report-exclude-network

Flaky test:

  • parallel.test-process-cpuUsage

Refs: #58582

CC @nodejs/platform-ibmi

@abmusse abmusse requested a review from richardlau June 5, 2025 04:13
@nodejs-github-bot nodejs-github-bot added esm Issues and PRs related to the ECMAScript Modules implementation. needs-ci PRs that need a full CI run. report Issues and PRs related to process.report. test Issues and PRs related to the tests. labels Jun 5, 2025
Underlying issues still need to be investigated
but this will prevent these failures from failing
the daily builds.

Timeouts:
- parallel.test-http-proxy-fetch
- parallel.test-https-proxy-fetch
- benchmark.test-benchmark-websocket
- parallel.test-inspector-network-fetch
- es-module.test-wasm-web-api
- parallel.test-fetch
- parallel.test-without-async-context-frame
- report.test-report-exclude-network

Flaky test:
- parallel.test-process-cpuUsage
@abmusse abmusse force-pushed the ibmi-mark-timeouts-flaky branch from 90d32a5 to 27c2c7f Compare June 5, 2025 04:34
Copy link

codecov bot commented Jun 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.19%. Comparing base (9e35ddc) to head (27c2c7f).
Report is 8 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #58583      +/-   ##
==========================================
- Coverage   90.20%   90.19%   -0.01%     
==========================================
  Files         635      635              
  Lines      187501   187501              
  Branches    36839    36841       +2     
==========================================
- Hits       169133   169121      -12     
- Misses      11146    11148       +2     
- Partials     7222     7232      +10     

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

@joyeecheung joyeecheung added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 5, 2025
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 5, 2025
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@richardlau richardlau added the fast-track PRs that do not need to wait for 48 hours to land. label Jun 5, 2025
Copy link
Contributor

github-actions bot commented Jun 5, 2025

Fast-track has been requested by @richardlau. Please 👍 to approve.

@richardlau richardlau added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 5, 2025
@lpinca lpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 6, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 6, 2025
@nodejs-github-bot nodejs-github-bot merged commit 25f2f27 into nodejs:main Jun 6, 2025
68 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 25f2f27

targos pushed a commit that referenced this pull request Jun 6, 2025
Underlying issues still need to be investigated
but this will prevent these failures from failing
the daily builds.

Timeouts:
- parallel.test-http-proxy-fetch
- parallel.test-https-proxy-fetch
- benchmark.test-benchmark-websocket
- parallel.test-inspector-network-fetch
- es-module.test-wasm-web-api
- parallel.test-fetch
- parallel.test-without-async-context-frame
- report.test-report-exclude-network

Flaky test:
- parallel.test-process-cpuUsage

PR-URL: #58583
Refs: #58582
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
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. esm Issues and PRs related to the ECMAScript Modules implementation. fast-track PRs that do not need to wait for 48 hours to land. needs-ci PRs that need a full CI run. report Issues and PRs related to process.report. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants