Skip to content
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

test_runner: finish marking snapshot testing as stable #56425

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Jan 1, 2025

Snapshot testing was marked stable in #55897. These were overlooked at the time.

Refs: #55897

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/test_runner

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test_runner Issues and PRs related to the test runner subsystem. labels Jan 1, 2025
@cjihrig cjihrig added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 1, 2025
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 1, 2025
@nodejs-github-bot
Copy link
Collaborator

doc/api/test.md Show resolved Hide resolved
Copy link

codecov bot commented Jan 1, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.53%. Comparing base (0dbbaba) to head (08e500f).
Report is 16 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #56425      +/-   ##
==========================================
- Coverage   88.53%   88.53%   -0.01%     
==========================================
  Files         657      657              
  Lines      190718   190738      +20     
  Branches    36603    36609       +6     
==========================================
+ Hits       168861   168869       +8     
- Misses      15024    15041      +17     
+ Partials     6833     6828       -5     
Files with missing lines Coverage Δ
lib/internal/test_runner/snapshot.js 100.00% <100.00%> (ø)

... and 37 files with indirect coverage changes

Snapshot testing was marked stable in nodejs#55897. These were
overlooked at the time.

Refs: nodejs#55897
@cjihrig cjihrig added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 2, 2025
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 2, 2025
@nodejs-github-bot
Copy link
Collaborator

@cjihrig cjihrig 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. and removed needs-ci PRs that need a full CI run. labels Jan 2, 2025
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 3, 2025
@nodejs-github-bot nodejs-github-bot merged commit afafee2 into nodejs:main Jan 3, 2025
65 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in afafee2

@cjihrig cjihrig deleted the oops branch January 3, 2025 19:20
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. test_runner Issues and PRs related to the test runner subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants