From ebd5c8d242883ef6bd3ef57235143089d5cc830a Mon Sep 17 00:00:00 2001
From: Andrew Goldstein <andrew.goldstein@elastic.co>
Date: Thu, 5 Dec 2019 18:05:46 -0700
Subject: [PATCH] ## [SIEM] Remove placeholder from pinned event tooltips

Similar to signals, pinned timeline events should be copied from source indexes, which are subject to ILM, to separate (space-aware) indexes (with different ILM), such that pinned events can be viewed in a timeline after the events have aged out of the original indexes.

The backend APIs and UI patterns in development now for signals can likely be reused to implement the above, but until then, the placeholder tooltip text for unpinned / pinned events, which mentions persistence, should be removed from the SIEM beta.

- [x] Changed the _unpinned_ event tooltip text from (sic) `This is event is NOT persisted with the timeline` to `Unpinned event`
- [x] Changed the pinned event tooltip text from `This event is persisted with the timeline` to `Pinned event`

https://github.com/elastic/siem-team/issues/482
---
 .../components/timeline/body/helpers.test.ts  | 20 +++++++------------
 .../components/timeline/body/translations.ts  |  4 ++--
 2 files changed, 9 insertions(+), 15 deletions(-)

diff --git a/x-pack/legacy/plugins/siem/public/components/timeline/body/helpers.test.ts b/x-pack/legacy/plugins/siem/public/components/timeline/body/helpers.test.ts
index 83099c68e4ca9..602b88b7ae6d6 100644
--- a/x-pack/legacy/plugins/siem/public/components/timeline/body/helpers.test.ts
+++ b/x-pack/legacy/plugins/siem/public/components/timeline/body/helpers.test.ts
@@ -203,28 +203,22 @@ describe('helpers', () => {
   });
 
   describe('getPinTooltip', () => {
-    test('it informs the user the event may not be unpinned when the event is pinned and has notes', () => {
+    test('it indicates the event may NOT be unpinned when `isPinned` is `true` and the event has notes', () => {
       expect(getPinTooltip({ isPinned: true, eventHasNotes: true })).toEqual(
         'This event cannot be unpinned because it has notes'
       );
     });
 
-    test('it tells the user the event is persisted when the event is pinned, but has no notes', () => {
-      expect(getPinTooltip({ isPinned: true, eventHasNotes: false })).toEqual(
-        'This event is persisted with the timeline'
-      );
+    test('it indicates the event is pinned when `isPinned` is `true` and the event does NOT have notes', () => {
+      expect(getPinTooltip({ isPinned: true, eventHasNotes: false })).toEqual('Pinned event');
     });
 
-    test('it tells the user the event is NOT persisted when the event is not pinned, but it has notes', () => {
-      expect(getPinTooltip({ isPinned: false, eventHasNotes: true })).toEqual(
-        'This is event is NOT persisted with the timeline'
-      );
+    test('it indicates the event is NOT pinned when `isPinned` is `false` and the event has notes', () => {
+      expect(getPinTooltip({ isPinned: false, eventHasNotes: true })).toEqual('Unpinned event');
     });
 
-    test('it tells the user the event is NOT persisted when the event is not pinned, and has no notes', () => {
-      expect(getPinTooltip({ isPinned: false, eventHasNotes: false })).toEqual(
-        'This is event is NOT persisted with the timeline'
-      );
+    test('it indicates the event is NOT pinned when `isPinned` is `false` and the event does NOT have notes', () => {
+      expect(getPinTooltip({ isPinned: false, eventHasNotes: false })).toEqual('Unpinned event');
     });
   });
 
diff --git a/x-pack/legacy/plugins/siem/public/components/timeline/body/translations.ts b/x-pack/legacy/plugins/siem/public/components/timeline/body/translations.ts
index 40e94cb099644..03efc26bbc9b1 100644
--- a/x-pack/legacy/plugins/siem/public/components/timeline/body/translations.ts
+++ b/x-pack/legacy/plugins/siem/public/components/timeline/body/translations.ts
@@ -21,11 +21,11 @@ export const COPY_TO_CLIPBOARD = i18n.translate(
 );
 
 export const UNPINNED = i18n.translate('xpack.siem.timeline.body.pinning.unpinnedTooltip', {
-  defaultMessage: 'This is event is NOT persisted with the timeline',
+  defaultMessage: 'Unpinned event',
 });
 
 export const PINNED = i18n.translate('xpack.siem.timeline.body.pinning.pinnedTooltip', {
-  defaultMessage: 'This event is persisted with the timeline',
+  defaultMessage: 'Pinned event',
 });
 
 export const PINNED_WITH_NOTES = i18n.translate(