Spy not working when depdendent service is destructured

Hi there,

Following the issue noted in #106, I found out why my spy won’t work, despite following the documentation properly:

Scenario: You’re testing a module and its function called MyModule.getStuff(), which itself depends on another service that you therefore want to stub, say using expect.spyOn(otherService, 'getData');.

Issue: The spy will not be called if MyModule’s getStuff() is imported using ES6 destructuring, e.g.:

/* MyModule.js */

// will be caught by the spy:
const otherService = require('./otherService');

// won't be caught by the spy:
const { getData } = require('./otherService');

That’s what stumped me on the issue for a good day. Changing the import back to the oldschool version made my test with the spy work just fine.

[ljharb]

@batjko because the mechanism by which all kinds of spies in JS functions is by replacing an object property with a spy function. There’s nothing in JS that allows replacing a direct reference to a value with another one when it’s not in scope (via reassignment)

[didaquis]

Maybe this can help: https://stackoverflow.com/questions/60149391/how-to-use-jasmine-js-spy-on-a-required-function