search

skanaar / nomnoml

The sassy UML diagram renderer
https://www.nomnoml.com
MIT License
2.66k stars 208 forks source link

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

Open vashnt707 opened 2 years ago

vashnt707 commented 2 years ago

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 commented 2 years ago

Thank you for reporting.

I cannot reproduce this. What system are you running?

vashnt707 commented 2 years ago

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