test(indexDocuments): test and fix impl #35

hyper63 · Jun 14, 2023 · b9658b8 · b9658b8
1 parent a5d7a2f
commit b9658b8
Show file tree

Hide file tree

Showing 3 changed files with 67 additions and 3 deletions.
diff --git a/adapter.ts b/adapter.ts
@@ -293,7 +293,7 @@ export function adapter({
     name: string
     fields: string[] | { [field: string]: 'ASC' | 'DESC' }[]
   }) {
-    meta
+    return meta
       .get(db)
       .chain(() =>
         $database(db)

diff --git a/test/suite.ts b/test/suite.ts
@@ -389,11 +389,69 @@ async (
   })
 
   await t.step('indexDocuments', async (t) => {
-    await t.step('should create the index', async () => {})
+    await t.step('should create the index', async () => {
+      await a.createDatabase(DB)
+
+      await a.createDocument({
+        db: DB,
+        id: '10-caddyshack',
+        doc: { title: 'Caddyshack', year: '1978', genre: ['comedy'] },
+      })
+
+      await a.createDocument({
+        db: DB,
+        id: '12-ghostbusters',
+        doc: { title: 'Ghostbusters', year: '1980', genre: ['comedy'] },
+      })
+
+      const res = await a.indexDocuments({
+        db: DB,
+        name: 'idx-title-year',
+        fields: ['title', 'year'],
+      })
+
+      assertObjectMatch(res as any, { ok: true })
+
+      await a.removeDatabase(DB)
+    })
+
+    await t.step('should create the index with obj fields', async () => {
+      await a.createDatabase(DB)
+
+      await a.createDocument({
+        db: DB,
+        id: '10-caddyshack',
+        doc: { title: 'Caddyshack', year: '1978', genre: ['comedy'] },
+      })
+
+      await a.createDocument({
+        db: DB,
+        id: '12-ghostbusters',
+        doc: { title: 'Ghostbusters', year: '1980', genre: ['comedy'] },
+      })
+
+      const res = await a.indexDocuments({
+        db: DB,
+        name: 'idx-title-year',
+        fields: [{ title: 'DESC' }, { year: 'ASC' }],
+      })
+
+      assertObjectMatch(res as any, { ok: true })
+
+      await a.removeDatabase(DB)
+    })
 
     await t.step(
       'should return a HyperErr(404) if the database does not exist',
-      async () => {},
+      async () => {
+        const res = await a.indexDocuments({
+          db: DB,
+          name: 'idx-title-year',
+          fields: [{ title: 'DESC' }, { year: 'ASC' }],
+        })
+
+        assertObjectMatch(res as any, { ok: false, status: 404 })
+      },
     )
   })
 

diff --git a/utils.ts b/utils.ts
@@ -34,6 +34,12 @@ export const queryOptions = ({
   fields?: string[]
   sort?: string[] | { [field: string]: 'ASC' | 'DESC' }[]
 }) => {
+  /**
+   * Notice use_index is not mapped here, as MongoDB
+   * internally chooses an index to use.
+   *
+   * So use_index is effectively ignored
+   */
   const options: {
     limit?: number
     projection?: { [field: string]: 0 | 1 }