fix: filter out Symbol.hasInstance

[nieyuyao]

fix #1671

[QGerrit]

This unfortunately doesn't seem to quite be enough to fix #1671.... at least fully. After manually applying this patch locally, my tests started failing with TypeError: Cannot read properties of undefined (reading 'dataEmitted'). jsdom test environment in a very basic vue3 app.... oddly, dataEmitted doesn't seem to appear anywhere in my node_modules tree.

[nieyuyao]

This unfortunately doesn't seem to quite be enough to fix #1671.... at least fully. After manually applying this patch locally, my tests started failing with TypeError: Cannot read properties of undefined (reading 'dataEmitted'). jsdom test environment in a very basic vue3 app.... oddly, dataEmitted doesn't seem to appear anywhere in my node_modules tree.

emmm.... I tried to find the reason and converted the PR into draft

[nieyuyao]

This TypeError: Cannot read properties of undefined (reading 'dataEmitted') comes from here. Try to read the readableDidRead property and assign to the container when mocking. However, the Readable has not been initialized yet, so an error is thrown.

[QGerrit]

Wow, good find! I forgot to mention I was doing vi.mock('axios'), that break location makes more sense now.

[simon-abbott]

I don't like this solution. There are valid cases where we do want to mock Symbol.hasInstance, and, as you've noticed, this doesn't actually fix the root cause.

I'm working on a fix for the underlying issue which should be up shortly.

[simon-abbott]

I've solved #1671 in a more general way over in #1786.