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

npm run build -> Error in TestSuite "include edges in canvas size calculation" #168

Open
vashnt707 opened this issue Apr 29, 2022 · 2 comments

Comments

@vashnt707
Copy link

vashnt707 commented Apr 29, 2022

npm run build throws the following error. Anything needs to be changed. This is on top of latest git clone from repo.

$ npm run build

> nomnoml@1.5.1 build
> npm run lib && npm run webapp && npm run test


> nomnoml@1.5.1 lib
> node build/build-parser.js && rollup -c src/rollup.config.js && npm test


src/index.ts → dist/nomnoml.js...
created dist/nomnoml.js in 7.5s

> nomnoml@1.5.1 test
> node test/index.js && npm run test_cli

include edges in canvas size calculation
   Error: 288 ≯ 300
V:\nomnoml\test\TestSuite.js:61
    if (failures.length) throw new Error(failures.length + ' test failures')
                               ^

Error: 1 test failures
    at report_console (V:\nomnoml\test\TestSuite.js:61:32)
    at Object.report (V:\nomnoml\test\TestSuite.js:26:10)
npm notice
npm notice New minor version of npm available! 8.5.5 -> 8.8.0
npm notice Changelog: <https://github.com/npm/cli/releases/tag/v8.8.0>
npm notice Run `npm install -g npm@8.8.0` to update!
npm notice
npm notice
npm notice New minor version of npm available! 8.5.5 -> 8.8.0
npm notice Changelog: <https://github.com/npm/cli/releases/tag/v8.8.0>
npm notice Run `npm install -g npm@8.8.0` to update!
npm notice
@skanaar
Copy link
Owner

skanaar commented May 2, 2022

Thank you for reporting.

I cannot reproduce this. What system are you running?

@vashnt707
Copy link
Author

Win 10 using either CMD prompt or Git bash with latest node.

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

2 participants