STDTC-100 Formatting a list of jobs without the progress field result…

…s in an error on the UI (#280)
folio-org · Dec 13, 2023 · 1adca1a · 1adca1a
1 parent 88bb67c
commit 1adca1a
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 ## IN PROGRESS
 
 * Add onfilter function to ViewAllLogsFilter. Refs STDTC-99.
+* Formatting a list of jobs without the "progress" field results in an error on the UI. Refs STDTC-100.
 
 ## [6.0.0](https://github.com/folio-org/stripes-data-transfer-components/tree/v6.0.0) (2023-10-12)
 [Full Changelog](https://github.com/folio-org/stripes-data-transfer-components/compare/v5.4.1...v6.0.0)

diff --git a/lib/ListTemplate/listTemplate.js b/lib/ListTemplate/listTemplate.js
@@ -162,7 +162,7 @@ export const listTemplate = ({
     );
   },
   jobProfileName: record => record.jobProfileInfo?.name,
-  totalRecords: record => formatNumber(record.progress.total),
+  totalRecords: record => formatNumber(record.progress?.total || 0),
   fileName: record => record.fileName,
   hrId: record => record.hrId,
 });
diff --git a/lib/ListTemplate/listTemplate.test.js b/lib/ListTemplate/listTemplate.test.js
@@ -260,6 +260,12 @@ describe('List template', () => {
 
       expect(getByText('0')).toBeDefined();
     });
+
+    it('then appropriate text should be rendered when progress is not defined', () => {
+      const { getByText } = renderWithIntl(templates.totalRecords({ ...testRecord, progress: undefined }));
+
+      expect(getByText('0')).toBeDefined();
+    });
   });
 
   describe('when column is `fileName`', () => {