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: update tap + fix tests #426

Closed
wants to merge 1 commit into from

Conversation

louis-bompart
Copy link
Contributor

@louis-bompart louis-bompart commented Jan 28, 2022

I had the following error when running npm test

> semver@7.3.5 test
> tap

TypeError: Cannot read properties of undefined (reading 'getFileName')
    at module.exports (node-semver\node_modules\tap\node_modules\caller-path\index.js:4:40) 
    at importJSX (node-semver\node_modules\tap\node_modules\import-jsx\index.js:24:46)      
    at module.exports (node-semver\node_modules\tap\node_modules\treport\lib\index.js:13:18)
    at exports.makeReporter (node-semver\node_modules\tap\bin\run.js:422:23)
    at runTests (node-semver\node_modules\tap\bin\run.js:744:3)
    at mainAsync (node-semver\node_modules\tap\bin\run.js:244:5)
TAP version 13
1..0
# time=1.581ms

Updating tap allowed the to run but failed on test/functions/coerce.js:

 FAIL ​ test/functions/coerce.js 1 failed of 93 43.195ms
 ✖ t.done is not a function

I replaced t.done with t.end, which, as far as I can tell, is used everywhere else.

The test run was then fully green.

References

Related to #401 - Actually the same, just with a more recent version

 - Updated tap
 - Replaced sole occurence of `tap.done()` by `tap.end()`
@lukekarrys
Copy link
Member

Closing in favor of #433

@lukekarrys lukekarrys closed this Feb 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants