Testing onMessage listeners

[clarkbw]

Testing onMessage listeners

[clarkbw]

Looks like we check that we call the callback

jest-webextension-mock/__tests__/tabs.test.js

Line 120 in f66b8b8

expect(callback).toHaveBeenCalledTimes(1);

And yes, looks like you're right that we don't pass it along

jest-webextension-mock/src/runtime.js

Line 18 in f66b8b8

sendMessage: jest.fn((message, cb) => {

Can you check that this is the correct behaviour and open a PR fixing the library? thx!

[kmarple1]

After some reading on MDN, it seems like the callback itself is deprecated, so there's probably no point in adding it now. The expected behavior would be for everything to be Promise-based, but that would likely be a major revision.

[clarkbw]

We handle promises but we probably don't do it correctly

jest-webextension-mock/src/runtime.js

Line 23 in f66b8b8

return Promise.resolve();