From 52e9c51d152a831ee9df0cd50bda95c54aef11bb Mon Sep 17 00:00:00 2001
From: Dominique Clarke <doclarke71@gmail.com>
Date: Mon, 14 Feb 2022 12:10:47 -0500
Subject: [PATCH] [Uptime] handle null duration on ping list (#125438)

* uptime - handle null duration

* adjust types

* Update x-pack/plugins/uptime/server/lib/synthetics_service/service_api_client.ts

* raise per page of synthetics monitors

* Remove tooltip
---
 .../plugins/uptime/common/runtime_types/ping/ping.ts  |  6 +++---
 .../components/monitor/ping_list/ping_list.test.tsx   | 11 +++++++----
 .../components/monitor/ping_list/ping_list_table.tsx  |  7 ++++++-
 .../components/overview/monitor_list/monitor_list.tsx |  2 +-
 .../lib/synthetics_service/synthetics_service.ts      |  1 +
 5 files changed, 18 insertions(+), 9 deletions(-)

diff --git a/x-pack/plugins/uptime/common/runtime_types/ping/ping.ts b/x-pack/plugins/uptime/common/runtime_types/ping/ping.ts
index 989d6d8ef941a..e78f026277d3a 100644
--- a/x-pack/plugins/uptime/common/runtime_types/ping/ping.ts
+++ b/x-pack/plugins/uptime/common/runtime_types/ping/ping.ts
@@ -90,14 +90,14 @@ export type Tls = t.TypeOf<typeof TlsType>;
 
 export const MonitorType = t.intersection([
   t.type({
-    duration: t.type({
-      us: t.number,
-    }),
     id: t.string,
     status: t.string,
     type: t.string,
   }),
   t.partial({
+    duration: t.type({
+      us: t.number,
+    }),
     check_group: t.string,
     ip: t.string,
     name: t.string,
diff --git a/x-pack/plugins/uptime/public/components/monitor/ping_list/ping_list.test.tsx b/x-pack/plugins/uptime/public/components/monitor/ping_list/ping_list.test.tsx
index c185303447854..bfcf359ac0525 100644
--- a/x-pack/plugins/uptime/public/components/monitor/ping_list/ping_list.test.tsx
+++ b/x-pack/plugins/uptime/public/components/monitor/ping_list/ping_list.test.tsx
@@ -42,7 +42,6 @@ describe('PingList component', () => {
         type: 'io',
       },
       monitor: {
-        duration: { us: 1370 },
         id: 'auto-tcp-0X81440A68E839814D',
         ip: '255.255.255.0',
         name: '',
@@ -161,9 +160,6 @@ describe('PingList component', () => {
                 "type": "io",
               },
               "monitor": Object {
-                "duration": Object {
-                  "us": 1370,
-                },
                 "id": "auto-tcp-0X81440A68E839814D",
                 "ip": "255.255.255.0",
                 "name": "",
@@ -186,6 +182,13 @@ describe('PingList component', () => {
       });
     });
 
+    describe('duration column', () => {
+      it('shows -- when duration is null', () => {
+        const { getByTestId } = render(<PingList />);
+        expect(getByTestId('ping-list-duration-unavailable-tool-tip')).toBeInTheDocument();
+      });
+    });
+
     describe('formatDuration', () => {
       it('returns zero for < 1 millisecond', () => {
         expect(formatDuration(984)).toBe('0 ms');
diff --git a/x-pack/plugins/uptime/public/components/monitor/ping_list/ping_list_table.tsx b/x-pack/plugins/uptime/public/components/monitor/ping_list/ping_list_table.tsx
index 84a2d6a5d6a31..5e2737684b333 100644
--- a/x-pack/plugins/uptime/public/components/monitor/ping_list/ping_list_table.tsx
+++ b/x-pack/plugins/uptime/public/components/monitor/ping_list/ping_list_table.tsx
@@ -140,7 +140,12 @@ export function PingListTable({ loading, error, pings, pagination, onChange, fai
       name: i18n.translate('xpack.uptime.pingList.durationMsColumnLabel', {
         defaultMessage: 'Duration',
       }),
-      render: (duration: number) => formatDuration(duration),
+      render: (duration: number | null) =>
+        duration ? (
+          formatDuration(duration)
+        ) : (
+          <span data-test-subj="ping-list-duration-unavailable-tool-tip">{'--'}</span>
+        ),
     },
     ...(hasError
       ? [
diff --git a/x-pack/plugins/uptime/public/components/overview/monitor_list/monitor_list.tsx b/x-pack/plugins/uptime/public/components/overview/monitor_list/monitor_list.tsx
index 2dd4ed7bed481..a2d823cd90af1 100644
--- a/x-pack/plugins/uptime/public/components/overview/monitor_list/monitor_list.tsx
+++ b/x-pack/plugins/uptime/public/components/overview/monitor_list/monitor_list.tsx
@@ -135,7 +135,7 @@ export const MonitorListComponent: ({
               timestamp={timestamp}
               summaryPings={summaryPings ?? []}
               monitorType={type}
-              duration={duration!.us}
+              duration={duration?.us}
               monitorId={monitorId}
             />
           );
diff --git a/x-pack/plugins/uptime/server/lib/synthetics_service/synthetics_service.ts b/x-pack/plugins/uptime/server/lib/synthetics_service/synthetics_service.ts
index 450ab324e7e48..12e13cd22f748 100644
--- a/x-pack/plugins/uptime/server/lib/synthetics_service/synthetics_service.ts
+++ b/x-pack/plugins/uptime/server/lib/synthetics_service/synthetics_service.ts
@@ -280,6 +280,7 @@ export class SyntheticsService {
     const findResult = await savedObjectsClient.find<SyntheticsMonitor>({
       type: syntheticsMonitorType,
       namespaces: ['*'],
+      perPage: 10000,
     });
 
     hydrateSavedObjects({