Skip to content

Commit 1ac4aaa

Browse files
wmaddenclaude
andcommitted
TML-2840: match extractDb error message to its condition
The guard is `!isMongoControlDriver(driver)` — a tag check — but the old message said "does not expose a db property", describing the property-presence check it replaced. Updated the message and the matching test assertion to reflect what the predicate actually checks. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: Will Madden <madden@prisma.io>
1 parent 640955a commit 1ac4aaa

2 files changed

Lines changed: 4 additions & 6 deletions

File tree

packages/3-mongo-target/2-mongo-adapter/src/core/runner-deps.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,8 @@ import { isMongoControlDriver } from './mongo-control-driver';
1919
export function extractDb(driver: ControlDriverInstance<'mongo', 'mongo'>): Db {
2020
if (!isMongoControlDriver(driver)) {
2121
throw new Error(
22-
'Mongo control driver does not expose a db property. ' +
23-
'Use mongoControlDriver.create() from `@prisma-next/driver-mongo/control`.',
22+
'Expected a Mongo control driver created by ' +
23+
'mongoControlDriver.create() from `@prisma-next/driver-mongo/control`.',
2424
);
2525
}
2626
return driver.db;

packages/3-mongo-target/2-mongo-adapter/test/runner-deps.test.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,8 @@ describe('extractDb', () => {
1717
expect(extractDb(driver)).toBe(fakeDb);
1818
});
1919

20-
it("throws when the mongo control driver doesn't expose a db property", () => {
20+
it('throws when the value is not a Mongo control driver', () => {
2121
const driver = {} as unknown as ControlDriverInstance<'mongo', 'mongo'>;
22-
expect(() => extractDb(driver)).toThrowError(
23-
/Mongo control driver does not expose a db property/,
24-
);
22+
expect(() => extractDb(driver)).toThrowError(/Expected a Mongo control driver/);
2523
});
2624
});

0 commit comments

Comments
 (0)