Make test for no proxy side-effects allow for no side-effects.

[jdalton]

Currently the tests for no proxy side-effects assume at least the ['getOwnPropertyDescriptor'] trap is triggered. When running say against require('node:util/types').isRegExp() it has for real real no side-effects and doesn't trigger ['getOwnPropertyDescriptor'], instead the result is []. The assumption could be tweaked to have ['getOwnPropertyDescriptor'] or less []. In my case I'm running against your unit tests so no pressure to change there. Just passing it along as a seen difference. If the tests are updated here I could pull them in directly.

[ljharb]

I'm not sure I understand - are you asking for the test to be slightly less precise so as to encompass node's builtin isRegExp?

That seems reasonable since the point is indeed to have no more traps than that.

[jdalton]

Yes!

I modified the test by:

handler.trapCalls.length ? ['getOwnPropertyDescriptor'] : []