tape-testing / tape

tap-producing test harness for node and browsers
MIT License
5.77k stars 307 forks source link

es-abstract tests blow the stack in node < 6 #598

Open ljharb opened 1 year ago

ljharb commented 1 year ago

See https://github.com/ljharb/es-abstract/commit/156bf41854821be8cdd4f5d595e7e7299ff401aa.

When that flag is removed, tape will blow the stack running tests in es-abstract - nvm exec 6.0 npm run tests-only passes, but nvm exec 5 npm run tests-only fails. nvm exec 5 node test/bufferTestCases also fails if you want a smaller repro.

I haven't yet been able to improve tape's stack performance by enough to get the tests to complete in node < 6 without providing the --stack-size option.