From 1498c2da902efb8df23b1e2dc6d56e78e9b36ce2 Mon Sep 17 00:00:00 2001
From: LeeDr <lee.drengenberg@elastic.co>
Date: Wed, 12 Feb 2020 17:27:03 -0600
Subject: [PATCH] re-work change to not break jest test

---
 test/functional/apps/home/_sample_data.ts |  8 ++++----
 test/functional/page_objects/home_page.ts | 15 +++++++--------
 2 files changed, 11 insertions(+), 12 deletions(-)

diff --git a/test/functional/apps/home/_sample_data.ts b/test/functional/apps/home/_sample_data.ts
index 8088b5a0f9da9..8bc528e045566 100644
--- a/test/functional/apps/home/_sample_data.ts
+++ b/test/functional/apps/home/_sample_data.ts
@@ -84,7 +84,7 @@ export default function({ getService, getPageObjects }: FtrProviderContext) {
       });
 
       it('should launch sample flights data set dashboard', async () => {
-        await PageObjects.home.launchSampleDataSet('flights');
+        await PageObjects.home.launchSampleDashboard('flights');
         await PageObjects.header.waitUntilLoadingHasFinished();
         await renderable.waitForRender();
         const todayYearMonthDay = moment().format('MMM D, YYYY');
@@ -96,7 +96,7 @@ export default function({ getService, getPageObjects }: FtrProviderContext) {
       });
 
       it('should render visualizations', async () => {
-        await PageObjects.home.launchSampleDataSet('flights');
+        await PageObjects.home.launchSampleDashboard('flights');
         await PageObjects.header.waitUntilLoadingHasFinished();
         await renderable.waitForRender();
         log.debug('Checking pie charts rendered');
@@ -115,7 +115,7 @@ export default function({ getService, getPageObjects }: FtrProviderContext) {
       });
 
       it('should launch sample logs data set dashboard', async () => {
-        await PageObjects.home.launchSampleDataSet('logs');
+        await PageObjects.home.launchSampleDashboard('logs');
         await PageObjects.header.waitUntilLoadingHasFinished();
         await renderable.waitForRender();
         const todayYearMonthDay = moment().format('MMM D, YYYY');
@@ -127,7 +127,7 @@ export default function({ getService, getPageObjects }: FtrProviderContext) {
       });
 
       it('should launch sample ecommerce data set dashboard', async () => {
-        await PageObjects.home.launchSampleDataSet('ecommerce');
+        await PageObjects.home.launchSampleDashboard('ecommerce');
         await PageObjects.header.waitUntilLoadingHasFinished();
         await renderable.waitForRender();
         const todayYearMonthDay = moment().format('MMM D, YYYY');
diff --git a/test/functional/page_objects/home_page.ts b/test/functional/page_objects/home_page.ts
index 63cb3356de720..6225b4e3aca62 100644
--- a/test/functional/page_objects/home_page.ts
+++ b/test/functional/page_objects/home_page.ts
@@ -66,20 +66,19 @@ export function HomePageProvider({ getService, getPageObjects }: FtrProviderCont
       });
     }
 
-    async launchSampleDataSet(id: string) {
-      await this.addSampleDataSet(id);
-      await testSubjects.click(`launchSampleDataSet${id}`);
-      // On OSS there's currently only sample dashboards so the launch button opens the dashboard
-      // but on default dist there's more items. The only tests that are calling this method are in OSS
-      // so they always expect Dashboard.  isOss isn't really the right test. It should be based on
-      // the number of items.
-      // x-pack sample data tests seem to skip this navigation and open the saved objects directly.
+    async launchSampleDashboard(id: string) {
+      await this.launchSampleDataSet(id);
       isOss = await PageObjects.common.isOss();
       if (!isOss) {
         await find.clickByLinkText('Dashboard');
       }
     }
 
+    async launchSampleDataSet(id: string) {
+      await this.addSampleDataSet(id);
+      await testSubjects.click(`launchSampleDataSet${id}`);
+    }
+
     async loadSavedObjects() {
       await retry.try(async () => {
         await testSubjects.click('loadSavedObjects');