fix: ts and lint

mongodb · Jun 29, 2021 · 3fd26fe · 3fd26fe
1 parent 78f6e72
commit 3fd26fe
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/src/operations/is_capped.ts b/src/operations/is_capped.ts
@@ -22,7 +22,7 @@ export class IsCappedOperation extends AbstractOperation<boolean> {
     coll.s.db
       .listCollections(
         { name: coll.collectionName },
-        { ...this.options, readPreference: this.readPreference, session }
+        { ...this.options, nameOnly: false, readPreference: this.readPreference, session }
       )
       .toArray((err, collections) => {
         if (err || !collections) return callback(err);

diff --git a/src/operations/options_operation.ts b/src/operations/options_operation.ts
@@ -23,7 +23,7 @@ export class OptionsOperation extends AbstractOperation<Document> {
     coll.s.db
       .listCollections(
         { name: coll.collectionName },
-        { ...this.options, readPreference: this.readPreference, session }
+        { ...this.options, nameOnly: false, readPreference: this.readPreference, session }
       )
       .toArray((err, collections) => {
         if (err || !collections) return callback(err);

diff --git a/test/types/list_collections.test-d.ts b/test/types/list_collections.test-d.ts
@@ -5,6 +5,8 @@ import type { CollectionInfo, ListCollectionsCursor } from '../../src/operations
 
 const db = new MongoClient('').db();
 
+type EitherCollectionInfoResult = CollectionInfo | Pick<CollectionInfo, 'name' | 'type'>;
+
 // We default to the CollectionInfo result type
 expectType<ListCollectionsCursor<Pick<CollectionInfo, 'name' | 'type'> | CollectionInfo>>(
   db.listCollections()
@@ -19,7 +21,7 @@ db.listCollections({ a: 2 });
 db.listCollections({ a: 2 }, { batchSize: 2 });
 
 const collections = await db.listCollections().toArray();
-expectType<(CollectionInfo | string)[]>(collections);
+expectType<EitherCollectionInfoResult[]>(collections);
 
 const nameOnly = await db.listCollections({}, { nameOnly: true }).toArray();
 expectType<Pick<CollectionInfo, 'name' | 'type'>[]>(nameOnly);
@@ -28,7 +30,7 @@ const fullInfo = await db.listCollections({}, { nameOnly: false }).toArray();
 expectType<CollectionInfo[]>(fullInfo);
 
 const couldBeEither = await db.listCollections({}, { nameOnly: Math.random() > 0.5 }).toArray();
-expectType<(CollectionInfo | string)[]>(couldBeEither);
+expectType<EitherCollectionInfoResult[]>(couldBeEither);
 
 // Showing here that:
 // regardless of the option the generic parameter can be used to coerce the result if need be