fix: db delete key return value (#766)

web3-storage · Dec 9, 2021 · 34f5f82 · 34f5f82
1 parent 62db538
commit 34f5f82
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/packages/api/test/mocks/pgrest/patch_auth_key.json b/packages/api/test/mocks/pgrest/patch_auth_key.json
@@ -1,3 +1,3 @@
-{
-  "id": 1
-}
+[{
+  "id": "1"
+}]
diff --git a/packages/db/index.js b/packages/db/index.js
@@ -699,8 +699,12 @@ export class DBClient {
       throw new DBError(error)
     }
 
+    if (!data.length) {
+      return undefined
+    }
+
     return {
-      _id: data.id
+      _id: data[0].id
     }
   }
 

diff --git a/packages/db/test/user.spec.js b/packages/db/test/user.spec.js
@@ -127,7 +127,8 @@ describe('user operations', () => {
     const keys = await client.listKeys(user._id)
 
     // Delete previously created key
-    await client.deleteKey(user._id, authKey._id)
+    const { _id } = await client.deleteKey(user._id, authKey._id)
+    assert(_id, 'key deleted')
 
     const finalKeys = await client.listKeys(user._id)
     assert(finalKeys, 'final keys fetched')