1550791 - WebConsole mocha tests are failing with `TypeError: dispatch is not a function`

Status: RESOLVED FIXED

(DevTools :: Console, defect, P1)

Description

[Julian Descottes [:jdescottes]]

Regression from Bug 1230194, precisely part 3 https://hg.mozilla.org/mozilla-central/rev/26b7736f1f87

I will disable the necessary tests in order to be able to move back the devtools node test suite as tier 1 in Bug 1535484

Comment 1

[Julian Descottes [:jdescottes]]

Hi Nicolas, I am skipping the following mocha test:

  it("displays a [Learn more] link", () => {
    const store = setupStore();

    const message = stubPreparedMessages.get("asdf()");

    serviceContainer.openLink = sinon.spy();
    const wrapper = mount(
      Provider({ store }, EvaluationResult({ message, serviceContainer }))
    );

    const url =
      "https://developer.mozilla.org/docs/Web/JavaScript/Reference/Errors/Not_defined";
    const learnMore = wrapper.find(".learn-more-link");
    expect(learnMore.length).toBe(1);
    expect(learnMore.prop("title")).toBe(url);

    learnMore.simulate("click");
    const call = serviceContainer.openLink.getCall(0);
    expect(call.args[0]).toEqual(message.exceptionDocURL);
  });

from https://searchfox.org/mozilla-central/rev/b418634cb3fe83ebb8d2c019cc1ba76974da1a0d/devtools/client/webconsole/test/components/evaluation-result.test.js#43

The failure log is

  1) EvaluationResult component:
       displays a [Learn more] link:
     TypeError: dispatch is not a function
      at Message.toggleMessage (/Users/jdescottes/Development/hg/fx-team/devtools/client/webconsole/components/Message.js:117:7)
      at Object.invokeGuardedCallback (/Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom.js:77:10)
      at Object.invokeGuardedCallback (/Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom.js:107:27)
      at Object.invokeGuardedCallbackAndCatchFirstError (/Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom.js:121:43)
      at executeDispatch (/Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom.js:399:19)
      at executeDispatchesInOrder (/Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom.js:415:7)
      at executeDispatchesAndRelease (/Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom.js:516:5)
      at executeDispatchesAndReleaseSimulated (/Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom.js:524:10)
      at forEachAccumulated (/Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom.js:497:8)
      at Object.runEventsInBatch (/Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom.js:656:5)
      at /Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom-test-utils-dev.js:1184:22
      at Object.batchedUpdates$1 [as unstable_batchedUpdates] (/Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom.js:13458:12)
      at /Users/jdescottes/Development/hg/fx-team/devtools/client/shared/vendor/react-dom-test-utils-dev.js:1180:14
      at Object.simulateEvent (node_modules/enzyme-adapter-react-16/build/ReactSixteenAdapter.js:492:15)
      at ReactWrapper.<anonymous> (node_modules/enzyme/build/ReactWrapper.js:953:29)
      at ReactWrapper.single (node_modules/enzyme/build/ReactWrapper.js:1720:25)
      at ReactWrapper.simulate (node_modules/enzyme/build/ReactWrapper.js:952:21)
      at Context.it (components/evaluation-result.test.js:116:15)

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

Thanks Julian, I'm going to fix this

Comment 3

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1550791 - Re-enable commented mocha test. r=jdescottes.

The test was disabled in Bug 1535484 because it was failing.
To fix the test, we need to pass a dispatch prop to the
EvaluationResult component.

Comment 4

[Nicolas Chevobbe [:nchevobbe]]

TRY build: https://treeherder.mozilla.org/#/jobs?repo=try&revision=16f31cf5d7e52c40eaaa9865c2c3e41c39718de3

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/83c6540f1fc1
Re-enable commented mocha test. r=jdescottes.

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/83c6540f1fc1