Added comment for mock onError

Added check for arg getting used

Author: Druue

packages/language-server/src/__test__/artificial-panic.test.ts

@@ -39,16 +39,21 @@ suite('Artificial Panics', () => {
     process.env.FORCE_PANIC_PRISMA_FMT = '1'
 
     let calledCount = 0
+    let calledArg: undefined | unknown = undefined
 
-    const onError = () => {
+    // No official mock implementation in mocha
+    // -> DIY mock for onError
+    const onError = (arg: unknown) => {
       calledCount += 1
+      calledArg = arg
     }
 
     try {
       const _codeActions = handleCodeActions(params, document, onError)
 
       assert.fail("This shouldn't happen!")
     } catch (e) {
+      assert.ok(calledArg)
       assert.ok(calledCount == 1)
     } finally {
       process.env = { ...OLD_ENV }
@@ -70,16 +75,19 @@ suite('Artificial Panics', () => {
     process.env.FORCE_PANIC_PRISMA_FMT = '1'
 
     let calledCount = 0
+    let calledArg: undefined | unknown = undefined
 
-    const onError = () => {
+    const onError = (arg: unknown) => {
       calledCount += 1
+      calledArg = arg
     }
 
     try {
       const _formatResult: TextEdit[] = handleDocumentFormatting(params, document, onError)
 
       assert.fail("This shouldn't happen!")
     } catch (e) {
+      assert.ok(calledArg)
       assert.ok(calledCount == 1)
     } finally {
       process.env = { ...OLD_ENV }
@@ -93,16 +101,19 @@ suite('Artificial Panics', () => {
     process.env.FORCE_PANIC_PRISMA_FMT = '1'
 
     let calledCount = 0
+    let calledArg: undefined | unknown = undefined
 
-    const onError = () => {
+    const onError = (arg: unknown) => {
       calledCount += 1
+      calledArg = arg
     }
 
     try {
       const _diagnostics = handleDiagnosticsRequest(document, onError)
 
       assert.fail("This shouldn't happen!")
     } catch (e) {
+      assert.ok(calledArg)
       assert.ok(calledCount == 1)
     } finally {
       process.env = { ...OLD_ENV }
@@ -132,16 +143,19 @@ suite('Artificial Panics', () => {
     process.env.FORCE_PANIC_PRISMA_FMT_LOCAL = '1'
 
     let calledCount = 0
+    let calledArg: undefined | unknown = undefined
 
-    const onError = () => {
+    const onError = (arg: unknown) => {
       calledCount += 1
+      calledArg = arg
     }
 
     try {
       const _completions = handleCompletionRequest(params, document, onError)
 
       assert.fail("This shouldn't happen!")
     } catch (e) {
+      assert.ok(calledArg)
       assert.ok(calledCount == 1)
     } finally {
       process.env = { ...OLD_ENV }
@@ -171,16 +185,19 @@ suite('Artificial Panics', () => {
     process.env.FORCE_PANIC_PRISMA_FMT_LOCAL = '1'
 
     let calledCount = 0
+    let calledArg: undefined | unknown = undefined
 
-    const onError = () => {
+    const onError = (arg: unknown) => {
       calledCount += 1
+      calledArg = arg
     }
 
     try {
       const _completions = handleCompletionRequest(params, document, onError)
 
       assert.fail("This shouldn't happen!")
     } catch (e) {
+      assert.ok(calledArg)
       assert.ok(calledCount == 1)
     } finally {
       process.env = { ...OLD_ENV }
@@ -199,16 +216,19 @@ suite('Artificial Panics', () => {
     process.env.FORCE_PANIC_PRISMA_FMT = '1'
 
     let calledCount = 0
+    let calledArg: undefined | unknown = undefined
 
-    const onError = () => {
+    const onError = (arg: unknown) => {
       calledCount += 1
+      calledArg = arg
     }
 
     try {
       const _completions = handleCompletionRequest(params, document, onError)
 
       assert.fail("This shouldn't happen!")
     } catch (e) {
+      assert.ok(calledArg)
       assert.ok(calledCount == 1)
     } finally {
       process.env = { ...OLD_ENV }