Bug 1438531 - Remove leftovers of mocha test from netmonitor. r=Honza draft
authorAlexandre Poirot <poirot.alex@gmail.com>
Mon, 19 Feb 2018 06:58:51 -0800
changeset 756994 ab61f0d2c044aad985491f854c5de6347b6a7765
parent 756852 ad133cd410a719c0b67e61b8d3b1c77a32fd80a9
push id99627
push userbmo:poirot.alex@gmail.com
push dateMon, 19 Feb 2018 16:23:27 +0000
reviewersHonza
bugs1438531
milestone60.0a1
Bug 1438531 - Remove leftovers of mocha test from netmonitor. r=Honza MozReview-Commit-ID: qea532Kc7Q
devtools/client/netmonitor/test/components/filter-buttons.test.js
devtools/client/netmonitor/test/fixtures/create-store.js
devtools/client/netmonitor/test/fixtures/l10n.js
devtools/client/netmonitor/test/fixtures/localization-helper.js
devtools/client/netmonitor/test/require-helper.js
deleted file mode 100644
--- a/devtools/client/netmonitor/test/components/filter-buttons.test.js
+++ /dev/null
@@ -1,102 +0,0 @@
-/* Any copyright is dedicated to the Public Domain.
-   http://creativecommons.org/publicdomain/zero/1.0/ */
-
-/* eslint-env node, mocha */
-
-"use strict";
-
-const expect = require("expect");
-const { mount } = require("enzyme");
-const { createFactory } = require("devtools/client/shared/vendor/react");
-const { configureStore } = require("devtools/client/netmonitor/store");
-const Provider = createFactory(require("devtools/client/shared/vendor/react-redux").Provider);
-const Actions = require("devtools/client/netmonitor/actions/index");
-const FilterButtons = createFactory(require("devtools/client/netmonitor/components/filter-buttons"));
-
-const expectDefaultTypes = {
-  all: true,
-  html: false,
-  css: false,
-  js: false,
-  xhr: false,
-  fonts: false,
-  images: false,
-  media: false,
-  flash: false,
-  ws: false,
-  other: false,
-};
-
-// unit test
-describe("FilterButtons component:", () => {
-  const store = configureStore();
-  const wrapper = mount(FilterButtons({ store }));
-
-  asExpected(wrapper, expectDefaultTypes, "by default");
-});
-
-// integration test with redux store, action, reducer
-describe("FilterButtons::enableFilterOnly:", () => {
-  const expectXHRTypes = {
-    all: false,
-    html: false,
-    css: false,
-    js: false,
-    xhr: true,
-    fonts: false,
-    images: false,
-    media: false,
-    flash: false,
-    ws: false,
-    other: false,
-  };
-
-  const store = configureStore();
-  const wrapper = mount(Provider(
-    { store },
-    FilterButtons()
-  ));
-
-  store.dispatch(Actions.enableRequestFilterTypeOnly("xhr"));
-  asExpected(wrapper, expectXHRTypes, `when enableFilterOnly("xhr") is called`);
-});
-
-// integration test with redux store, action, reducer
-describe("FilterButtons::toggleFilter:", () => {
-  const expectXHRJSTypes = {
-    all: false,
-    html: false,
-    css: false,
-    js: true,
-    xhr: true,
-    fonts: false,
-    images: false,
-    media: false,
-    flash: false,
-    ws: false,
-    other: false,
-  };
-
-  const store = configureStore();
-  const wrapper = mount(Provider(
-    { store },
-    FilterButtons()
-  ));
-
-  store.dispatch(Actions.toggleRequestFilterType("xhr"));
-  store.dispatch(Actions.toggleRequestFilterType("js"));
-  asExpected(wrapper, expectXHRJSTypes, "when xhr, js is toggled");
-});
-
-function asExpected(wrapper, expectTypes, description) {
-  for (let type of Object.keys(expectTypes)) {
-    let checked = expectTypes[type] ? "checked" : "not checked";
-    let className = expectTypes[type] ?
-        "devtools-button checked" : "devtools-button";
-    it(`'${type}' button is ${checked} ${description}`, () => {
-      expect(wrapper.find(`.requests-list-filter-${type}-button`).html())
-      .toBe(`<button class="` + className +
-            `" data-key="${type}">netmonitor.toolbar.filter.${type}</button>`);
-    });
-  }
-}
deleted file mode 100644
--- a/devtools/client/netmonitor/test/fixtures/create-store.js
+++ /dev/null
@@ -1,9 +0,0 @@
-/* Any copyright is dedicated to the Public Domain.
-   http://creativecommons.org/publicdomain/zero/1.0/ */
-
-"use strict";
-
-const { createStore } = require("devtools/client/shared/vendor/redux");
-
-// Current mockup does not support any middleware
-module.exports = () => createStore;
deleted file mode 100644
--- a/devtools/client/netmonitor/test/fixtures/l10n.js
+++ /dev/null
@@ -1,20 +0,0 @@
-/* Any copyright is dedicated to the Public Domain.
-   http://creativecommons.org/publicdomain/zero/1.0/ */
-
-"use strict";
-
-// @TODO Load the actual strings from netmonitor.properties instead.
-class L10n {
-  getStr(str) {
-    switch (str) {
-      default:
-        return str;
-    }
-  }
-
-  getFormatStr(str) {
-    return this.getStr(str);
-  }
-}
-
-module.exports = L10n;
deleted file mode 100644
--- a/devtools/client/netmonitor/test/fixtures/localization-helper.js
+++ /dev/null
@@ -1,10 +0,0 @@
-/* Any copyright is dedicated to the Public Domain.
-   http://creativecommons.org/publicdomain/zero/1.0/ */
-
-"use strict";
-
-const LocalizationHelper = require("devtools/client/netmonitor/test/fixtures/l10n");
-
-module.exports = {
-  LocalizationHelper
-};
deleted file mode 100644
--- a/devtools/client/netmonitor/test/require-helper.js
+++ /dev/null
@@ -1,36 +0,0 @@
-/* Any copyright is dedicated to the Public Domain.
-   http://creativecommons.org/publicdomain/zero/1.0/ */
-
-"use strict";
-
-const requireHacker = require("require-hacker");
-
-requireHacker.global_hook("default", path => {
-  switch (path) {
-    // For Enzyme
-    case "react-dom":
-      return `const React = require('devtools/client/shared/vendor/react-dev'); module.exports = React`;
-    case "react-dom/server":
-      return `const React = require('devtools/client/shared/vendor/react-dev'); module.exports = React`;
-    case "react-addons-test-utils":
-      return `const TestUtils = require('devtools/client/shared/vendor/react-dom-test-utils'); module.exports = TestUtils`;
-    // Use react-dev. This would be handled by browserLoader in Firefox.
-    case "react":
-    case "devtools/client/shared/vendor/react":
-      return `const React = require('devtools/client/shared/vendor/react-dev'); module.exports = React`;
-    // For Rep's use of AMD
-    case "devtools/client/shared/vendor/react.default":
-      return `const React = require('devtools/client/shared/vendor/react-dev'); module.exports = React`;
-  }
-
-  // Some modules depend on Chrome APIs which don't work in mocha. When such a module
-  // is required, replace it with a mock version.
-  switch (path) {
-    case "devtools/shared/l10n":
-      return `module.exports = require("devtools/client/netmonitor/test/fixtures/localization-helper")`;
-    case "devtools/client/shared/redux/create-store":
-      return `module.exports = require("devtools/client/netmonitor/test/fixtures/create-store")`;
-  }
-
-  return null;
-});