Bug 1459962 - Part 2: rename WorkerList* components to RegistrationList* r=jdescottes
authorBelén Albeza <balbeza@mozilla.com>
Fri, 24 Apr 2020 15:49:01 +0000
changeset 525959 9f1bbab4d4a5e45c8792a10874da35aee294b7bd
parent 525958 7dc5909509d06c69f93861bd19ec89276287fa76
child 525960 84937d890e28541877f554f1ed05f5470d3b704d
push id37348
push userrmaries@mozilla.com
push dateSat, 25 Apr 2020 09:49:23 +0000
treeherdermozilla-central@d8a8178627c4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjdescottes
bugs1459962
milestone77.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1459962 - Part 2: rename WorkerList* components to RegistrationList* r=jdescottes As a result of the first part review and some testing, there were some issues re: repeating things among workers that actually belonged to the registration (the ID's for React keys, `Unregister` button, `Last updated` timeā€¦). Initially I wanted to do the "grouping" of all the workers belonging to the same registration, but I think this is less confusing to use and test. This patch initiates that work by renaming our `WorkerList` and `WorkerListEmpty` components to `Registration*`. Differential Revision: https://phabricator.services.mozilla.com/D72172
devtools/client/application/application.css
devtools/client/application/src/components/service-workers/RegistrationList.css
devtools/client/application/src/components/service-workers/RegistrationList.js
devtools/client/application/src/components/service-workers/RegistrationListEmpty.css
devtools/client/application/src/components/service-workers/RegistrationListEmpty.js
devtools/client/application/src/components/service-workers/WorkerList.css
devtools/client/application/src/components/service-workers/WorkerList.js
devtools/client/application/src/components/service-workers/WorkerListEmpty.css
devtools/client/application/src/components/service-workers/WorkerListEmpty.js
devtools/client/application/src/components/service-workers/WorkersPage.js
devtools/client/application/src/components/service-workers/moz.build
devtools/client/application/test/browser/browser_application_panel_list-domain-workers.js
devtools/client/application/test/browser/browser_application_panel_list-single-worker.js
devtools/client/application/test/browser/browser_application_panel_list-workers-empty.js
devtools/client/application/test/browser/browser_application_panel_target-switching.js
devtools/client/application/test/browser/browser_application_panel_worker-states.js
devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-RegistrationList.test.js.snap
devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-RegistrationListEmpty.test.js.snap
devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-WorkerList.test.js.snap
devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-WorkerListEmpty.test.js.snap
devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-WorkersPage.test.js.snap
devtools/client/application/test/node/components/service-workers/components_application_panel-RegistrationList.test.js
devtools/client/application/test/node/components/service-workers/components_application_panel-RegistrationListEmpty.test.js
devtools/client/application/test/node/components/service-workers/components_application_panel-WorkerList.test.js
devtools/client/application/test/node/components/service-workers/components_application_panel-WorkerListEmpty.test.js
--- a/devtools/client/application/application.css
+++ b/devtools/client/application/application.css
@@ -17,19 +17,19 @@
 @import "resource://devtools/client/application/src/components/manifest/ManifestIssueList.css";
 @import "resource://devtools/client/application/src/components/manifest/ManifestItem.css";
 @import "resource://devtools/client/application/src/components/manifest/ManifestJsonLink.css";
 @import "resource://devtools/client/application/src/components/manifest/ManifestLoader.css";
 @import "resource://devtools/client/application/src/components/manifest/ManifestSection.css";
 @import "resource://devtools/client/application/src/components/routing/PageSwitcher.css";
 @import "resource://devtools/client/application/src/components/routing/Sidebar.css";
 @import "resource://devtools/client/application/src/components/routing/SidebarItem.css";
+@import "resource://devtools/client/application/src/components/service-workers/RegistrationList.css";
+@import "resource://devtools/client/application/src/components/service-workers/RegistrationListEmpty.css";
 @import "resource://devtools/client/application/src/components/service-workers/Worker.css";
-@import "resource://devtools/client/application/src/components/service-workers/WorkerList.css";
-@import "resource://devtools/client/application/src/components/service-workers/WorkerListEmpty.css";
 @import "resource://devtools/client/application/src/components/service-workers/WorkersPage.css";
 @import "resource://devtools/client/application/src/components/ui/UIButton.css";
 
 html,
 body,
 #mount {
   height: 100vh;
 }
rename from devtools/client/application/src/components/service-workers/WorkerList.css
rename to devtools/client/application/src/components/service-workers/RegistrationList.css
--- a/devtools/client/application/src/components/service-workers/WorkerList.css
+++ b/devtools/client/application/src/components/service-workers/RegistrationList.css
@@ -6,11 +6,11 @@
   text-align: right;
   padding: 1rem 0;
 }
 
 .aboutdebugging-plug__link {
   margin: 0;
 }
 
-.workers-container {
+.registrations-container {
   flex-grow: 1;
 }
rename from devtools/client/application/src/components/service-workers/WorkerList.js
rename to devtools/client/application/src/components/service-workers/RegistrationList.js
--- a/devtools/client/application/src/components/service-workers/WorkerList.js
+++ b/devtools/client/application/src/components/service-workers/RegistrationList.js
@@ -26,30 +26,33 @@ const Worker = createFactory(
   require("devtools/client/application/src/components/service-workers/Worker")
 );
 
 /**
  * This component handles the list of service workers displayed in the application panel
  * and also displays a suggestion to use about debugging for debugging other service
  * workers.
  */
-class WorkerList extends PureComponent {
+class RegistrationList extends PureComponent {
   static get propTypes() {
     return {
       canDebugWorkers: PropTypes.bool.isRequired,
       workers: Types.workerArray.isRequired,
     };
   }
 
   render() {
     const { canDebugWorkers, workers } = this.props;
 
     return [
       article(
-        { className: "workers-container", key: "workers-container" },
+        {
+          className: "registrations-container",
+          key: "registrations-container",
+        },
         Localized(
           { id: "serviceworker-list-header" },
           h1({
             className: "app-page__title",
           })
         ),
         ul(
           {},
@@ -73,9 +76,9 @@ class WorkerList extends PureComponent {
         },
         footer({ className: "aboutdebugging-plug" })
       ),
     ];
   }
 }
 
 // Exports
-module.exports = WorkerList;
+module.exports = RegistrationList;
rename from devtools/client/application/src/components/service-workers/WorkerListEmpty.css
rename to devtools/client/application/src/components/service-workers/RegistrationListEmpty.css
--- a/devtools/client/application/src/components/service-workers/WorkerListEmpty.css
+++ b/devtools/client/application/src/components/service-workers/RegistrationListEmpty.css
@@ -1,16 +1,16 @@
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 .app-page__title .external-link {
   font-weight: var(--title-30-font-weight);
 }
 
-.worker-list-empty__tips {
+.registration-list-empty__tips {
   margin-inline-start: 2rem;
 }
 
-.worker-list-empty__tips__item {
+.registration-list-empty__tips-item {
   margin-block-start: 0.5rem;
   margin-block-end: 0.5rem;
 }
rename from devtools/client/application/src/components/service-workers/WorkerListEmpty.js
rename to devtools/client/application/src/components/service-workers/RegistrationListEmpty.js
--- a/devtools/client/application/src/components/service-workers/WorkerListEmpty.js
+++ b/devtools/client/application/src/components/service-workers/RegistrationListEmpty.js
@@ -28,17 +28,17 @@ const {
 const DOC_URL =
   "https://developer.mozilla.org/docs/Web/API/Service_Worker_API/Using_Service_Workers" +
   "?utm_source=devtools&utm_medium=sw-panel-blank";
 
 /**
  * This component displays help information when no service workers are found for the
  * current target.
  */
-class WorkerListEmpty extends PureComponent {
+class RegistrationListEmpty extends PureComponent {
   switchToConsole() {
     services.selectTool("webconsole");
   }
 
   switchToDebugger() {
     services.selectTool("jsdebugger");
   }
 
@@ -47,53 +47,53 @@ class WorkerListEmpty extends PureCompon
   }
 
   openDocumentation() {
     openDocLink(DOC_URL);
   }
 
   render() {
     return article(
-      { className: "worker-list-empty js-worker-list-empty" },
+      { className: "registration-list-empty js-registration-list-empty" },
       Localized(
         {
           id: "serviceworker-empty-intro",
           a: a({
             className: "external-link",
             onClick: () => this.openDocumentation(),
           }),
         },
         h1({ className: "app-page__title" })
       ),
       Localized({ id: "serviceworker-empty-suggestions" }, p({})),
       ul(
-        { className: "worker-list-empty__tips" },
+        { className: "registration-list-empty__tips" },
         Localized(
           {
             id: "serviceworker-empty-suggestions-console",
             a: a({ className: "link", onClick: () => this.switchToConsole() }),
           },
-          li({ className: "worker-list-empty__tips__item" })
+          li({ className: "registration-list-empty__tips-item" })
         ),
         Localized(
           {
             id: "serviceworker-empty-suggestions-debugger",
             a: a({ className: "link", onClick: () => this.switchToDebugger() }),
           },
-          li({ className: "worker-list-empty__tips__item" })
+          li({ className: "registration-list-empty__tips-item" })
         ),
         Localized(
           {
             id: "serviceworker-empty-suggestions-aboutdebugging",
             a: a({
               className: "link js-trusted-link",
               onClick: () => this.openAboutDebugging(),
             }),
           },
-          li({ className: "worker-list-empty__tips__item" })
+          li({ className: "registration-list-empty__tips-item" })
         )
       )
     );
   }
 }
 
 // Exports
-module.exports = WorkerListEmpty;
+module.exports = RegistrationListEmpty;
--- a/devtools/client/application/src/components/service-workers/WorkersPage.js
+++ b/devtools/client/application/src/components/service-workers/WorkersPage.js
@@ -10,21 +10,21 @@ const {
 } = require("devtools/client/shared/vendor/react");
 const PropTypes = require("devtools/client/shared/vendor/react-prop-types");
 const {
   section,
 } = require("devtools/client/shared/vendor/react-dom-factories");
 const { connect } = require("devtools/client/shared/vendor/react-redux");
 
 const Types = require("devtools/client/application/src/types/index");
-const WorkerList = createFactory(
-  require("devtools/client/application/src/components/service-workers/WorkerList")
+const RegistrationList = createFactory(
+  require("devtools/client/application/src/components/service-workers/RegistrationList")
 );
-const WorkerListEmpty = createFactory(
-  require("devtools/client/application/src/components/service-workers/WorkerListEmpty")
+const RegistrationListEmpty = createFactory(
+  require("devtools/client/application/src/components/service-workers/RegistrationListEmpty")
 );
 
 class WorkersPage extends PureComponent {
   static get propTypes() {
     return {
       // mapped from state
       canDebugWorkers: PropTypes.bool.isRequired,
       domain: PropTypes.string.isRequired,
@@ -43,18 +43,18 @@ class WorkersPage extends PureComponent 
 
     return section(
       {
         className: `app-page js-service-workers-page ${
           isWorkerListEmpty ? "app-page--empty" : ""
         }`,
       },
       isWorkerListEmpty
-        ? WorkerListEmpty({})
-        : WorkerList({
+        ? RegistrationListEmpty({})
+        : RegistrationList({
             canDebugWorkers,
             workers: domainWorkers,
           })
     );
   }
 }
 
 function mapStateToProps(state) {
--- a/devtools/client/application/src/components/service-workers/moz.build
+++ b/devtools/client/application/src/components/service-workers/moz.build
@@ -1,14 +1,14 @@
 # This Source Code Form is subject to the terms of the Mozilla Public
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 DevToolsModules(
+    'RegistrationList.css',
+    'RegistrationList.js',
+    'RegistrationListEmpty.css',
+    'RegistrationListEmpty.js',
     'Worker.css',
     'Worker.js',
-    'WorkerList.css',
-    'WorkerList.js',
-    'WorkerListEmpty.css',
-    'WorkerListEmpty.js',
     'WorkersPage.css',
     'WorkersPage.js',
 )
--- a/devtools/client/application/test/browser/browser_application_panel_list-domain-workers.js
+++ b/devtools/client/application/test/browser/browser_application_panel_list-domain-workers.js
@@ -35,17 +35,19 @@ add_task(async function() {
   );
 
   info(
     "Navigate to another page for a different domain with no service worker"
   );
 
   await navigateTo(EMPTY_URL);
   info("Wait until the service worker list is updated");
-  await waitUntil(() => doc.querySelector(".worker-list-empty") !== null);
+  await waitUntil(
+    () => doc.querySelector(".js-registration-list-empty") !== null
+  );
   ok(
     true,
     "No service workers are shown for an empty page in a different domain."
   );
 
   info(
     "Navigate to another page for a different domain with another service worker"
   );
--- a/devtools/client/application/test/browser/browser_application_panel_list-single-worker.js
+++ b/devtools/client/application/test/browser/browser_application_panel_list-single-worker.js
@@ -10,17 +10,17 @@ add_task(async function() {
   await enableApplicationPanel();
 
   const { panel, tab } = await openNewTabAndApplicationPanel(TAB_URL);
   const doc = panel.panelWin.document;
 
   selectPage(panel, "service-workers");
 
   info("Check for non-existing service worker");
-  const isWorkerListEmpty = !!doc.querySelector(".worker-list-empty");
+  const isWorkerListEmpty = !!doc.querySelector(".js-registration-list-empty");
   ok(isWorkerListEmpty, "No Service Worker displayed");
 
   info("Register a service worker in the page.");
   await SpecialPowers.spawn(tab.linkedBrowser, [], async function() {
     content.wrappedJSObject.registerServiceWorker();
   });
 
   info("Wait until the service worker appears in the application panel");
--- a/devtools/client/application/test/browser/browser_application_panel_list-workers-empty.js
+++ b/devtools/client/application/test/browser/browser_application_panel_list-workers-empty.js
@@ -13,15 +13,17 @@ const EMPTY_URL = URL_ROOT + "resources/
 add_task(async function() {
   await enableApplicationPanel();
 
   const { panel, tab } = await openNewTabAndApplicationPanel(EMPTY_URL);
   const doc = panel.panelWin.document;
 
   selectPage(panel, "service-workers");
 
-  await waitUntil(() => doc.querySelector(".js-worker-list-empty") !== null);
+  await waitUntil(
+    () => doc.querySelector(".js-registration-list-empty") !== null
+  );
   ok(true, "No service workers are shown for an empty page");
 
   // close the tab
   info("Closing the tab.");
   await BrowserTestUtils.removeTab(tab);
 });
--- a/devtools/client/application/test/browser/browser_application_panel_target-switching.js
+++ b/devtools/client/application/test/browser/browser_application_panel_target-switching.js
@@ -21,17 +21,17 @@ add_task(async function() {
   info("Open a page that runs in the parent process");
   const { panel, toolbox, tab } = await openNewTabAndApplicationPanel(
     PARENT_PROCESS_URI
   );
   const doc = panel.panelWin.document;
 
   info("Check for non-existing service worker");
   selectPage(panel, "service-workers");
-  const isWorkerListEmpty = !!doc.querySelector(".worker-list-empty");
+  const isWorkerListEmpty = !!doc.querySelector(".js-registration-list-empty");
   ok(isWorkerListEmpty, "No Service Worker displayed");
 
   info("Navigate to a page that runs in the child process");
   await navigateTo(CONTENT_PROCESS_URI_WORKERS);
 
   info("Wait until the service worker appears in the application panel");
   await waitUntil(() => getWorkerContainers(doc).length === 1);
 
--- a/devtools/client/application/test/browser/browser_application_panel_worker-states.js
+++ b/devtools/client/application/test/browser/browser_application_panel_worker-states.js
@@ -9,17 +9,17 @@ add_task(async function() {
   await enableApplicationPanel();
 
   const { panel, tab } = await openNewTabAndApplicationPanel(TAB_URL);
   const doc = panel.panelWin.document;
 
   selectPage(panel, "service-workers");
 
   info("Check for non-existing service worker");
-  const isWorkerListEmpty = !!doc.querySelector(".worker-list-empty");
+  const isWorkerListEmpty = !!doc.querySelector(".js-registration-list-empty");
   ok(isWorkerListEmpty, "No Service Worker displayed");
 
   info("Register a service worker with a controlled install in the page.");
   await SpecialPowers.spawn(tab.linkedBrowser, [], async function() {
     content.wrappedJSObject.registerServiceWorker();
   });
 
   info("Wait until the service worker appears in the application panel");
rename from devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-WorkerList.test.js.snap
rename to devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-RegistrationList.test.js.snap
--- a/devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-WorkerList.test.js.snap
+++ b/devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-RegistrationList.test.js.snap
@@ -1,15 +1,15 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`WorkerList renders the expected snapshot for a list with a single worker 1`] = `
+exports[`RegistrationList renders the expected snapshot for a list with a single registration 1`] = `
 Array [
   <article
-    className="workers-container"
-    key="workers-container"
+    className="registrations-container"
+    key="registrations-container"
   >
     <Localized
       id="serviceworker-list-header"
     >
       <h1
         className="app-page__title"
       />
     </Localized>
@@ -44,21 +44,21 @@ Array [
   >
     <footer
       className="aboutdebugging-plug"
     />
   </Localized>,
 ]
 `;
 
-exports[`WorkerList renders the expected snapshot for a multiple workers list 1`] = `
+exports[`RegistrationList renders the expected snapshot for a multiple registration list 1`] = `
 Array [
   <article
-    className="workers-container"
-    key="workers-container"
+    className="registrations-container"
+    key="registrations-container"
   >
     <Localized
       id="serviceworker-list-header"
     >
       <h1
         className="app-page__title"
       />
     </Localized>
rename from devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-WorkerListEmpty.test.js.snap
rename to devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-RegistrationListEmpty.test.js.snap
--- a/devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-WorkerListEmpty.test.js.snap
+++ b/devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-RegistrationListEmpty.test.js.snap
@@ -1,13 +1,13 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`WorkerListEmpty renders the expected snapshot 1`] = `
+exports[`RegistrationListEmpty renders the expected snapshot 1`] = `
 <article
-  className="worker-list-empty js-worker-list-empty"
+  className="registration-list-empty js-registration-list-empty"
 >
   <Localized
     a={
       <a
         className="external-link"
         onClick={[Function]}
       />
     }
@@ -18,52 +18,52 @@ exports[`WorkerListEmpty renders the exp
     />
   </Localized>
   <Localized
     id="serviceworker-empty-suggestions"
   >
     <p />
   </Localized>
   <ul
-    className="worker-list-empty__tips"
+    className="registration-list-empty__tips"
   >
     <Localized
       a={
         <a
           className="link"
           onClick={[Function]}
         />
       }
       id="serviceworker-empty-suggestions-console"
     >
       <li
-        className="worker-list-empty__tips__item"
+        className="registration-list-empty__tips-item"
       />
     </Localized>
     <Localized
       a={
         <a
           className="link"
           onClick={[Function]}
         />
       }
       id="serviceworker-empty-suggestions-debugger"
     >
       <li
-        className="worker-list-empty__tips__item"
+        className="registration-list-empty__tips-item"
       />
     </Localized>
     <Localized
       a={
         <a
           className="link js-trusted-link"
           onClick={[Function]}
         />
       }
       id="serviceworker-empty-suggestions-aboutdebugging"
     >
       <li
-        className="worker-list-empty__tips__item"
+        className="registration-list-empty__tips-item"
       />
     </Localized>
   </ul>
 </article>
 `;
--- a/devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-WorkersPage.test.js.snap
+++ b/devtools/client/application/test/node/components/service-workers/__snapshots__/components_application_panel-WorkersPage.test.js.snap
@@ -1,15 +1,15 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`WorkersPage filters out workers from diferent domains 1`] = `
 <section
   className="app-page js-service-workers-page "
 >
-  <WorkerList
+  <RegistrationList
     canDebugWorkers={true}
     workers={
       Array [
         Object {
           "active": true,
           "id": "id-worker-1-example",
           "name": "worker1",
           "registrationFront": "",
@@ -33,25 +33,25 @@ exports[`WorkersPage filters out workers
   />
 </section>
 `;
 
 exports[`WorkersPage filters out workers from different domains and renders an empty list when there is none left 1`] = `
 <section
   className="app-page js-service-workers-page app-page--empty"
 >
-  <WorkerListEmpty />
+  <RegistrationListEmpty />
 </section>
 `;
 
 exports[`WorkersPage it renders a list with a single element if there's just 1 worker 1`] = `
 <section
   className="app-page js-service-workers-page "
 >
-  <WorkerList
+  <RegistrationList
     canDebugWorkers={true}
     workers={
       Array [
         Object {
           "active": true,
           "id": "id-worker-1-example",
           "name": "worker1",
           "registrationFront": "",
@@ -65,17 +65,17 @@ exports[`WorkersPage it renders a list w
   />
 </section>
 `;
 
 exports[`WorkersPage renders a list with multiple elements when there are multiple workers 1`] = `
 <section
   className="app-page js-service-workers-page "
 >
-  <WorkerList
+  <RegistrationList
     canDebugWorkers={true}
     workers={
       Array [
         Object {
           "active": true,
           "id": "id-worker-1-example",
           "name": "worker1",
           "registrationFront": "",
@@ -109,11 +109,11 @@ exports[`WorkersPage renders a list with
   />
 </section>
 `;
 
 exports[`WorkersPage renders an empty list if there are no workers 1`] = `
 <section
   className="app-page js-service-workers-page app-page--empty"
 >
-  <WorkerListEmpty />
+  <RegistrationListEmpty />
 </section>
 `;
rename from devtools/client/application/test/node/components/service-workers/components_application_panel-WorkerList.test.js
rename to devtools/client/application/test/node/components/service-workers/components_application_panel-RegistrationList.test.js
--- a/devtools/client/application/test/node/components/service-workers/components_application_panel-WorkerList.test.js
+++ b/devtools/client/application/test/node/components/service-workers/components_application_panel-RegistrationList.test.js
@@ -8,36 +8,36 @@ const { shallow } = require("enzyme");
 const { createFactory } = require("react");
 
 // Import constants
 const {
   SINGLE_WORKER_DEFAULT_DOMAIN_LIST,
   MULTIPLE_WORKER_LIST,
 } = require("devtools/client/application/test/node/fixtures/data/constants");
 
-const WorkerList = createFactory(
-  require("devtools/client/application/src/components/service-workers/WorkerList")
+const RegistrationList = createFactory(
+  require("devtools/client/application/src/components/service-workers/RegistrationList")
 );
 
 /**
- * Test for workerList.js component
+ * Test for RegistrationList.js component
  */
-describe("WorkerList", () => {
-  it("renders the expected snapshot for a list with a single worker", () => {
+describe("RegistrationList", () => {
+  it("renders the expected snapshot for a list with a single registration", () => {
     const wrapper = shallow(
-      WorkerList({
+      RegistrationList({
         workers: SINGLE_WORKER_DEFAULT_DOMAIN_LIST,
         canDebugWorkers: true,
       })
     );
     expect(wrapper).toMatchSnapshot();
   });
 
-  it("renders the expected snapshot for a multiple workers list", () => {
+  it("renders the expected snapshot for a multiple registration list", () => {
     const wrapper = shallow(
-      WorkerList({
+      RegistrationList({
         workers: MULTIPLE_WORKER_LIST,
         canDebugWorkers: true,
       })
     );
     expect(wrapper).toMatchSnapshot();
   });
 });
rename from devtools/client/application/test/node/components/service-workers/components_application_panel-WorkerListEmpty.test.js
rename to devtools/client/application/test/node/components/service-workers/components_application_panel-RegistrationListEmpty.test.js
--- a/devtools/client/application/test/node/components/service-workers/components_application_panel-WorkerListEmpty.test.js
+++ b/devtools/client/application/test/node/components/service-workers/components_application_panel-RegistrationListEmpty.test.js
@@ -2,22 +2,22 @@
    http://creativecommons.org/publicdomain/zero/1.0/ */
 
 "use strict";
 
 // Import libs
 const { shallow } = require("enzyme");
 const { createFactory } = require("react");
 
-const WorkerListEmpty = createFactory(
-  require("devtools/client/application/src/components/service-workers/WorkerListEmpty")
+const RegistrationListEmpty = createFactory(
+  require("devtools/client/application/src/components/service-workers/RegistrationListEmpty")
 );
 
 /**
- * Test for workerListEmpty.js component
+ * Test for RegistrationListEmpty.js component
  */
 
-describe("WorkerListEmpty", () => {
+describe("RegistrationListEmpty", () => {
   it("renders the expected snapshot", () => {
-    const wrapper = shallow(WorkerListEmpty({}));
+    const wrapper = shallow(RegistrationListEmpty({}));
     expect(wrapper).toMatchSnapshot();
   });
 });