GeezerVison

   ⚠  No approving reviews found
   ℹ  request-ci label was added by a Collaborator after the last push event.
- Validating Jenkins credentials
✔  Jenkins credentials valid
- Starting PR CI job
✔  PR CI job successfully started
[DEBUG] SyntaxError: Unexpected token '<', ")

at Request.json (file:///opt/hostedtoolcache/node/20.15.0/x64/lib/node_modules/@node-core/utils/lib/request.js:53:19)

at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

at async Request.query (file:///opt/hostedtoolcache/node/20.15.0/x64/lib/node_modules/@node-core/utils/lib/request.js:227:20)

at async Request.gql (file:///opt/hostedtoolcache/node/20.15.0/x64/lib/node_modules/@node-core/utils/lib/request.js:118:22)

at async PRData.getPR (file:///opt/hostedtoolcache/node/20.15.0/x64/lib/node_modules/@node-core/utils/lib/pr_data.js:82:20)

at async RunPRJob.start (file:///opt/hostedtoolcache/node/20.15.0/x64/lib/node_modules/@node-core/utils/lib/ci/run_ci.js:106:7)

at async RunPRJobCommand.start (file:///opt/hostedtoolcache/node/20.15.0/x64/lib/node_modules/@node-core/utils/bin/ncu-ci.js:300:11)

✘  Failed to start CI

- Loading data for nodejs/node/pull/53853
✔  Done loading data for nodejs/node/pull/53853
----------------------------------- PR info ------------------------------------
Title      test_runner: refactor snapshots to support multiple files in the same process (#53853)
Author     Colin Ihrig <[email protected]> (@cjihrig)
Branch     cjihrig:snapshot -> nodejs:main
Labels     semver-minor, author ready, test_runner
Commits    2
 - test_runner: add context.filePath
 - test_runner: refactor snapshots to get file from context
Committers 1
 - cjihrig <[email protected]>
PR-URL: https://github.com/nodejs/node/pull/53853
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Chemi Atlow <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/53853
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Chemi Atlow <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
--------------------------------------------------------------------------------
   ℹ  This PR was created on Mon, 15 Jul 2024 05:09:30 GMT
   ✔  Approvals: 3
   ✔  - Moshe Atlow (@MoLow) (TSC): https://github.com/nodejs/node/pull/53853#pullrequestreview-2178415741
   ✔  - Chemi Atlow (@atlowChemi): https://github.com/nodejs/node/pull/53853#pullrequestreview-2178504436
   ✔  - Benjamin Gruenbaum (@benjamingr) (TSC): https://github.com/nodejs/node/pull/53853#pullrequestreview-2178740224
   ✔  Last GitHub CI successful
   ℹ  Last Full PR CI on 2024-07-15T20:38:54Z: https://ci.nodejs.org/job/node-test-pull-request/60343/
- Querying data for job/node-test-pull-request/60343/
   ✔  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 53853
From https://github.com/nodejs/node
 * branch                  refs/pull/53853/merge -> FETCH_HEAD
✔  Fetched commits as 0b1ff6965e3b..f4bff7dca35b
--------------------------------------------------------------------------------
[main 60a4b34380] test_runner: add context.filePath
 Author: cjihrig <[email protected]>
 Date: Sun Jul 14 23:37:10 2024 -0400
 4 files changed, 84 insertions(+), 2 deletions(-)
 create mode 100644 test/parallel/test-runner-test-filepath.js
[main 9cfc8e0704] test_runner: refactor snapshots to get file from context
 Author: cjihrig <[email protected]>
 Date: Sun Jul 14 20:33:13 2024 -0400
 5 files changed, 171 insertions(+), 143 deletions(-)
 create mode 100644 test/fixtures/test-runner/snapshots/imported-tests.js
   ✔  Patches applied
There are 2 commits in the PR. Attempting autorebase.
Rebasing (2/4)
Executing: git node land --amend --yes

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

test_runner: add context.filePath
This commit adds a filePath getter to the TestContext and

SuiteContext classes. This allows a context to be mapped back to

the original test file that created it, even if it was imported

from another file. This is useful for mapping features like test

snapshots to the correct test file. This is also prep work for

supporting running test files in the test runner process.
PR-URL: #53853

Reviewed-By: Moshe Atlow <[email protected]>

Reviewed-By: Chemi Atlow <[email protected]>

Reviewed-By: Benjamin Gruenbaum <[email protected]>
[detached HEAD 40a472110e] test_runner: add context.filePath

Author: cjihrig <[email protected]>

Date: Sun Jul 14 23:37:10 2024 -0400

4 files changed, 84 insertions(+), 2 deletions(-)

create mode 100644 test/parallel/test-runner-test-filepath.js

Rebasing (3/4)

Rebasing (4/4)
Executing: git node land --amend --yes

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

test_runner: refactor snapshots to get file from context
This commit refactors the internals of snapshot tests to get the

name of the test file from the test context instead of passing

it to the SnapshotManager constructor. This is prep work for

supporting running test files in the test runner process.
PR-URL: #53853

Reviewed-By: Moshe Atlow <[email protected]>

Reviewed-By: Chemi Atlow <[email protected]>

Reviewed-By: Benjamin Gruenbaum <[email protected]>
[detached HEAD 62430463a2] test_runner: refactor snapshots to get file from context

Author: cjihrig <[email protected]>

Date: Sun Jul 14 20:33:13 2024 -0400

5 files changed, 171 insertions(+), 143 deletions(-)

create mode 100644 test/fixtures/test-runner/snapshots/imported-tests.js
Successfully rebased and updated refs/heads/main.
ℹ  Add commit-queue-squash label to land the PR as one commit, or commit-queue-rebase to land as separate commits.