chore: remove questionable test

cap-js · Oct 7, 2024 · b8ce92e · b8ce92e
1 parent 53a8a80
commit b8ce92e
Showing 1 changed file with 29 additions and 28 deletions.
diff --git a/test/tests/error-handling-dev.test.js b/test/tests/error-handling-dev.test.js
@@ -453,33 +453,34 @@ describe('graphql - error handling in development', () => {
       expect(response.data.errors[0].extensions.stacktrace).not.toHaveLength(0) // Stacktrace exists and is not empty
     })
 
-    test('req.reject with custom code', async () => {
-      const query = gql`
-        mutation {
-          CustomHandlerErrorsService {
-            Orders {
-              create(input: { id: 3, quantity: 20, stock: 10 }) {
-                id
-                quantity
-                stock
-              }
-            }
-          }
-        }
-      `
-      const errors = [
-        {
-          message: 'The order of NULL books exceeds the stock by NULL',
-          extensions: {
-            code: '500',
-            numericSeverity: 4,
-            stacktrace: expect.any(Array)
-          }
-        }
-      ]
-      const response = await POST('/graphql', { query })
-      expect(response.data).toMatchObject({ errors })
-      expect(response.data.errors[0].extensions.stacktrace).not.toHaveLength(0) // Stacktrace exists and is not empty
-    })
+    // NOTE: Never do such tests: it doesn't check expected behavior but just documents actual one which can change
+    // test('req.reject with custom code', async () => {
+    //   const query = gql`
+    //     mutation {
+    //       CustomHandlerErrorsService {
+    //         Orders {
+    //           create(input: { id: 3, quantity: 20, stock: 10 }) {
+    //             id
+    //             quantity
+    //             stock
+    //           }
+    //         }
+    //       }
+    //     }
+    //   `
+    //   const errors = [
+    //     {
+    //       message: 'The order of NULL books exceeds the stock by NULL',
+    //       extensions: {
+    //         code: '500',
+    //         numericSeverity: 4,
+    //         stacktrace: expect.any(Array)
+    //       }
+    //     }
+    //   ]
+    //   const response = await POST('/graphql', { query })
+    //   expect(response.data).toMatchObject({ errors })
+    //   expect(response.data.errors[0].extensions.stacktrace).not.toHaveLength(0) // Stacktrace exists and is not empty
+    // })
   })
 })