`tape` is loaded even without the `testing: tape` option about log-process-errors HOT 10 CLOSED

Hi @jgierer12, thanks for reporting this issue!

Are you using the testing option with tape as a value?

from log-process-errors.

No, I'm not using any custom config at all.

from log-process-errors.

Thanks. Are you using register/tape? https://github.com/ehmicky/log-process-errors/blob/master/docs/API.md#testing

from log-process-errors.

Nope, nothing test-related at all actually. Only a sibling Yarn workspace but outside the scope of the relevant package is using Jest.

from log-process-errors.

Ok so some background: the testing option is making unit tests fail when a process error is raised.

Each test runner works differently. For tape, I create a failing test using require('tape').test('..', t => t.error(error)). require('tape') is dynamically required and supposed to only be performed when the testing option is set to tape.

If you have a minimal reproduction demo, I would love to fix this bug, because this might impact other users with the same setup. Thanks!

from log-process-errors.

@jgierer12 Did you manage to reproduce it in a more minimal setup? Thanks!

from log-process-errors.

No, sorry, I tried for a while but didn't manage to reproduce this in a setup which I would be able to publish 😕

It must have something to do with the other dependencies I'm using, since I haven't been able to reproduce just with workspaces + netlify-lambda. On the upside, that probably makes it obscure enough for few other people to run into.

from log-process-errors.

Does it work if you uninstall tape? If not, could you please edit node_modules/log-process-errors/build/src/options/runners.js and add a console.trace() just before require('tape')? Like this you might get a better stack trace of what is calling that function.

from log-process-errors.

@jgierer12 if you could check this, this would help me fix this problem potentially for some users. Thanks!

from log-process-errors.

Closing but @jgierer12 feel free to comment.

from log-process-errors.