Merge pull request #1571 from fedspendingtransparency/FDG-10026

FDG-10026 GAS dataset is not displaying Published reports
fedspendingtransparency · Jan 7, 2025 · f21bcbf · f21bcbf
2 parents 031d8bc + f19e4ac
commit f21bcbf
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/...reports/download-report-table/download-report-table-row/download-report-table-row.spec.js b/...reports/download-report-table/download-report-table-row/download-report-table-row.spec.js
@@ -1,5 +1,5 @@
 import React from 'react';
-import { render, act, fireEvent } from '@testing-library/react';
+import { render, act, fireEvent, getByText } from '@testing-library/react';
 import DownloadReportTableRow from './download-report-table-row';
 import userEvent from '@testing-library/user-event';
 
@@ -9,6 +9,7 @@ describe('Download report table row component', () => {
     { path: '/test/file/path/another_file.xml', report_date: 'Fri Jul 19 2024 00:00:00 GMT-0500', report_group_desc: 'Another Download File (.xml)' },
     { path: '/test/file/path/another_file.xls', report_date: 'Fri Jul 19 2024 00:00:00 GMT-0500', report_group_desc: 'Another Download File (.xls)' },
     { path: '/test/file/path/another_file.txt', report_date: 'Fri Jul 19 2024 00:00:00 GMT-0500', report_group_desc: 'Another Download File (.txt)' },
+    { path: '/test/file/path/another_file.txt', report_date: 'Fri Jul 19 2024 00:00:00 GMT-0500', report_group_desc: 'TST (.txt)' },
   ];
 
   beforeEach(() => {
@@ -20,6 +21,11 @@ describe('Download report table row component', () => {
     expect(getByTestId('file-download-row')).toBeInTheDocument();
   });
 
+  it('renders a short file name in the row', () => {
+    const { getByText } = render(<DownloadReportTableRow reportFile={mockReports[4]} />);
+    expect(getByText('TST.txt')).toBeInTheDocument();
+  });
+
   it('renders a pdf icon with a pdf filename', () => {
     const { getByAltText } = render(<DownloadReportTableRow reportFile={mockReports[0]} />);
     expect(getByAltText('.pdf icon')).toBeInTheDocument();

diff --git a/...hed-reports/download-report-table/download-report-table-row/download-report-table-row.tsx b/...hed-reports/download-report-table/download-report-table-row/download-report-table-row.tsx
@@ -99,7 +99,7 @@ const DownloadReportTableRow: FunctionComponent<{ reportFile: IPublishedReportDa
 
   return (
     <>
-      {displayName?.start && displayName?.end && (
+      {displayName?.end && (
         <tr className={fileDescription} data-testid="file-download-row">
           <td>
             <a
@@ -116,7 +116,7 @@ const DownloadReportTableRow: FunctionComponent<{ reportFile: IPublishedReportDa
                   <div className={downloadFileContainer}>
                     <div className={downloadName}>
                       <img src={fileTypeImage} alt={`${fileType} icon`} />
-                      <span className={startName}>{displayName.start}</span>
+                      {displayName?.start && <span className={startName}>{displayName.start}</span>}
                       <span>{displayName.end}</span>
                     </div>
                     <div className={fileDate}>{publishedDate}</div>
@@ -132,7 +132,7 @@ const DownloadReportTableRow: FunctionComponent<{ reportFile: IPublishedReportDa
                   <img src={fileTypeImage} alt={`${fileType} icon`} />
                   <div className={downloadItem}>
                     <div className={downloadName}>
-                      <div className={startName}>{displayName.start}</div>
+                      {displayName?.start && <div className={startName}>{displayName.start}</div>}
                       <div className={endName}>{displayName.end}</div>
                     </div>
                     <div className={downloadInfo}>