test(adapter-tests): NotFound integer & add remove:NotFound tests (#3486

)
feathersjs · May 30, 2024 · a02febb · a02febb
1 parent c7b0111
commit a02febb
Show file tree

Hide file tree

Showing 2 changed files with 54 additions and 0 deletions.
diff --git a/packages/adapter-tests/src/declarations.ts b/packages/adapter-tests/src/declarations.ts
@@ -28,18 +28,22 @@ export type AdapterMethodsTestName =
   | '.get + $select'
   | '.get + id + query'
   | '.get + NotFound'
+  | '.get + NotFound (integer)'
   | '.get + id + query id'
   | '.find'
   | '.remove'
   | '.remove + $select'
   | '.remove + id + query'
+  | '.remove + NotFound'
+  | '.remove + NotFound (integer)'
   | '.remove + multi'
   | '.remove + multi no pagination'
   | '.remove + id + query id'
   | '.update'
   | '.update + $select'
   | '.update + id + query'
   | '.update + NotFound'
+  | '.update + NotFound (integer)'
   | '.update + query + NotFound'
   | '.update + id + query id'
   | '.patch'
@@ -50,6 +54,7 @@ export type AdapterMethodsTestName =
   | '.patch multi query same'
   | '.patch multi query changed'
   | '.patch + NotFound'
+  | '.patch + NotFound (integer)'
   | '.patch + query + NotFound'
   | '.patch + id + query id'
   | '.create'

diff --git a/packages/adapter-tests/src/methods.ts b/packages/adapter-tests/src/methods.ts
@@ -59,6 +59,15 @@ export default (test: AdapterMethodsTest, app: any, _errors: any, serviceName: s
         }
       })
 
+      test('.get + NotFound (integer)', async () => {
+        try {
+          await service.get(123456789)
+          throw new Error('Should never get here')
+        } catch (error: any) {
+          assert.strictEqual(error.name, 'NotFound', 'Error is a NotFound Feathers error')
+        }
+      })
+
       test('.get + id + query id', async () => {
         const alice = await service.create({
           name: 'Alice',
@@ -115,6 +124,24 @@ export default (test: AdapterMethodsTest, app: any, _errors: any, serviceName: s
         }
       })
 
+      test('.remove + NotFound', async () => {
+        try {
+          await service.remove('568225fbfe21222432e836ff')
+          throw new Error('Should never get here')
+        } catch (error: any) {
+          assert.strictEqual(error.name, 'NotFound', 'Error is a NotFound Feathers error')
+        }
+      })
+
+      test('.remove + NotFound (integer)', async () => {
+        try {
+          await service.remove(123456789)
+          throw new Error('Should never get here')
+        } catch (error: any) {
+          assert.strictEqual(error.name, 'NotFound', 'Error is a NotFound Feathers error')
+        }
+      })
+
       test('.remove + multi', async () => {
         try {
           await service.remove(null)
@@ -278,6 +305,17 @@ export default (test: AdapterMethodsTest, app: any, _errors: any, serviceName: s
         }
       })
 
+      test('.update + NotFound (integer)', async () => {
+        try {
+          await service.update(123456789, {
+            name: 'NotFound'
+          })
+          throw new Error('Should never get here')
+        } catch (error: any) {
+          assert.strictEqual(error.name, 'NotFound', 'Error is a NotFound Feathers error')
+        }
+      })
+
       test('.update + query + NotFound', async () => {
         const dave = await service.create({ name: 'Dave' })
         try {
@@ -541,6 +579,17 @@ export default (test: AdapterMethodsTest, app: any, _errors: any, serviceName: s
         }
       })
 
+      test('.patch + NotFound (integer)', async () => {
+        try {
+          await service.patch(123456789, {
+            name: 'PatchDoug'
+          })
+          throw new Error('Should never get here')
+        } catch (error: any) {
+          assert.strictEqual(error.name, 'NotFound', 'Error is a NotFound Feathers error')
+        }
+      })
+
       test('.patch + query + NotFound', async () => {
         const dave = await service.create({ name: 'Dave' })
         try {