From a6764f267f21bd8dd4fca3b6b70032c247825f95 Mon Sep 17 00:00:00 2001
From: Justin Kambic <justin.kambic@elastic.co>
Date: Fri, 2 Oct 2020 15:11:16 -0400
Subject: [PATCH] Fix console log and add a test.

---
 .../step_screenshot_display.test.tsx          | 74 ++++++++++++++++++-
 .../uptime/public/state/selectors/index.ts    |  5 +-
 2 files changed, 74 insertions(+), 5 deletions(-)

diff --git a/x-pack/plugins/uptime/public/components/monitor/synthetics/__tests__/step_screenshot_display.test.tsx b/x-pack/plugins/uptime/public/components/monitor/synthetics/__tests__/step_screenshot_display.test.tsx
index 5b3eca6727eb90..16db430dbd73a1 100644
--- a/x-pack/plugins/uptime/public/components/monitor/synthetics/__tests__/step_screenshot_display.test.tsx
+++ b/x-pack/plugins/uptime/public/components/monitor/synthetics/__tests__/step_screenshot_display.test.tsx
@@ -16,7 +16,6 @@ describe('StepScreenshotDisplayProps', () => {
   }));
 
   it('displays screenshot thumbnail when present', () => {
-    // reactUseSpy.
     const wrapper = mountWithIntl(
       <StepScreenshotDisplay
         checkGroup="check_group"
@@ -96,6 +95,79 @@ describe('StepScreenshotDisplayProps', () => {
     `);
   });
 
+  it('uses alternative text when step name not available', () => {
+    const wrapper = mountWithIntl(
+      <StepScreenshotDisplay checkGroup="check_group" screenshotExists={true} stepIndex={1} />
+    );
+
+    wrapper.update();
+
+    expect(wrapper.find('EuiPopover')).toMatchInlineSnapshot(`
+      <EuiPopover
+        anchorPosition="rightCenter"
+        button={
+          <input
+            alt="Screenshot"
+            onClick={[Function]}
+            onMouseEnter={[Function]}
+            onMouseLeave={[Function]}
+            src="/api/uptime/journey/screenshot/check_group/1"
+            style={
+              Object {
+                "height": 180,
+                "objectFit": "cover",
+                "objectPosition": "center top",
+                "width": 320,
+              }
+            }
+            type="image"
+          />
+        }
+        closePopover={[Function]}
+        display="inlineBlock"
+        hasArrow={true}
+        isOpen={false}
+        ownFocus={false}
+        panelPaddingSize="m"
+      >
+        <EuiOutsideClickDetector
+          isDisabled={true}
+          onOutsideClick={[Function]}
+        >
+          <div
+            className="euiPopover euiPopover--anchorRightCenter"
+            onKeyDown={[Function]}
+            onMouseDown={[Function]}
+            onMouseUp={[Function]}
+            onTouchEnd={[Function]}
+            onTouchStart={[Function]}
+          >
+            <div
+              className="euiPopover__anchor"
+            >
+              <input
+                alt="Screenshot"
+                onClick={[Function]}
+                onMouseEnter={[Function]}
+                onMouseLeave={[Function]}
+                src="/api/uptime/journey/screenshot/check_group/1"
+                style={
+                  Object {
+                    "height": 180,
+                    "objectFit": "cover",
+                    "objectPosition": "center top",
+                    "width": 320,
+                  }
+                }
+                type="image"
+              />
+            </div>
+          </div>
+        </EuiOutsideClickDetector>
+      </EuiPopover>
+    `);
+  });
+
   it('displays No Image message when screenshot does not exist', () => {
     expect(
       shallowWithIntl(
diff --git a/x-pack/plugins/uptime/public/state/selectors/index.ts b/x-pack/plugins/uptime/public/state/selectors/index.ts
index 7acb3befa0d6b5..6bfe67468aae58 100644
--- a/x-pack/plugins/uptime/public/state/selectors/index.ts
+++ b/x-pack/plugins/uptime/public/state/selectors/index.ts
@@ -95,7 +95,4 @@ export const selectedFiltersSelector = ({ selectedFilters }: AppState) => select
 
 export const monitorIdSelector = ({ ui: { monitorId } }: AppState) => monitorId;
 
-export const journeySelector = ({ journeys }: AppState) => {
-  console.log('journeys', journeys);
-  return journeys;
-};
+export const journeySelector = ({ journeys }: AppState) => journeys;