Skip to content

test_runner: report file in test runner events - #46030

Merged
nodejs-github-bot merged 2 commits into
nodejs:mainfrom
MoLow:test-runner-report-files
Jan 2, 2023
Merged

test_runner: report file in test runner events#46030
nodejs-github-bot merged 2 commits into
nodejs:mainfrom
MoLow:test-runner-report-files

Conversation

@MoLow

@MoLowMoLow commented Dec 31, 2022•
edited
Loading

Copy link
Copy Markdown
Member

this adds the test file related to the current event.
this will be specifically useful if we implement #45833 (comment) since that will then be the only way to access this information

@nodejs-github-bot

nodejs-github-bot commented Dec 31, 2022

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/test_runner

@nodejs-github-botnodejs-github-bot added lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Dec 31, 2022
@MoLowMoLow added the test_runner Issues and PRs related to the test runner subsystem. label Dec 31, 2022
@MoLow

MoLow commented Jan 2, 2023

Copy link
Copy Markdown
MemberAuthor

CC @nodejs/test_runner

@MoLowMoLow added the request-ci Add this label to start a Jenkins CI on a PR. label Jan 2, 2023
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 2, 2023
@nodejs-github-bot

nodejs-github-bot commented Jan 2, 2023

Copy link
Copy Markdown
Collaborator

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

@nodejs-github-bot

nodejs-github-bot commented Jan 2, 2023

Copy link
Copy Markdown
Collaborator

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

@MoLowMoLow 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 Jan 2, 2023
cjihrig
cjihrig reviewed Jan 2, 2023
View reviewed changes
Comment threadtest/fixtures/test-runner/custom_reporters/custom.js Outdated
Comment threaddoc/api/test.md Outdated
@MoLowMoLow removed commit-queue Add this label to land a pull request using GitHub Actions. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Jan 2, 2023
@MoLow
MoLow requested a review from cjihrigJanuary 2, 2023 18:15
@MoLowMoLow added the commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. label Jan 2, 2023
@MoLowMoLow added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Jan 2, 2023
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 2, 2023
@nodejs-github-bot

nodejs-github-bot commented Jan 2, 2023

Copy link
Copy Markdown
Collaborator

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

@MoLowMoLow added the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 2, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 2, 2023
@nodejs-github-bot
nodejs-github-bot merged commit 9eb363a into nodejs:mainJan 2, 2023
@nodejs-github-bot

nodejs-github-bot commented Jan 2, 2023

Copy link
Copy Markdown
Collaborator

Landed in 9eb363a

@MoLow
MoLow deleted the test-runner-report-files branch January 2, 2023 21:51
@MoLow

MoLow commented Jan 18, 2023

Copy link
Copy Markdown
MemberAuthor

I won't be in front a computer for another week or so, will handle then

@juanarbol

juanarbol commented Jan 24, 2023

Copy link
Copy Markdown
Member

After resolving multiple merge conflicts, this breaks some tests in the v18.x release line.

@MoLowMoLow mentioned this pull request Jan 26, 2023
Closed
@MoLow

MoLow commented Jan 26, 2023

Copy link
Copy Markdown
MemberAuthor

This commit didn't land cleanly on v19.x-staging, can you create a manual backport? I believe it didn't work because it depends on #45712, which is blocked.

@RafaelGSS Why is #45712 blocked?

MoLow added a commit to MoLow/node that referenced this pull request Jan 26, 2023
@MoLow
test_runner: report file in test runner events
cfa5c37
PR-URL: nodejs#46030
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@MoLowMoLow mentioned this pull request Jan 26, 2023
Closed
@RafaelGSS

RafaelGSS commented Jan 26, 2023•
edited
Loading

Copy link
Copy Markdown
Member

This commit didn't land cleanly on v19.x-staging, can you create a manual backport? I believe it didn't work because it depends on #45712, which is blocked.

@RafaelGSS Why is #45712 blocked?

Because of #45712 (comment).

if that was already addressed/released. Feel free to remove the block labels.

MoLow added a commit to MoLow/node that referenced this pull request Jan 26, 2023
@MoLow
test_runner: report file in test runner events
d90d1b3
PR-URL: nodejs#46030
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
ruyadorno pushed a commit that referenced this pull request Jan 31, 2023
@MoLow@ruyadorno
test_runner: report file in test runner events
a49e17e
Backport-PR-URL: #46361
PR-URL: #46030
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@ruyadornoruyadorno mentioned this pull request Feb 1, 2023
Merged
MoLow added a commit to MoLow/node-core-test that referenced this pull request Feb 6, 2023
@MoLow
feat: report file in test runner events
bf09974
PR-URL: nodejs/node#46030
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
(cherry picked from commit 9eb363a3e00dbba572756c7ed314273f17ea8e2e)
MoLow added a commit to MoLow/node-core-test that referenced this pull request Feb 7, 2023
@MoLow
feat: report file in test runner events
a47cacd
PR-URL: nodejs/node#46030
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
(cherry picked from commit 9eb363a3e00dbba572756c7ed314273f17ea8e2e)
MoLow added a commit to MoLow/node-core-test that referenced this pull request Feb 7, 2023
@MoLow
feat: report file in test runner events
f501f9c
PR-URL: nodejs/node#46030
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
(cherry picked from commit 9eb363a3e00dbba572756c7ed314273f17ea8e2e)
MoLow added a commit to MoLow/node-core-test that referenced this pull request Feb 7, 2023
@MoLow
feat: report file in test runner events
29c73b4
PR-URL: nodejs/node#46030
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
(cherry picked from commit 9eb363a3e00dbba572756c7ed314273f17ea8e2e)
MoLow added a commit to nodejs/node-core-test that referenced this pull request Feb 8, 2023
@MoLow
feat: report file in test runner events
10d6603
PR-URL: nodejs/node#46030
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
(cherry picked from commit 9eb363a3e00dbba572756c7ed314273f17ea8e2e)
MoLow added a commit to MoLow/node that referenced this pull request Feb 25, 2023
@MoLow
test_runner: report file in test runner events
b628161
PR-URL: nodejs#46030
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
juanarbol pushed a commit that referenced this pull request Mar 3, 2023
@MoLow@juanarbol
test_runner: report file in test runner events
b4fca7c
PR-URL: #46030
Backport-PR-URL: #46839
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@juanarboljuanarbol mentioned this pull request Mar 3, 2023
Merged
juanarbol pushed a commit that referenced this pull request Mar 5, 2023
@MoLow@juanarbol
test_runner: report file in test runner events
79f4b42
PR-URL: #46030
Backport-PR-URL: #46839
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@benjamingrbenjamingrbenjamingr approved these changes
@cjihrigcjihrigcjihrig approved these changes
@aduh95aduh95aduh95 approved these changes

Assignees

No one assigned

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.commit-queue-squashAdd this label to instruct the Commit Queue to squash all the PR commits into the first one.lib / srcIssues and PRs related to general changes in the lib or src directory.needs-ciPRs that need a full CI run.test_runnerIssues and PRs related to the test runner subsystem.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@MoLow@nodejs-github-bot@RafaelGSS@juanarbol@benjamingr@cjihrig@aduh95