From ef672da08d8e078bf49ea73631fb46eabf8a5dae Mon Sep 17 00:00:00 2001 From: "Benjamin E. Coe" Date: Wed, 3 Jan 2024 11:49:22 -0500 Subject: [PATCH] test: fix snapshot (#511) build: update release-please action version test: switch module name checked for in assertions --- .github/workflows/release-please.yml | 3 +- test/integration.js | 2 +- test/integration.js.snap | 260 +++++++++++++-------------- 3 files changed, 132 insertions(+), 133 deletions(-) diff --git a/.github/workflows/release-please.yml b/.github/workflows/release-please.yml index ebd973ea..1093afbe 100644 --- a/.github/workflows/release-please.yml +++ b/.github/workflows/release-please.yml @@ -8,12 +8,11 @@ jobs: runs-on: ubuntu-latest if: github.repository == 'bcoe/c8' steps: - - uses: GoogleCloudPlatform/release-please-action@v3 + - uses: GoogleCloudPlatform/release-please-action@v4 id: release with: token: ${{ secrets.GITHUB_TOKEN }} release-type: node - package-name: c8 # The logic below handles the npm publication: - uses: actions/checkout@v3 # these if statements ensure that a publication only occurs when diff --git a/test/integration.js b/test/integration.js index c39a2337..d4aaeb13 100644 --- a/test/integration.js +++ b/test/integration.js @@ -81,7 +81,7 @@ beforeEach(function () { env: { NODE_DEBUG: 'c8' } }) output.toString('utf8').should.match( - /Error: ENOENT: no such file or directory.*loaders\.js/ + /Error: ENOENT: no such file or directory.*loader\.js/ ) }) diff --git a/test/integration.js.snap b/test/integration.js.snap index 821d9907..08cb2402 100644 --- a/test/integration.js.snap +++ b/test/integration.js.snap @@ -153,71 +153,71 @@ what hey what hey ----------------------------------------|---------|----------|---------|---------|------------------- -File | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s ----------------------------------------|---------|----------|---------|---------|------------------- -All files | 1.81 | 12.24 | 6.38 | 1.81 | - c8 | 0 | 0 | 0 | 0 | - index.js | 0 | 0 | 0 | 0 | 1 - c8/bin | 0 | 0 | 0 | 0 | - c8.js | 0 | 0 | 0 | 0 | 1-52 - c8/coverage | 0 | 0 | 0 | 0 | - block-navigation.js | 0 | 0 | 0 | 0 | 1-87 - prettify.js | 0 | 0 | 0 | 0 | 1-2 - sorter.js | 0 | 0 | 0 | 0 | 1-196 - c8/lib | 0 | 0 | 0 | 0 | - is-cjs-esm-bridge.js | 0 | 0 | 0 | 0 | 1-10 - parse-args.js | 0 | 0 | 0 | 0 | 1-224 - report.js | 0 | 0 | 0 | 0 | 1-417 - source-map-from-file.js | 0 | 0 | 0 | 0 | 1-100 - c8/lib/commands | 0 | 0 | 0 | 0 | - check-coverage.js | 0 | 0 | 0 | 0 | 1-70 - report.js | 0 | 0 | 0 | 0 | 1-42 - c8/test/fixtures | 16.3 | 37.5 | 21.42 | 16.3 | - async.js | 100 | 100 | 100 | 100 | - c8-ignore-next.js | 0 | 0 | 0 | 0 | 1-22 - c8-ignore-start-stop.js | 0 | 0 | 0 | 0 | 1-21 - computed-method.js | 0 | 0 | 0 | 0 | 1-15 - custom-ext.special | 75 | 66.66 | 33.33 | 75 | 14-16,18-20 - custom-ext2.special | 0 | 0 | 0 | 0 | 1-24 - issue-254.js | 0 | 0 | 0 | 0 | 1-7 - multiple-spawn.js | 0 | 0 | 0 | 0 | 1-12 - normal.js | 0 | 0 | 0 | 0 | 1-24 - shebang.js | 0 | 0 | 0 | 0 | 1-8 - subprocess.js | 0 | 0 | 0 | 0 | 1-15 - c8/test/fixtures/all/ts-compiled | 0 | 0 | 0 | 0 | - loaded.ts | 0 | 0 | 0 | 0 | 1-19 - main.ts | 0 | 0 | 0 | 0 | 1-4 - c8/test/fixtures/all/ts-compiled/dir | 0 | 0 | 0 | 0 | - unloaded.ts | 0 | 0 | 0 | 0 | 1-5 - c8/test/fixtures/all/vanilla | 0 | 0 | 0 | 0 | - loaded.js | 0 | 0 | 0 | 0 | 1-19 - main.js | 0 | 0 | 0 | 0 | 1-4 - c8/test/fixtures/all/vanilla/dir | 0 | 0 | 0 | 0 | - unloaded.js | 0 | 0 | 0 | 0 | 1-5 - c8/test/fixtures/multidir1 | 0 | 0 | 0 | 0 | - file1.js | 0 | 0 | 0 | 0 | 1 - c8/test/fixtures/multidir2 | 0 | 0 | 0 | 0 | - file2.js | 0 | 0 | 0 | 0 | 1 - c8/test/fixtures/report | 0 | 0 | 0 | 0 | - allowExternal.js | 0 | 0 | 0 | 0 | 1 - report-multi-dir-external.js | 0 | 0 | 0 | 0 | 1-12 - report-single-dir-external.js | 0 | 0 | 0 | 0 | 1-12 - srcOverride.js | 0 | 0 | 0 | 0 | 1 - c8/test/fixtures/source-maps | 0 | 0 | 0 | 0 | - branches.js | 0 | 0 | 0 | 0 | 1-20 - fake-source-map.js | 0 | 0 | 0 | 0 | 1-7 - c8/test/fixtures/source-maps/branches | 0 | 0 | 0 | 0 | - branch-1.js | 0 | 0 | 0 | 0 | 1-12 - branch-2.js | 0 | 0 | 0 | 0 | 1-9 - branches.js | 0 | 0 | 0 | 0 | 1-20 - branches.typescript.ts | 0 | 0 | 0 | 0 | 1-25 - c8/test/fixtures/source-maps/classes | 0 | 0 | 0 | 0 | - class-1.js | 0 | 0 | 0 | 0 | 1-5 - class-2.js | 0 | 0 | 0 | 0 | 1-23 - classes.js | 0 | 0 | 0 | 0 | 1-27 - classes.typescript.ts | 0 | 0 | 0 | 0 | 1-33 ----------------------------------------|---------|----------|---------|---------|------------------- +---------------------------------------|---------|----------|---------|---------|------------------------ +File | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s +---------------------------------------|---------|----------|---------|---------|------------------------ +All files | 3.47 | 12.24 | 6.38 | 3.47 | + c8 | 0 | 0 | 0 | 0 | + index.js | 0 | 0 | 0 | 0 | 1 + c8/bin | 0 | 0 | 0 | 0 | + c8.js | 0 | 0 | 0 | 0 | 1-43 + c8/coverage | 0 | 0 | 0 | 0 | + block-navigation.js | 0 | 0 | 0 | 0 | 1-87 + prettify.js | 0 | 0 | 0 | 0 | 1-2 + sorter.js | 0 | 0 | 0 | 0 | 1-196 + c8/lib | 0 | 0 | 0 | 0 | + is-cjs-esm-bridge.js | 0 | 0 | 0 | 0 | 1-10 + parse-args.js | 0 | 0 | 0 | 0 | 1-224 + report.js | 0 | 0 | 0 | 0 | 1-417 + source-map-from-file.js | 0 | 0 | 0 | 0 | 1-100 + c8/lib/commands | 0 | 0 | 0 | 0 | + check-coverage.js | 0 | 0 | 0 | 0 | 1-70 + report.js | 0 | 0 | 0 | 0 | 1-42 + c8/test/fixtures | 30.97 | 37.5 | 21.42 | 30.97 | + async.js | 100 | 100 | 100 | 100 | + c8-ignore-next.js | 54.54 | 0 | 0 | 54.54 | 1,3-4,9,12,17-19,21-22 + c8-ignore-start-stop.js | 61.9 | 0 | 0 | 61.9 | 1-4,10,16-18 + computed-method.js | 0 | 0 | 0 | 0 | 1-15 + custom-ext.special | 75 | 66.66 | 33.33 | 75 | 14-16,18-20 + custom-ext2.special | 0 | 0 | 0 | 0 | 1-24 + issue-254.js | 28.57 | 0 | 0 | 28.57 | 1-2,5-7 + multiple-spawn.js | 0 | 0 | 0 | 0 | 1-12 + normal.js | 0 | 0 | 0 | 0 | 1-24 + shebang.js | 0 | 0 | 0 | 0 | 1-8 + subprocess.js | 0 | 0 | 0 | 0 | 1-15 + c8/test/fixtures/all/ts-compiled | 0 | 0 | 0 | 0 | + loaded.ts | 0 | 0 | 0 | 0 | 1-19 + main.ts | 0 | 0 | 0 | 0 | 1-4 + c8/test/fixtures/all/ts-compiled/dir | 0 | 0 | 0 | 0 | + unloaded.ts | 0 | 0 | 0 | 0 | 1-5 + c8/test/fixtures/all/vanilla | 0 | 0 | 0 | 0 | + loaded.js | 0 | 0 | 0 | 0 | 1-19 + main.js | 0 | 0 | 0 | 0 | 1-4 + c8/test/fixtures/all/vanilla/dir | 0 | 0 | 0 | 0 | + unloaded.js | 0 | 0 | 0 | 0 | 1-5 + c8/test/fixtures/multidir1 | 0 | 0 | 0 | 0 | + file1.js | 0 | 0 | 0 | 0 | 1 + c8/test/fixtures/multidir2 | 0 | 0 | 0 | 0 | + file2.js | 0 | 0 | 0 | 0 | 1 + c8/test/fixtures/report | 0 | 0 | 0 | 0 | + allowExternal.js | 0 | 0 | 0 | 0 | 1 + report-multi-dir-external.js | 0 | 0 | 0 | 0 | 1-12 + report-single-dir-external.js | 0 | 0 | 0 | 0 | 1-12 + srcOverride.js | 0 | 0 | 0 | 0 | 1 + c8/test/fixtures/source-maps | 0 | 0 | 0 | 0 | + branches.js | 0 | 0 | 0 | 0 | 1-20 + fake-source-map.js | 0 | 0 | 0 | 0 | 1-7 + c8/test/fixtures/source-maps/branches | 0 | 0 | 0 | 0 | + branch-1.js | 0 | 0 | 0 | 0 | 1-12 + branch-2.js | 0 | 0 | 0 | 0 | 1-9 + branches.js | 0 | 0 | 0 | 0 | 1-20 + branches.typescript.ts | 0 | 0 | 0 | 0 | 1-25 + c8/test/fixtures/source-maps/classes | 0 | 0 | 0 | 0 | + class-1.js | 0 | 0 | 0 | 0 | 1-5 + class-2.js | 0 | 0 | 0 | 0 | 1-23 + classes.js | 0 | 0 | 0 | 0 | 1-27 + classes.typescript.ts | 0 | 0 | 0 | 0 | 1-33 +---------------------------------------|---------|----------|---------|---------|------------------------ ," `; @@ -519,71 +519,71 @@ what hey what hey ----------------------------------------|---------|----------|---------|---------|------------------- -File | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s ----------------------------------------|---------|----------|---------|---------|------------------- -All files | 1.81 | 12.24 | 6.38 | 1.81 | - c8 | 0 | 0 | 0 | 0 | - index.js | 0 | 0 | 0 | 0 | 1 - c8/bin | 0 | 0 | 0 | 0 | - c8.js | 0 | 0 | 0 | 0 | 1-52 - c8/coverage | 0 | 0 | 0 | 0 | - block-navigation.js | 0 | 0 | 0 | 0 | 1-87 - prettify.js | 0 | 0 | 0 | 0 | 1-2 - sorter.js | 0 | 0 | 0 | 0 | 1-196 - c8/lib | 0 | 0 | 0 | 0 | - is-cjs-esm-bridge.js | 0 | 0 | 0 | 0 | 1-10 - parse-args.js | 0 | 0 | 0 | 0 | 1-224 - report.js | 0 | 0 | 0 | 0 | 1-417 - source-map-from-file.js | 0 | 0 | 0 | 0 | 1-100 - c8/lib/commands | 0 | 0 | 0 | 0 | - check-coverage.js | 0 | 0 | 0 | 0 | 1-70 - report.js | 0 | 0 | 0 | 0 | 1-42 - c8/test/fixtures | 16.3 | 37.5 | 21.42 | 16.3 | - async.js | 100 | 100 | 100 | 100 | - c8-ignore-next.js | 0 | 0 | 0 | 0 | 1-22 - c8-ignore-start-stop.js | 0 | 0 | 0 | 0 | 1-21 - computed-method.js | 0 | 0 | 0 | 0 | 1-15 - custom-ext.special | 75 | 66.66 | 33.33 | 75 | 14-16,18-20 - custom-ext2.special | 0 | 0 | 0 | 0 | 1-24 - issue-254.js | 0 | 0 | 0 | 0 | 1-7 - multiple-spawn.js | 0 | 0 | 0 | 0 | 1-12 - normal.js | 0 | 0 | 0 | 0 | 1-24 - shebang.js | 0 | 0 | 0 | 0 | 1-8 - subprocess.js | 0 | 0 | 0 | 0 | 1-15 - c8/test/fixtures/all/ts-compiled | 0 | 0 | 0 | 0 | - loaded.ts | 0 | 0 | 0 | 0 | 1-19 - main.ts | 0 | 0 | 0 | 0 | 1-4 - c8/test/fixtures/all/ts-compiled/dir | 0 | 0 | 0 | 0 | - unloaded.ts | 0 | 0 | 0 | 0 | 1-5 - c8/test/fixtures/all/vanilla | 0 | 0 | 0 | 0 | - loaded.js | 0 | 0 | 0 | 0 | 1-19 - main.js | 0 | 0 | 0 | 0 | 1-4 - c8/test/fixtures/all/vanilla/dir | 0 | 0 | 0 | 0 | - unloaded.js | 0 | 0 | 0 | 0 | 1-5 - c8/test/fixtures/multidir1 | 0 | 0 | 0 | 0 | - file1.js | 0 | 0 | 0 | 0 | 1 - c8/test/fixtures/multidir2 | 0 | 0 | 0 | 0 | - file2.js | 0 | 0 | 0 | 0 | 1 - c8/test/fixtures/report | 0 | 0 | 0 | 0 | - allowExternal.js | 0 | 0 | 0 | 0 | 1 - report-multi-dir-external.js | 0 | 0 | 0 | 0 | 1-12 - report-single-dir-external.js | 0 | 0 | 0 | 0 | 1-12 - srcOverride.js | 0 | 0 | 0 | 0 | 1 - c8/test/fixtures/source-maps | 0 | 0 | 0 | 0 | - branches.js | 0 | 0 | 0 | 0 | 1-20 - fake-source-map.js | 0 | 0 | 0 | 0 | 1-7 - c8/test/fixtures/source-maps/branches | 0 | 0 | 0 | 0 | - branch-1.js | 0 | 0 | 0 | 0 | 1-12 - branch-2.js | 0 | 0 | 0 | 0 | 1-9 - branches.js | 0 | 0 | 0 | 0 | 1-20 - branches.typescript.ts | 0 | 0 | 0 | 0 | 1-25 - c8/test/fixtures/source-maps/classes | 0 | 0 | 0 | 0 | - class-1.js | 0 | 0 | 0 | 0 | 1-5 - class-2.js | 0 | 0 | 0 | 0 | 1-23 - classes.js | 0 | 0 | 0 | 0 | 1-27 - classes.typescript.ts | 0 | 0 | 0 | 0 | 1-33 ----------------------------------------|---------|----------|---------|---------|------------------- +---------------------------------------|---------|----------|---------|---------|------------------------ +File | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s +---------------------------------------|---------|----------|---------|---------|------------------------ +All files | 3.47 | 12.24 | 6.38 | 3.47 | + c8 | 0 | 0 | 0 | 0 | + index.js | 0 | 0 | 0 | 0 | 1 + c8/bin | 0 | 0 | 0 | 0 | + c8.js | 0 | 0 | 0 | 0 | 1-43 + c8/coverage | 0 | 0 | 0 | 0 | + block-navigation.js | 0 | 0 | 0 | 0 | 1-87 + prettify.js | 0 | 0 | 0 | 0 | 1-2 + sorter.js | 0 | 0 | 0 | 0 | 1-196 + c8/lib | 0 | 0 | 0 | 0 | + is-cjs-esm-bridge.js | 0 | 0 | 0 | 0 | 1-10 + parse-args.js | 0 | 0 | 0 | 0 | 1-224 + report.js | 0 | 0 | 0 | 0 | 1-417 + source-map-from-file.js | 0 | 0 | 0 | 0 | 1-100 + c8/lib/commands | 0 | 0 | 0 | 0 | + check-coverage.js | 0 | 0 | 0 | 0 | 1-70 + report.js | 0 | 0 | 0 | 0 | 1-42 + c8/test/fixtures | 30.97 | 37.5 | 21.42 | 30.97 | + async.js | 100 | 100 | 100 | 100 | + c8-ignore-next.js | 54.54 | 0 | 0 | 54.54 | 1,3-4,9,12,17-19,21-22 + c8-ignore-start-stop.js | 61.9 | 0 | 0 | 61.9 | 1-4,10,16-18 + computed-method.js | 0 | 0 | 0 | 0 | 1-15 + custom-ext.special | 75 | 66.66 | 33.33 | 75 | 14-16,18-20 + custom-ext2.special | 0 | 0 | 0 | 0 | 1-24 + issue-254.js | 28.57 | 0 | 0 | 28.57 | 1-2,5-7 + multiple-spawn.js | 0 | 0 | 0 | 0 | 1-12 + normal.js | 0 | 0 | 0 | 0 | 1-24 + shebang.js | 0 | 0 | 0 | 0 | 1-8 + subprocess.js | 0 | 0 | 0 | 0 | 1-15 + c8/test/fixtures/all/ts-compiled | 0 | 0 | 0 | 0 | + loaded.ts | 0 | 0 | 0 | 0 | 1-19 + main.ts | 0 | 0 | 0 | 0 | 1-4 + c8/test/fixtures/all/ts-compiled/dir | 0 | 0 | 0 | 0 | + unloaded.ts | 0 | 0 | 0 | 0 | 1-5 + c8/test/fixtures/all/vanilla | 0 | 0 | 0 | 0 | + loaded.js | 0 | 0 | 0 | 0 | 1-19 + main.js | 0 | 0 | 0 | 0 | 1-4 + c8/test/fixtures/all/vanilla/dir | 0 | 0 | 0 | 0 | + unloaded.js | 0 | 0 | 0 | 0 | 1-5 + c8/test/fixtures/multidir1 | 0 | 0 | 0 | 0 | + file1.js | 0 | 0 | 0 | 0 | 1 + c8/test/fixtures/multidir2 | 0 | 0 | 0 | 0 | + file2.js | 0 | 0 | 0 | 0 | 1 + c8/test/fixtures/report | 0 | 0 | 0 | 0 | + allowExternal.js | 0 | 0 | 0 | 0 | 1 + report-multi-dir-external.js | 0 | 0 | 0 | 0 | 1-12 + report-single-dir-external.js | 0 | 0 | 0 | 0 | 1-12 + srcOverride.js | 0 | 0 | 0 | 0 | 1 + c8/test/fixtures/source-maps | 0 | 0 | 0 | 0 | + branches.js | 0 | 0 | 0 | 0 | 1-20 + fake-source-map.js | 0 | 0 | 0 | 0 | 1-7 + c8/test/fixtures/source-maps/branches | 0 | 0 | 0 | 0 | + branch-1.js | 0 | 0 | 0 | 0 | 1-12 + branch-2.js | 0 | 0 | 0 | 0 | 1-9 + branches.js | 0 | 0 | 0 | 0 | 1-20 + branches.typescript.ts | 0 | 0 | 0 | 0 | 1-25 + c8/test/fixtures/source-maps/classes | 0 | 0 | 0 | 0 | + class-1.js | 0 | 0 | 0 | 0 | 1-5 + class-2.js | 0 | 0 | 0 | 0 | 1-23 + classes.js | 0 | 0 | 0 | 0 | 1-27 + classes.typescript.ts | 0 | 0 | 0 | 0 | 1-33 +---------------------------------------|---------|----------|---------|---------|------------------------ ," `;