creevey 0.9.0-beta.13 → 0.9.0-beta.14

package/.yarnrc.yml

@@ -0,0 +1 @@
+nodeLinker: node-modules

package/lib/cjs/cli.js

@@ -1,4 +1,5 @@
 #!/usr/bin/env node
+
 // eslint-disable-next-line node/shebang
 "use strict";
 

package/lib/cjs/client/addon/Manager.js

@@ -4,244 +4,173 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.CreeveyManager = void 0;
-
 var _coreEvents = require("@storybook/core-events");
-
 var _shared = require("../../shared");
-
 var _types = require("../../types");
-
 var _creeveyClientApi = require("../shared/creeveyClientApi");
-
 var _helpers = require("../shared/helpers");
-
 var _register = require("./register");
-
 var _utils = require("./utils");
-
-function _defineProperty(obj, key, value) { if (key in obj) { Object.defineProperty(obj, key, { value: value, enumerable: true, configurable: true, writable: true }); } else { obj[key] = value; } return obj; }
-
 class CreeveyManager {
+  storyId = '';
+  activeBrowser = '';
+  selectedTestId = '';
+  status = {
+    isRunning: false,
+    tests: {},
+    browsers: []
+  };
+  creeveyApi = null;
+  stories = {};
+  updateStatusListeners = [];
+  changeTestListeners = [];
   constructor(storybookApi) {
     this.storybookApi = storybookApi;
-
-    _defineProperty(this, "storyId", '');
-
-    _defineProperty(this, "activeBrowser", '');
-
-    _defineProperty(this, "selectedTestId", '');
-
-    _defineProperty(this, "status", {
-      isRunning: false,
-      tests: {},
-      browsers: []
-    });
-
-    _defineProperty(this, "creeveyApi", null);
-
-    _defineProperty(this, "stories", {});
-
-    _defineProperty(this, "updateStatusListeners", []);
-
-    _defineProperty(this, "changeTestListeners", []);
-
-    _defineProperty(this, "initAll", async () => {
-      this.storybookApi.on(_coreEvents.STORY_RENDERED, this.onStoryRendered);
-      this.storybookApi.on(_coreEvents.SET_STORIES, this.onSetStories);
-      this.creeveyApi = await (0, _creeveyClientApi.initCreeveyClientApi)();
-      this.creeveyApi.onUpdate(this.handleCreeveyUpdate);
-      this.status = await this.creeveyApi.status;
-    });
-
-    _defineProperty(this, "handleCreeveyUpdate", update => {
-      const {
-        tests,
-        removedTests = [],
-        isRunning
-      } = update;
-
-      if ((0, _types.isDefined)(isRunning)) {
-        this.status.isRunning = isRunning;
-      }
-
-      if ((0, _types.isDefined)(tests)) {
-        const prevTests = this.status.tests;
-        const prevStories = this.stories || {};
-        Object.values(tests).filter(_types.isDefined).forEach(update => {
-          const {
-            id,
-            skip,
-            status,
-            results,
-            approved,
-            storyId
-          } = update;
-          const test = prevTests[id];
-
-          if (!test) {
-            return prevTests[id] = update;
-          }
-
-          if ((0, _types.isDefined)(skip)) test.skip = skip;
-
-          if ((0, _types.isDefined)(status)) {
-            test.status = status;
-
-            if ((0, _types.isDefined)(storyId) && (0, _types.isDefined)(prevStories[storyId])) {
-              const story = prevStories[storyId];
-              const storyStatus = this.getStoryTests(storyId);
-              const oldStatus = storyStatus.map(x => x.id === id ? status : x.status).reduce((oldStatus, newStatus) => (0, _helpers.calcStatus)(oldStatus, newStatus), undefined);
-              story.name = this.addStatusToStoryName(story.name, (0, _helpers.calcStatus)(oldStatus, status), skip || false);
-            }
-          }
-
-          if ((0, _types.isDefined)(results)) test.results ? test.results.push(...results) : test.results = results;
-
-          if ((0, _types.isDefined)(approved)) {
-            Object.entries(approved).forEach(_ref => {
-              let [image, retry] = _ref;
-              return retry !== undefined && test && ((test.approved = (test === null || test === void 0 ? void 0 : test.approved) || {})[image] = retry);
-            });
-          }
-        });
-        removedTests.forEach(_ref2 => {
-          let {
-            id
-          } = _ref2;
-          return delete prevTests[id];
-        });
-        this.status.tests = prevTests;
-        this.stories = prevStories;
-        this.setPanelsTitle();
-        void this.storybookApi.setStories(this.stories);
-      }
-
-      this.updateStatusListeners.forEach(x => x(update));
-    });
-
-    _defineProperty(this, "getCurrentTest", () => {
-      return this.status.tests[this.selectedTestId];
-    });
-
-    _defineProperty(this, "onStoryRendered", storyId => {
-      if (this.storyId === '') void this.addStatusesToSideBar();
-
-      if (this.storyId !== storyId) {
-        var _this$getTestsByStory, _this$getTestsByStory2;
-
-        this.storyId = storyId;
-        this.selectedTestId = (_this$getTestsByStory = (_this$getTestsByStory2 = this.getTestsByStoryIdAndBrowser(this.activeBrowser)[0]) === null || _this$getTestsByStory2 === void 0 ? void 0 : _this$getTestsByStory2.id) !== null && _this$getTestsByStory !== void 0 ? _this$getTestsByStory : '';
-        this.setPanelsTitle();
-        this.changeTestListeners.forEach(x => x(this.selectedTestId));
-      }
-    });
-
-    _defineProperty(this, "onStart", () => {
-      var _this$creeveyApi;
-
-      (_this$creeveyApi = this.creeveyApi) === null || _this$creeveyApi === void 0 ? void 0 : _this$creeveyApi.start([this.selectedTestId]);
-    });
-
-    _defineProperty(this, "onStop", () => {
-      var _this$creeveyApi2;
-
-      (_this$creeveyApi2 = this.creeveyApi) === null || _this$creeveyApi2 === void 0 ? void 0 : _this$creeveyApi2.stop();
-    });
-
-    _defineProperty(this, "onImageApprove", (id, retry, image) => {
-      var _this$creeveyApi3;
-
-      return (_this$creeveyApi3 = this.creeveyApi) === null || _this$creeveyApi3 === void 0 ? void 0 : _this$creeveyApi3.approve(id, retry, image);
-    });
-
-    _defineProperty(this, "onStartAllStoryTests", () => {
-      var _this$creeveyApi4;
-
-      const ids = Object.values(this.status.tests).filter(_types.isDefined).filter(x => x.storyId === this.storyId).map(x => x.id);
-      (_this$creeveyApi4 = this.creeveyApi) === null || _this$creeveyApi4 === void 0 ? void 0 : _this$creeveyApi4.start(ids);
-    });
-
-    _defineProperty(this, "onStartAllTests", () => {
-      var _this$creeveyApi5;
-
-      const ids = Object.values(this.status.tests).filter(_types.isDefined).map(x => x.id);
-      (_this$creeveyApi5 = this.creeveyApi) === null || _this$creeveyApi5 === void 0 ? void 0 : _this$creeveyApi5.start(ids);
-    });
-
-    _defineProperty(this, "onSetStories", data => {
-      const stories = data.v ? (0, _shared.denormalizeStoryParameters)(data) : data.stories;
-      this.stories = stories;
-    });
-
-    _defineProperty(this, "setActiveBrowser", browser => {
-      var _this$getTestsByStory3, _this$getTestsByStory4;
-
-      this.activeBrowser = browser;
-      this.selectedTestId = (_this$getTestsByStory3 = (_this$getTestsByStory4 = this.getTestsByStoryIdAndBrowser(this.activeBrowser)[0]) === null || _this$getTestsByStory4 === void 0 ? void 0 : _this$getTestsByStory4.id) !== null && _this$getTestsByStory3 !== void 0 ? _this$getTestsByStory3 : '';
-      this.changeTestListeners.forEach(x => x(this.selectedTestId));
-    });
-
-    _defineProperty(this, "setSelectedTestId", testId => {
-      this.selectedTestId = testId;
-      this.changeTestListeners.forEach(x => x(this.selectedTestId));
-    });
-
-    _defineProperty(this, "getStoryTests", storyId => {
-      if (!this.status || !this.status.tests) return [];
-      return Object.values(this.status.tests).filter(result => (result === null || result === void 0 ? void 0 : result.storyId) === storyId).filter(_types.isDefined);
-    });
-
-    _defineProperty(this, "getBrowsers", () => {
-      return this.status.browsers;
-    });
-
-    _defineProperty(this, "getTestsByStoryIdAndBrowser", browser => {
-      return Object.values(this.status.tests).filter(x => (x === null || x === void 0 ? void 0 : x.browser) === browser && x.storyId === this.storyId).filter(_types.isDefined);
-    });
-
-    _defineProperty(this, "getTabTitle", browser => {
-      const tests = this.getTestsByStoryIdAndBrowser(browser);
-      const browserStatus = tests.map(x => x && x.status).reduce((oldStatus, newStatus) => (0, _helpers.calcStatus)(oldStatus, newStatus), undefined);
-      const browserSkip = tests.length > 0 ? tests.every(x => x && x.skip) : false;
-      const emojiStatus = (0, _utils.getEmojiByTestStatus)(browserStatus, browserSkip);
-      return "".concat(emojiStatus ? "".concat(emojiStatus, " ") : '', "Creevey/").concat(browser);
-    });
-
-    _defineProperty(this, "setPanelsTitle", () => {
-      var _this$storybookApi;
-
-      const panels = (_this$storybookApi = this.storybookApi) === null || _this$storybookApi === void 0 ? void 0 : _this$storybookApi.getPanels();
-      if (!panels) return;
-      let firstPanelBrowser = this.activeBrowser;
-
-      for (const p in panels) {
-        var _panel$id;
-
-        const panel = panels[p];
-
-        if (((_panel$id = panel.id) === null || _panel$id === void 0 ? void 0 : _panel$id.indexOf(_register.ADDON_ID)) === 0 && panel.paramKey) {
-          panel.title = this.getTabTitle(panel.paramKey);
-          if (!firstPanelBrowser) firstPanelBrowser = panel.paramKey;
-        }
-      }
-
-      this.storybookApi.setSelectedPanel("".concat(_register.ADDON_ID, "/panel/").concat(firstPanelBrowser));
-    });
-
     this.storybookApi = storybookApi;
   }
-
+  initAll = async () => {
+    this.storybookApi.on(_coreEvents.STORY_RENDERED, this.onStoryRendered);
+    this.storybookApi.on(_coreEvents.SET_STORIES, this.onSetStories);
+    this.creeveyApi = await (0, _creeveyClientApi.initCreeveyClientApi)();
+    this.creeveyApi.onUpdate(this.handleCreeveyUpdate);
+    this.status = await this.creeveyApi.status;
+  };
   onUpdateStatus(listener) {
     this.updateStatusListeners.push(listener);
     return () => void (this.updateStatusListeners = this.updateStatusListeners.filter(x => x != listener));
   }
-
   onChangeTest(listener) {
     this.changeTestListeners.push(listener);
     return () => void (this.changeTestListeners = this.changeTestListeners.filter(x => x != listener));
   }
-
-  async addStatusesToSideBar() {
+  handleCreeveyUpdate = update => {
+    const {
+      tests,
+      removedTests = [],
+      isRunning
+    } = update;
+    if ((0, _types.isDefined)(isRunning)) {
+      this.status.isRunning = isRunning;
+    }
+    if ((0, _types.isDefined)(tests)) {
+      const prevTests = this.status.tests;
+      const prevStories = this.stories || {};
+      Object.values(tests).filter(_types.isDefined).forEach(update => {
+        const {
+          id,
+          skip,
+          status,
+          results,
+          approved,
+          storyId
+        } = update;
+        const test = prevTests[id];
+        if (!test) {
+          return prevTests[id] = update;
+        }
+        if ((0, _types.isDefined)(skip)) test.skip = skip;
+        if ((0, _types.isDefined)(status)) {
+          test.status = status;
+          if ((0, _types.isDefined)(storyId) && (0, _types.isDefined)(prevStories[storyId])) {
+            const story = prevStories[storyId];
+            const storyStatus = this.getStoryTests(storyId);
+            const oldStatus = storyStatus.map(x => x.id === id ? status : x.status).reduce((oldStatus, newStatus) => (0, _helpers.calcStatus)(oldStatus, newStatus), undefined);
+            story.name = this.addStatusToStoryName(story.name, (0, _helpers.calcStatus)(oldStatus, status), skip || false);
+          }
+        }
+        if ((0, _types.isDefined)(results)) test.results ? test.results.push(...results) : test.results = results;
+        if ((0, _types.isDefined)(approved)) {
+          Object.entries(approved).forEach(_ref => {
+            let [image, retry] = _ref;
+            return retry !== undefined && test && ((test.approved = test?.approved || {})[image] = retry);
+          });
+        }
+      });
+      removedTests.forEach(_ref2 => {
+        let {
+          id
+        } = _ref2;
+        return delete prevTests[id];
+      });
+      this.status.tests = prevTests;
+      this.stories = prevStories;
+      this.setPanelsTitle();
+      // TODO Check setStories method in 6.x and migrate properly
+      this.storybookApi.emit(_coreEvents.SET_STORIES, this.stories);
+    }
+    this.updateStatusListeners.forEach(x => x(update));
+  };
+  getCurrentTest = () => {
+    return this.status.tests[this.selectedTestId];
+  };
+  onStoryRendered = storyId => {
+    if (this.storyId === '') this.addStatusesToSideBar();
+    if (this.storyId !== storyId) {
+      this.storyId = storyId;
+      this.selectedTestId = this.getTestsByStoryIdAndBrowser(this.activeBrowser)[0]?.id ?? '';
+      this.setPanelsTitle();
+      this.changeTestListeners.forEach(x => x(this.selectedTestId));
+    }
+  };
+  onStart = () => {
+    this.creeveyApi?.start([this.selectedTestId]);
+  };
+  onStop = () => {
+    this.creeveyApi?.stop();
+  };
+  onImageApprove = (id, retry, image) => this.creeveyApi?.approve(id, retry, image);
+  onStartAllStoryTests = () => {
+    const ids = Object.values(this.status.tests).filter(_types.isDefined).filter(x => x.storyId === this.storyId).map(x => x.id);
+    this.creeveyApi?.start(ids);
+  };
+  onStartAllTests = () => {
+    const ids = Object.values(this.status.tests).filter(_types.isDefined).map(x => x.id);
+    this.creeveyApi?.start(ids);
+  };
+  onSetStories = data => {
+    const stories = data.v ? (0, _shared.denormalizeStoryParameters)(data) : data.stories;
+    this.stories = stories;
+  };
+  setActiveBrowser = browser => {
+    this.activeBrowser = browser;
+    this.selectedTestId = this.getTestsByStoryIdAndBrowser(this.activeBrowser)[0]?.id ?? '';
+    this.changeTestListeners.forEach(x => x(this.selectedTestId));
+  };
+  setSelectedTestId = testId => {
+    this.selectedTestId = testId;
+    this.changeTestListeners.forEach(x => x(this.selectedTestId));
+  };
+  getStoryTests = storyId => {
+    if (!this.status || !this.status.tests) return [];
+    return Object.values(this.status.tests).filter(result => result?.storyId === storyId).filter(_types.isDefined);
+  };
+  getBrowsers = () => {
+    return this.status.browsers;
+  };
+  getTestsByStoryIdAndBrowser = browser => {
+    return Object.values(this.status.tests).filter(x => x?.browser === browser && x.storyId === this.storyId).filter(_types.isDefined);
+  };
+  getTabTitle = browser => {
+    const tests = this.getTestsByStoryIdAndBrowser(browser);
+    const browserStatus = tests.map(x => x && x.status).reduce((oldStatus, newStatus) => (0, _helpers.calcStatus)(oldStatus, newStatus), undefined);
+    const browserSkip = tests.length > 0 ? tests.every(x => x && x.skip) : false;
+    const emojiStatus = (0, _utils.getEmojiByTestStatus)(browserStatus, browserSkip);
+    return `${emojiStatus ? `${emojiStatus} ` : ''}Creevey/${browser}`;
+  };
+  setPanelsTitle = () => {
+    const panels = this.storybookApi?.getPanels();
+    if (!panels) return;
+    let firstPanelBrowser = this.activeBrowser;
+    for (const p in panels) {
+      const panel = panels[p];
+      if (panel.id?.indexOf(_register.ADDON_ID) === 0 && panel.paramKey) {
+        panel.title = this.getTabTitle(panel.paramKey);
+        if (!firstPanelBrowser) firstPanelBrowser = panel.paramKey;
+      }
+    }
+    this.storybookApi.setSelectedPanel(`${_register.ADDON_ID}/panel/${firstPanelBrowser}`);
+  };
+  addStatusesToSideBar() {
     if (!Object.keys(this.stories).length) return;
     const stories = this.stories;
     Object.keys(this.stories).forEach(storyId => {
@@ -250,15 +179,14 @@ class CreeveyManager {
       const skip = storyStatus.length > 0 ? storyStatus.every(x => x.skip) : false;
       this.stories[storyId].name = this.addStatusToStoryName(stories[storyId].name, status, skip);
     });
-    await this.storybookApi.setStories(this.stories);
-  }
 
+    // TODO Check setStories method in 6.x and migrate properly
+    this.storybookApi.emit(_coreEvents.SET_STORIES, this.stories);
+  }
   addStatusToStoryName(name, status, skip) {
     name = name.replace(/^(❌|✔|🟡|🕗|⏸) /, '');
     const emojiStatus = (0, _utils.getEmojiByTestStatus)(status, skip);
-    return "".concat(emojiStatus ? "".concat(emojiStatus, " ") : '', " ").concat(name);
+    return `${emojiStatus ? `${emojiStatus} ` : ''} ${name}`;
   }
-
 }
-
 exports.CreeveyManager = CreeveyManager;

package/lib/cjs/client/addon/components/Addon.js

@@ -4,17 +4,11 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.Addon = void 0;
-
 var _components = require("@storybook/components");
-
 var _react = _interopRequireWildcard(require("react"));
-
 var _Panel = require("./Panel");
-
-function _getRequireWildcardCache(nodeInterop) { if (typeof WeakMap !== "function") return null; var cacheBabelInterop = new WeakMap(); var cacheNodeInterop = new WeakMap(); return (_getRequireWildcardCache = function (nodeInterop) { return nodeInterop ? cacheNodeInterop : cacheBabelInterop; })(nodeInterop); }
-
-function _interopRequireWildcard(obj, nodeInterop) { if (!nodeInterop && obj && obj.__esModule) { return obj; } if (obj === null || typeof obj !== "object" && typeof obj !== "function") { return { default: obj }; } var cache = _getRequireWildcardCache(nodeInterop); if (cache && cache.has(obj)) { return cache.get(obj); } var newObj = {}; var hasPropertyDescriptor = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var key in obj) { if (key !== "default" && Object.prototype.hasOwnProperty.call(obj, key)) { var desc = hasPropertyDescriptor ? Object.getOwnPropertyDescriptor(obj, key) : null; if (desc && (desc.get || desc.set)) { Object.defineProperty(newObj, key, desc); } else { newObj[key] = obj[key]; } } } newObj.default = obj; if (cache) { cache.set(obj, newObj); } return newObj; }
-
+function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function (e) { return e ? t : r; })(e); }
+function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && {}.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 const Addon = _ref => {
   let {
     active,
@@ -51,5 +45,4 @@ const Addon = _ref => {
     onImageApprove: manager.onImageApprove
   }) : /*#__PURE__*/_react.default.createElement(_components.Placeholder, null, "No test results") : null;
 };
-
 exports.Addon = Addon;

package/lib/cjs/client/addon/components/Icons.js

@@ -4,11 +4,8 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.NextIcon = exports.ForwardIcon = void 0;
-
 var _react = _interopRequireDefault(require("react"));
-
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 const NextIcon = _ref => {
   let {
     width,
@@ -24,9 +21,7 @@ const NextIcon = _ref => {
     fill: "currentColor"
   }));
 };
-
 exports.NextIcon = NextIcon;
-
 const ForwardIcon = _ref2 => {
   let {
     width,
@@ -42,5 +37,4 @@ const ForwardIcon = _ref2 => {
     d: "M460.9,246.8l-209-164.2c-7.8-6.1-19.2-0.6-19.2,9.2v328.4c0,9.8,11.4,15.3,19.2,9.2l209-164.2c5.1-4,6-11.4,2-16.5 C462.4,248,461.7,247.3,460.9,246.8L460.9,246.8z M228.2,246.8L19.2,82.5C11.4,76.4,0,82,0,91.8v328.4c0,9.8,11.4,15.3,19.2,9.2 l209-164.2c3-2.3,4.5-5.8,4.5-9.2C232.7,252.6,231.2,249.1,228.2,246.8z M507.3,64h-37.2c-2.5,0-4.7,2-4.7,4.4v375.3 c0,2.4,2.1,4.4,4.7,4.4h37.2c2.5,0,4.7-2,4.7-4.4V68.4C512,66,509.9,64,507.3,64z"
   }));
 };
-
 exports.ForwardIcon = ForwardIcon;

package/lib/cjs/client/addon/components/Panel.js

@@ -4,21 +4,13 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.Panel = void 0;
-
 var _react = _interopRequireDefault(require("react"));
-
 var _components = require("@storybook/components");
-
 var _ResultsPage = require("../../shared/components/ResultsPage");
-
 var _theming = require("@storybook/theming");
-
 var _helpers = require("../../shared/helpers");
-
 var _TestSelect = _interopRequireDefault(require("./TestSelect"));
-
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 const Wrapper = _theming.styled.div(_ref => {
   let {
     isRunning
@@ -28,20 +20,16 @@ const Wrapper = _theming.styled.div(_ref => {
     height: '100%'
   };
 });
-
 const TestSelectContainer = _theming.styled.div(_ref2 => {
   let {
     theme
   } = _ref2;
   return {
     padding: '8px',
-    border: "1px solid ".concat(theme.appBorderColor)
+    border: `1px solid ${theme.appBorderColor}`
   };
 });
-
 const Panel = _ref3 => {
-  var _result$results, _result$results$lengt, _result$results2;
-
   let {
     tests,
     selectedTestId,
@@ -49,16 +37,16 @@ const Panel = _ref3 => {
     onImageApprove
   } = _ref3;
   const result = tests.find(x => x.id === selectedTestId);
-  const isRunning = (result === null || result === void 0 ? void 0 : result.status) === 'running';
+  const isRunning = result?.status === 'running';
   return /*#__PURE__*/_react.default.createElement("div", null, tests.length > 1 && /*#__PURE__*/_react.default.createElement(TestSelectContainer, null, /*#__PURE__*/_react.default.createElement(_TestSelect.default, {
     tests: tests,
     selectedTestId: selectedTestId,
     onChangeTest: onChangeTest
-  })), isRunning && /*#__PURE__*/_react.default.createElement(_components.Loader, null), result !== null && result !== void 0 && (_result$results = result.results) !== null && _result$results !== void 0 && _result$results.length ? /*#__PURE__*/_react.default.createElement(Wrapper, {
+  })), isRunning && /*#__PURE__*/_react.default.createElement(_components.Loader, null), result?.results?.length ? /*#__PURE__*/_react.default.createElement(Wrapper, {
     isRunning: isRunning
   }, /*#__PURE__*/_react.default.createElement(_ResultsPage.ResultsPage, {
     height: '100%',
-    key: "".concat(result.id, "_").concat((_result$results$lengt = (_result$results2 = result.results) === null || _result$results2 === void 0 ? void 0 : _result$results2.length) !== null && _result$results$lengt !== void 0 ? _result$results$lengt : 0) // TODO
+    key: `${result.id}_${result.results?.length ?? 0}` // TODO
     ,
     id: result.id,
     path: (0, _helpers.getTestPath)(result) // TODO Memo?
@@ -68,5 +56,4 @@ const Panel = _ref3 => {
     onImageApprove: onImageApprove
   })) : null);
 };
-
 exports.Panel = Panel;

package/lib/cjs/client/addon/components/TestSelect.js

@@ -4,17 +4,11 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = TestSelect;
-
 var _components = require("@storybook/components");
-
 var _theming = require("@storybook/theming");
-
 var _react = _interopRequireDefault(require("react"));
-
 var _utils = require("../utils");
-
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-
+function _interopRequireDefault(e) { return e && e.__esModule ? e : { default: e }; }
 const LinkIconContainer = _theming.styled.span(_ref => {
   let {
     theme
@@ -26,11 +20,8 @@ const LinkIconContainer = _theming.styled.span(_ref => {
     textAlign: 'center'
   };
 });
-
 function TestSelect(props) {
-  var _props$tests$find$tes, _props$tests$find;
-
-  const testName = (_props$tests$find$tes = (_props$tests$find = props.tests.find(x => x.id === props.selectedTestId)) === null || _props$tests$find === void 0 ? void 0 : _props$tests$find.testName) !== null && _props$tests$find$tes !== void 0 ? _props$tests$find$tes : '';
+  const testName = props.tests.find(x => x.id === props.selectedTestId)?.testName ?? '';
   return /*#__PURE__*/_react.default.createElement(_components.WithTooltip, {
     trigger: "click",
     placement: "bottom",
@@ -40,20 +31,16 @@ function TestSelect(props) {
         onHide
       } = _ref2;
       return /*#__PURE__*/_react.default.createElement(_components.TooltipLinkList, {
-        links: props.tests.map(x => {
-          var _x$testName;
-
-          return {
-            id: x.id,
-            title: (_x$testName = x.testName) !== null && _x$testName !== void 0 ? _x$testName : '',
-            active: props.selectedTestId === x.id,
-            onClick: () => {
-              props.onChangeTest(x.id);
-              onHide();
-            },
-            left: /*#__PURE__*/_react.default.createElement(LinkIconContainer, null, (0, _utils.getEmojiByTestStatus)(x.status, x.skip))
-          };
-        })
+        links: props.tests.map(x => ({
+          id: x.id,
+          title: x.testName ?? '',
+          active: props.selectedTestId === x.id,
+          onClick: () => {
+            props.onChangeTest(x.id);
+            onHide();
+          },
+          left: /*#__PURE__*/_react.default.createElement(LinkIconContainer, null, (0, _utils.getEmojiByTestStatus)(x.status, x.skip))
+        }))
       });
     }
   }, /*#__PURE__*/_react.default.createElement(_components.Button, {