[Reporting/Download CSV] Get the file name from savedSearch data

src/plugins/embeddable/public/lib/panel/embeddable_panel.tsx

@@ -211,6 +211,7 @@ export class EmbeddablePanel extends React.Component<Props, State> {
       <EuiPanel
         className={classes}
         data-test-subj="embeddablePanel"
+        data-test-embeddable-id={this.props.embeddable.id}
         paddingSize="none"
         role="figure"
         aria-labelledby={headerId}

x-pack/plugins/reporting/public/panel_actions/get_csv_panel_action.tsx

@@ -105,7 +105,7 @@ export class GetCsvReportPanelAction implements ActionDefinition<ActionContext>
     const kibanaTimezone = this.core.uiSettings.get('dateFormat:tz');
 
     const id = `search:${embeddable.getSavedSearch().id}`;
-    const filename = embeddable.getTitle();
+    const filename = embeddable.getSavedSearch().title;
     const timezone = kibanaTimezone === 'Browser' ? moment.tz.guess() : kibanaTimezone;
     const fromTime = dateMath.parse(from);
     const toTime = dateMath.parse(to);

x-pack/test/functional/apps/dashboard/reporting/download_csv.ts

@@ -14,15 +14,27 @@ import { FtrProviderContext } from '../../../ftr_provider_context';
 
 const csvPath = path.resolve(REPO_ROOT, 'target/functional-tests/downloads/Ecommerce Data.csv');
 
+// checks every 100ms for the file to exist in the download dir
+// just wait up to 5 seconds
+const getDownload$ = (filePath: string) => {
+  return Rx.interval(100).pipe(
+    map(() => fs.existsSync(filePath)),
+    filter((value) => value === true),
+    first(),
+    timeout(5000)
+  );
+};
+
 export default function ({ getService, getPageObjects }: FtrProviderContext) {
   const esArchiver = getService('esArchiver');
   const browser = getService('browser');
   const dashboardPanelActions = getService('dashboardPanelActions');
   const log = getService('log');
   const testSubjects = getService('testSubjects');
+  const find = getService('find');
   const PageObjects = getPageObjects(['reporting', 'common', 'dashboard']);
 
-  describe('Reporting Download CSV', () => {
+  describe('Download CSV', () => {
     before('initialize tests', async () => {
       log.debug('ReportingPage:initTests');
       await esArchiver.loadIfNeeded('reporting/ecommerce');
@@ -33,10 +45,13 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
     after('clean up archives and previous file download', async () => {
       await esArchiver.unload('reporting/ecommerce');
       await esArchiver.unload('reporting/ecommerce_kibana');
+    });
+
+    afterEach('remove download', () => {
       try {
         fs.unlinkSync(csvPath);
       } catch (e) {
-        // nothing to worry
+        // it might not have been there to begin with
       }
     });
 
@@ -50,19 +65,26 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
       await testSubjects.click('embeddablePanelAction-downloadCsvReport');
       await testSubjects.existOrFail('csvDownloadStarted'); // validate toast panel
 
-      // check every 100ms for the file to exist in the download dir
-      // just wait up to 5 seconds
-      const success$ = Rx.interval(100).pipe(
-        map(() => fs.existsSync(csvPath)),
-        filter((value) => value === true),
-        first(),
-        timeout(5000)
-      );
-
-      const fileExists = await success$.toPromise();
+      const fileExists = await getDownload$(csvPath).toPromise();
       expect(fileExists).to.be(true);
 
       // no need to validate download contents, API Integration tests do that some different variations
     });
+
+    it('Gets the correct filename if panel titles are hidden', async () => {
+      await PageObjects.common.navigateToApp('dashboard');
+      await PageObjects.dashboard.loadSavedDashboard('Ecom Dashboard Hidden Panel Titles');
+      const savedSearchPanel = await find.byCssSelector(
+        '[data-test-embeddable-id="94eab06f-60ac-4a85-b771-3a8ed475c9bb"]'
+      ); // panel title is hidden
+      await dashboardPanelActions.toggleContextMenu(savedSearchPanel);
+
+      await testSubjects.existOrFail('embeddablePanelAction-downloadCsvReport');
+      await testSubjects.click('embeddablePanelAction-downloadCsvReport');
+      await testSubjects.existOrFail('csvDownloadStarted');
+
+      const fileExists = await getDownload$(csvPath).toPromise(); // file exists with proper name
+      expect(fileExists).to.be(true);
+    });
   });
 }