Skip to content

Chore: run tests daily against node nightly#2969

Merged
Uzlopak merged 2 commits into
nodejs:mainfrom
mweberxyz:chore/nightly-tests
Mar 16, 2024
Merged

Chore: run tests daily against node nightly#2969
Uzlopak merged 2 commits into
nodejs:mainfrom
mweberxyz:chore/nightly-tests

Conversation

@mweberxyz

Copy link
Copy Markdown
Contributor

This relates to...

#2932

Rationale

In order to catch regressions either caused by or a result of upstream Node changes, add a workflow that is invoked on a daily basis to run the main branch's unit tests utilizing the most recent daily build.

In the event of test failure, create a new issue if needed, otherwise comment the most recent open "Nightly tests are failing" issue with the most recent failure.

Changes

  1. Factor tests out of existing Node CI workflow
  2. Add new nightly workflow, triggered on a daily schedule (or alternative, manually via workflow_dispatch to test changes in main without waiting for the next day's scheduled run.)

Features

N/A

Bug Fixes

N/A

Breaking Changes and Deprecations

N/A

Status


jobs:
test:
if: github.repository == 'nodejs/undici'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Be kind to forks, only run in upstream repo.

on:
workflow_dispatch:
schedule:
- cron: "0 10 * * *"

@mweberxyz mweberxyz Mar 15, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most days the nightly lands between 7:30-9:30 - see dates here: https://nodejs.org/download/nightly/

@Uzlopak Uzlopak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Great Job

@mweberxyz

Copy link
Copy Markdown
Contributor Author

Forgot to add:

@codecov-commenter

codecov-commenter commented Mar 15, 2024

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.48%. Comparing base (dcab693) to head (6326bf4).
⚠️ Report is 934 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2969   +/-   ##
=======================================
  Coverage   93.48%   93.48%           
=======================================
  Files          89       89           
  Lines       24171    24171           
=======================================
  Hits        22597    22597           
  Misses       1574     1574           

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

@Uzlopak Uzlopak linked an issue Mar 15, 2024 that may be closed by this pull request

@metcoder95 metcoder95 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Uzlopak Uzlopak merged commit 3be7ec9 into nodejs:main Mar 16, 2024
@Uzlopak

Uzlopak commented Mar 17, 2024

Copy link
Copy Markdown
Contributor

@mweberxyz

Copy link
Copy Markdown
Contributor Author

Thanks for the shout out! Also good news: this thing has been working -- https://github.com/nodejs/undici/actions/workflows/nightly.yml

@mweberxyz mweberxyz deleted the chore/nightly-tests branch March 28, 2024 04:51
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.

test against Node.js nightlies

6 participants