tape failing on multiple versions and platforms
Open
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 606
- Forks
- 157
- Avg merge
- 3d 23h
- Merged PRs (30d)
- 3
Description
Extracting this from #894 because I have discovered that there are different failures across multiple versions/platforms.
Most platforms fail with this error [Resolved]:
> tape@5.5.2 prepublish
> !(type not-in-publish) || not-in-publish || npm run prepublishOnly
not-in-publish is /home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/.bin/not-in-publish
added 759 packages in 43s
> tape@5.5.2 pretest
> npm run lint
> tape@5.5.2 prelint
> eclint check $(git ls-files 2>/dev/null | xargs find 2> /dev/null | grep -vE 'node_modules|\.git' || echo '*.md *.js test/*.js')
node:events:504
throw er; // Unhandled 'error' event
^
Error: File not found with singular glob: /home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/test/has (if this was purposeful, use `allowEmpty` option)
at Glob.<anonymous> (/home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/glob-stream/readable.js:84:17)
at Object.onceWrapper (node:events:646:26)
at Glob.emit (node:events:526:28)
at Glob._finish (/home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/glob/glob.js:194:8)
at done (/home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/glob/glob.js:179:14)
at Glob._processSimple2 (/home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/glob/glob.js:685:12)
at /home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/glob/glob.js:673:10
at Glob._stat2 (/home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/glob/glob.js:769:12)
at lstatcb_ (/home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/glob/glob.js:761:12)
at RES (/home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/inflight/inflight.js:31:16)
Emitted 'error' event on DestroyableTransform instance at:
at Pumpify.emit (node:events:526:28)
at Pumpify.Duplexify._destroy (/home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/duplexify/index.js:191:15)
at /home/iojs/tmp/citgm_tmp/bc3d8cd6-beed-4db0-ae20-8394e5f7f0c5/tape/node_modules/duplexify/index.js:182:10
at processTicksAndRejections (node:internal/process/task_queues:78:11)
Windows bails out earlier with a separate error:
> tape@5.5.2 prepublish
> !(type not-in-publish) || not-in-publish || npm run prepublishOnly
added 759 packages in 34s
> tape@5.5.2 pretest
> npm run lint
> tape@5.5.2 prelint
> eclint check $(git ls-files 2>/dev/null | xargs find 2> /dev/null | grep -vE 'node_modules|\.git' || echo '*.md *.js test/*.js')
'!' is not recognized as an internal or external command,
operable program or batch file.
The system cannot find the path specified.
Interestingly macOS and AIX seem to consistently pass:
- osx1015 - https://ci.nodejs.org/job/citgm-smoker-nobuild/nodes=osx1015/1150/console
- aix71-ppc64 - https://ci.nodejs.org/job/citgm-smoker-nobuild/nodes=aix71-ppc64/1150/console
Test run links:
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the prepublish, pretest, and prelint scripts and reproduce the linked Node.js v17.8.0, v16.14.2, and v14.19.1 test runs. Compare the eclint command failure involving test/has with the Windows shell error. Done means the pretest checks pass consistently on the affected versions and platforms.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, nodejs
- Domain
- ci-cd, testing-qa
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100