test: enable / remove skipped tests

Enable tests that seem to work.
Remove tests for removed functionality.
Enable "mocha/no-pending-tests" lint.

Author: hasezoey

test/document.test.js

@@ -9,7 +9,6 @@ const start = require('./common');
 const Document = require('../lib/document');
 const EventEmitter = require('events').EventEmitter;
 const ArraySubdocument = require('../lib/types/arraySubdocument');
-const Query = require('../lib/query');
 const assert = require('assert');
 const idGetter = require('../lib/helpers/schema/idGetter');
 const sinon = require('sinon');
@@ -1203,38 +1202,6 @@ describe('document', function() {
     });
   });
 
-  describe.skip('#update', function() {
-    it('returns a Query', function() {
-      const mg = new mongoose.Mongoose();
-      const M = mg.model('Test', { s: String });
-      const doc = new M();
-      assert.ok(doc.update() instanceof Query);
-    });
-    it('calling update on document should relay to its model (gh-794)', async function() {
-      const Docs = new Schema({ text: String });
-      const docs = db.model('Test', Docs);
-      const d = new docs({ text: 'A doc' });
-      let called = false;
-      await d.save();
-
-      const oldUpdate = docs.update;
-      docs.update = function(query, operation) {
-        assert.equal(Object.keys(query).length, 1);
-        assert.equal(d._id, query._id);
-        assert.equal(Object.keys(operation).length, 1);
-        assert.equal(Object.keys(operation.$set).length, 1);
-        assert.equal(operation.$set.text, 'A changed doc');
-        called = true;
-        docs.update = oldUpdate;
-        oldUpdate.apply(docs, arguments);
-      };
-
-      await d.update({ $set: { text: 'A changed doc' } });
-
-      assert.equal(called, true);
-    });
-  });
-
   it('toObject should not set undefined values to null', function() {
     const doc = new TestDocument();
     const obj = doc.toObject();
@@ -10500,8 +10467,9 @@ describe('document', function() {
         assert.equal(user.init, 12);
       });
     });
-    xdescribe('Document#collection', () => {
-      it('is available as `$collection`', async() => {
+
+    describe('Document#collection', function() {
+      it('is available as `$collection`', async function() {
         const userSchema = new Schema({ name: String });
         const User = db.model('User', userSchema);
 
@@ -10754,8 +10722,7 @@ describe('document', function() {
     });
 
     // skip until gh-10367 is implemented
-    xit('support `pathsToSkip` option for `Model.validate()`', async() => {
-
+    it.skip('support `pathsToSkip` option for `Model.validate()`', async function() {
       const User = getUserModel();
       const err1 = await User.validate({}, { pathsToSkip: ['age'] });
       assert.deepEqual(Object.keys(err1.errors), ['name']);

test/model.indexes.test.js

@@ -384,7 +384,7 @@ describe('model', function() {
       });
     });
 
-    describe.skip('model.ensureIndexes()', function() {
+    describe('model.ensureIndexes()', function() {
       it('is a function', function() {
         const schema = mongoose.Schema({ x: 'string' });
         const Test = mongoose.createConnection().model('ensureIndexes-' + random, schema);

test/model.test.js

@@ -3186,6 +3186,7 @@ describe('Model', function() {
 
     });
 
+    // the following causes "MongoServerError: ns not found" errors in mongodb 6.0.x
     it.skip('Compound index with 2dsphere field without value is saved', async function() {
       const PersonSchema = new Schema({
         name: String,
@@ -3217,9 +3218,9 @@ describe('Model', function() {
       assert.equal(personDoc.loc, undefined);
 
       await Person.collection.drop();
-
     });
 
+    // the following causes "MongoServerError: ns not found" errors in mongodb 6.0.x
     it.skip('Compound index on field earlier declared with 2dsphere index is saved', async function() {
       const PersonSchema = new Schema({
         name: String,
@@ -3252,7 +3253,6 @@ describe('Model', function() {
       assert.equal(personDoc.loc, undefined);
 
       await Person.collection.drop();
-
     });
   });
 

test/query.test.js

@@ -1498,9 +1498,9 @@ describe('Query', function() {
           then(() => Product.find().sort({ _id: 1 }).countDocuments({}).exec());
       });
 
-      it.skip('ignores count when passed to sort', function() {
+      it('ignores count when passed to sort', function() {
         const Product = db.model('Product', productSchema);
-        return Product.find().count({}).sort({ _id: 1 }).exec();
+        return Product.find().countDocuments({}).sort({ _id: 1 }).exec();
       });
     });