fix(schema): Check for undefined value in resolveQueryObjectId resolver (#2914)

philipimperato · web-flow · commit d9449fa835f5 · 2022-12-11T15:44:27.000-08:00
diff --git a/packages/mongodb/src/resolvers.ts b/packages/mongodb/src/resolvers.ts
@@ -19,6 +19,10 @@ export async function resolveQueryObjectId(
 ): Promise<IdQueryObject<ObjectId>>
 export async function resolveQueryObjectId(value: ObjectIdParam): Promise<ObjectId>
 export async function resolveQueryObjectId(value: ObjectIdParam | IdQueryObject<ObjectIdParam>) {
+  if (!value) {
+    return undefined
+  }
+
   if (typeof value === 'string' || typeof value === 'number' || value instanceof ObjectId) {
     return toObjectId(value)
   }
diff --git a/packages/mongodb/test/resolvers.test.ts b/packages/mongodb/test/resolvers.test.ts
@@ -24,4 +24,12 @@ describe('ObjectId resolvers', () => {
     assert.ok(oids.$in && oids.$in[0] instanceof ObjectId)
     assert.ok(oids.$ne instanceof ObjectId)
   })
+
+  it('resolveQueryObjectId with falsey value', async () => {
+    await resolveQueryObjectId(undefined)
+    await resolveQueryObjectId(null)
+    await resolveQueryObjectId(0)
+
+    assert.ok('Falsey value does not throw exception')
+  })
 })
