test: make sure bulk tests catch all errors

Won't change the actual result of the test, but makes sure errors cause a quick error rather than an uncaught error causing a stall
mongodb · Mar 7, 2019 · f73bd06 · f73bd06
1 parent d5873f8
commit f73bd06
Showing 1 changed file with 18 additions and 12 deletions.
diff --git a/test/functional/promises_collection_tests.js b/test/functional/promises_collection_tests.js
@@ -320,13 +320,16 @@ describe('Promises (Collection)', function() {
         var db = client.db(configuration.db);
         var bulk = db.collection('unordered_bulk_promise_form').initializeUnorderedBulkOp({ w: 1 });
         bulk.insert({ a: 1 });
-        bulk.execute().then(function(r) {
-          test.ok(r);
-          test.deepEqual({ w: 1 }, bulk.s.writeConcern);
+        return bulk
+          .execute()
+          .then(function(r) {
+            test.ok(r);
+            test.deepEqual({ w: 1 }, bulk.s.writeConcern);
 
-          client.close();
-          done();
-        });
+            client.close();
+            done();
+          })
+          .catch(done);
       });
     }
   });
@@ -352,13 +355,16 @@ describe('Promises (Collection)', function() {
         var db = client.db(configuration.db);
         var bulk = db.collection('unordered_bulk_promise_form').initializeOrderedBulkOp({ w: 1 });
         bulk.insert({ a: 1 });
-        bulk.execute().then(function(r) {
-          test.ok(r);
-          test.deepEqual({ w: 1 }, bulk.s.writeConcern);
+        return bulk
+          .execute()
+          .then(function(r) {
+            test.ok(r);
+            test.deepEqual({ w: 1 }, bulk.s.writeConcern);
 
-          client.close();
-          done();
-        });
+            client.close();
+            done();
+          })
+          .catch(done);
       });
     }
   });