Fix

alxndrsn · Apr 12, 2024 · aeb723d · aeb723d
1 parent 2e41bd9
commit aeb723d
Showing 1 changed file with 9 additions and 31 deletions.
diff --git a/tests/integration/test.basics.js b/tests/integration/test.basics.js
@@ -747,8 +747,8 @@ adapters.forEach(function (adapter) {
     });
 
     [
-      null,
       undefined,
+      null,
       [],
       [{ _id: 'foo' }, { _id: 'bar' }],
       'this is not an object',
@@ -758,48 +758,26 @@ adapters.forEach(function (adapter) {
       describe(`Should error when document is not an object #${idx}`, () => {
         let db;
 
-        beforeEach(() => {
-          db = new PouchDB(dbs.name);
-        });
-
-        it('should error for .post()', async () => {
+        const expectNotAnObject = fn => async () => {
           let threw;
           try {
-            await db.post(badDoc);
+            await fn();
           } catch (err) {
             threw = true;
             err.message.should.equal('Document must be a JSON object');
           }
           if (!threw) {
             throw new Error('should have thrown');
           }
-        });
+        };
 
-        it('should error for .put()', async () => {
-          let threw;
-          try {
-            await db.post(badDoc);
-          } catch (err) {
-            threw = true;
-            err.message.should.equal('Document must be a JSON object');
-          }
-          if (!threw) {
-            throw new Error('should have thrown');
-          }
+        beforeEach(() => {
+          db = new PouchDB(dbs.name);
         });
 
-        it('should error for .bulkDocs()', async () => {
-          let threw;
-          try {
-            await db.bulkDocs({docs: [badDoc]});
-          } catch (err) {
-            threw = true;
-            err.message.should.equal('Document must be a JSON object');
-          }
-          if (!threw) {
-            throw new Error('should have thrown');
-          }
-        });
+        it('should error for .post()', expectNotAnObject(() => db.post(badDoc)));
+        it('should error for .put()', expectNotAnObject(() => db.put(badDoc)));
+        it('should error for .bulkDocs()', expectNotAnObject(() => db.bulkDocs({docs: [badDoc]})));
       });
     });