chore: use insertOne() instead of insert() in tests re: #6880

Automattic · Aug 20, 2018 · 44acd50 · 44acd50
1 parent 7ecce58
commit 44acd50
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 3 deletions.
diff --git a/lib/collection.js b/lib/collection.js
@@ -198,6 +198,22 @@ Collection.prototype.insert = function() {
   throw new Error('Collection#insert unimplemented by driver');
 };
 
+/**
+ * Abstract method that drivers must implement.
+ */
+
+Collection.prototype.insertOne = function() {
+  throw new Error('Collection#insertOne unimplemented by driver');
+};
+
+/**
+ * Abstract method that drivers must implement.
+ */
+
+Collection.prototype.insertMany = function() {
+  throw new Error('Collection#insertMany unimplemented by driver');
+};
+
 /**
  * Abstract method that drivers must implement.
  */

diff --git a/test/collection.test.js b/test/collection.test.js
@@ -23,7 +23,7 @@ describe('collections:', function() {
       done();
     }
 
-    collection.insert({}, {}, function() {
+    collection.insertOne({}, {}, function() {
       assert.ok(connected);
       inserted = true;
       db.close();
@@ -72,9 +72,9 @@ describe('collections:', function() {
     thrown = false;
 
     try {
-      collection.insert();
+      collection.insertOne();
     } catch (e) {
-      assert.ok(/unimplemented/.test(e.message));
+      assert.ok(/unimplemented/.test(e.message), e.message);
       thrown = true;
     }