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

error from convertLcovToCoveralls with nyc version 14 #219

Open
zhaoqin-github opened this issue Apr 22, 2019 · 4 comments
Open

error from convertLcovToCoveralls with nyc version 14 #219

zhaoqin-github opened this issue Apr 22, 2019 · 4 comments

Comments

@zhaoqin-github
Copy link

zhaoqin-github commented Apr 22, 2019

My Travis CI job fails with the latest nyc version 14.0.0

> [email protected] coveralls /home/travis/build/F5Networks/f5-adcaas-openstack/app/waf
> lb-nyc report --reporter=text-lcov | coveralls ../../
[error] "2019-04-22T02:23:19.206Z"  'error from lcovParse: ' 'Failed to parse string'
[error] "2019-04-22T02:23:19.208Z"  'input: ' ''
[error] "2019-04-22T02:23:19.208Z"  'error from convertLcovToCoveralls'
/home/travis/build/F5Networks/f5-adcaas-openstack/app/waf/node_modules/coveralls/bin/coveralls.js:18
        throw err;
        ^
Failed to parse string

After I downgrade nyc version to 13.3.0, coveralls command works perfectly.

@stevenvachon
Copy link

I think this is a duplicate of #207

@zhaoqin-github
Copy link
Author

@stevenvachon My node version is 8.9.4. I guess the lcov report format made by nyc v14 might have some small changes.

You may see the error at the end of my Travis log.

https://travis-ci.org/F5Networks/f5-adcaas-openstack/builds/522088170

@zhaoqin-github
Copy link
Author

@stevenvachon I save the nyc output to files. Nyc.13.txt is made by nyc v13.3.0, which can be processed by coveralls. Nyc.14.txt is made by nyc v14.0.0, which can not be processed by coveralls. If you or anybody can help to figure out the root cause, that will be great.

nyc.13.txt

nyc.14.txt

pithu added a commit to js-joda/js-joda that referenced this issue Jul 3, 2019
Downgrading could fix the issue with coveralls as described here nickmerwin/node-coveralls#219
@XhmikosR
Copy link
Contributor

Works fine in the PR I have #235

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

No branches or pull requests

3 participants