Skip to content

Commit cc16e89

Browse files
committed
pull up error from executeLegacyOperation
1 parent 51929a5 commit cc16e89

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

src/gridfs-stream/index.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { MongoError } from '../error';
1+
import { MongoClientClosedError, MongoError } from '../error';
22
import { EventEmitter } from 'events';
33
import {
44
GridFSBucketReadStream,
@@ -134,6 +134,7 @@ export class GridFSBucket extends EventEmitter {
134134
delete(id: TFileId): Promise<undefined>;
135135
delete(id: TFileId, callback: Callback<void>): void;
136136
delete(id: TFileId, callback?: Callback<void>): Promise<undefined> | void {
137+
if (!this.s.db.topology) throw new MongoClientClosedError();
137138
return executeLegacyOperation(this.s.db.topology, _delete, [this, id, callback], {
138139
skipSessions: true
139140
});
@@ -185,6 +186,7 @@ export class GridFSBucket extends EventEmitter {
185186
rename(id: TFileId, filename: string): Promise<void>;
186187
rename(id: TFileId, filename: string, callback: Callback<void>): void;
187188
rename(id: TFileId, filename: string, callback?: Callback<void>): Promise<void> | void {
189+
if (!this.s.db.topology) throw new MongoClientClosedError();
188190
return executeLegacyOperation(this.s.db.topology, _rename, [this, id, filename, callback], {
189191
skipSessions: true
190192
});
@@ -194,6 +196,7 @@ export class GridFSBucket extends EventEmitter {
194196
drop(): Promise<void>;
195197
drop(callback: Callback<void>): void;
196198
drop(callback?: Callback<void>): Promise<void> | void {
199+
if (!this.s.db.topology) throw new MongoClientClosedError();
197200
return executeLegacyOperation(this.s.db.topology, _drop, [this, callback], {
198201
skipSessions: true
199202
});

src/utils.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -263,7 +263,7 @@ export function mergeOptionsAndWriteConcern(
263263
* @param options - Options that modify the behavior of the method
264264
*/
265265
export function executeLegacyOperation<T extends OperationBase>(
266-
topology: Topology | undefined,
266+
topology: Topology,
267267
operation: (...args: any[]) => void | Promise<Document>,
268268
args: any[],
269269
options?: AnyOptions

0 commit comments

Comments
 (0)