fix exporter tests

Signed-off-by: Kerry Archibald <[email protected]>

Author: Kerry Archibald

test/utils/export-test.tsx

@@ -50,24 +50,6 @@ describe('export', function() {
         attachmentsIncluded: false,
     };
 
-    const invalidExportOptions: IExportOptions[] = [
-        {
-            numberOfMessages: 10**9,
-            maxSize: 1024 * 1024 * 1024,
-            attachmentsIncluded: false,
-        },
-        {
-            numberOfMessages: -1,
-            maxSize: 4096 * 1024 * 1024,
-            attachmentsIncluded: false,
-        },
-        {
-            numberOfMessages: 0,
-            maxSize: 0,
-            attachmentsIncluded: false,
-        },
-    ];
-
     function createRoom() {
         const room = new Room(generateRoomId(), null, client.getUserId());
         return room;
@@ -212,13 +194,28 @@ describe('export', function() {
         ).toBeTruthy();
     });
 
-    it('checks if the export options are valid', function() {
-        for (const exportOption of invalidExportOptions) {
-            expect(
-                () =>
-                    new PlainTextExporter(mockRoom, ExportType.Beginning, exportOption, null),
-            ).toThrowError("Invalid export options");
-        }
+    const testCases: [string, IExportOptions][] = [
+        ['numberOfMessages exceeds max', {
+            numberOfMessages: 10 ** 9,
+            maxSize: 1024 * 1024 * 1024,
+            attachmentsIncluded: false,
+        }],
+        ['maxSize exceeds 1000GB', {
+            numberOfMessages: -1,
+            maxSize: 1024001 * 1024 * 1024,
+            attachmentsIncluded: false,
+        }],
+        ['maxSize is less than 1mb', {
+            numberOfMessages: 0,
+            maxSize: 0,
+            attachmentsIncluded: false,
+        }],
+    ];
+    it.each(testCases)('%s', (_d, options) => {
+        expect(
+            () =>
+                new PlainTextExporter(mockRoom, ExportType.Beginning, options, null),
+        ).toThrowError("Invalid export options");
     });
 
     it('tests the file extension splitter', function() {