Fix test timeouts

Author: RichDom2185

src/cse-machine/__tests__/cse-machine-errors.test.ts

@@ -2,8 +2,8 @@ import { expect, test } from 'vitest'
 import { Chapter, Variant } from '../../langs'
 import { stripIndent } from '../../utils/formatters'
 import {
-  expectParsedError,
   expectFinishedResult,
+  expectParsedError,
   testFailure,
   testSuccess
 } from '../../utils/testing'
@@ -168,17 +168,21 @@ test("Builtins don't create additional errors when it's not their fault", () =>
   ).toEqual('Line 2: Name a not declared.')
 })
 
-test('Infinite recursion with a block bodied function', { timeout: 15_000 }, () => {
-  return expectParsedError(
-    stripIndent`
+test(
+  'Infinite recursion with a block bodied function',
+  { timeout: process.env.GITHUB_ACTIONS ? 30_000 : 15_000 },
+  () => {
+    return expectParsedError(
+      stripIndent`
     function i(n) {
       return n === 0 ? 0 : 1 + i(n-1);
     }
     i(300000);
   `,
-    optionEC4
-  ).toEqual(expect.stringMatching(/Maximum call stack size exceeded\n *(i\(\d*\)[^i]{2,4}){3}/))
-})
+      optionEC4
+    ).toEqual(expect.stringMatching(/Maximum call stack size exceeded\n *(i\(\d*\)[^i]{2,4}){3}/))
+  }
+)
 
 test(
   'Infinite recursion with function calls in argument',

src/cse-machine/__tests__/cse-machine-unique-id.test.ts

@@ -1,9 +1,9 @@
 import { expect, test } from 'vitest'
-import { mockContext } from '../../utils/testing/mocks'
-import type { Context, Environment } from '../../types'
 import { Chapter } from '../../langs'
-import { stripIndent } from '../../utils/formatters'
 import { runCodeInSource } from '../../runner'
+import type { Context, Environment } from '../../types'
+import { stripIndent } from '../../utils/formatters'
+import { mockContext } from '../../utils/testing/mocks'
 import { createProgramEnvironment } from '../utils'
 
 const getContextFrom = async (code: string, envSteps?: number) => {
@@ -91,7 +91,7 @@ const getProgramEnv = (context: Context) => {
 
 test(
   'Program environment id stays the same regardless of amount of steps',
-  { timeout: 10_000 },
+  { timeout: process.env.GITHUB_ACTIOS ? 20_000 : 10_000 },
   async () => {
     const code = stripIndent`
         let x = 0;

src/runner/__tests__/execMethod.test.ts

@@ -1,14 +1,14 @@
 import { beforeEach, describe, expect, test, vi } from 'vitest'
-import runners, { type RunnerTypes } from '../sourceRunner'
-import type { ExecutionMethod } from '../../types'
-import { Chapter, Variant } from '../../langs'
-import type { Runner } from '../types'
 import { runCodeInSource } from '..'
-import { mockContext } from '../../utils/testing/mocks'
-import { getChapterName, objectKeys } from '../../utils/misc'
 import { parseError } from '../..'
-import * as validator from '../../validator/validator'
+import { Chapter, Variant } from '../../langs'
+import type { ExecutionMethod } from '../../types'
+import { getChapterName, objectKeys } from '../../utils/misc'
 import { wrapWithSkipAndOnly } from '../../utils/testing/misc'
+import { mockContext } from '../../utils/testing/mocks'
+import * as validator from '../../validator/validator'
+import runners, { type RunnerTypes } from '../sourceRunner'
+import type { Runner } from '../types'
 
 vi.spyOn(validator, 'validateAndAnnotate')
 
@@ -73,6 +73,7 @@ interface TestCase {
   expectedValidate: boolean
 
   verboseErrors?: boolean
+  timeout?: number
 }
 
 const sourceCases: TestCase[] = [
@@ -217,30 +218,37 @@ async function caseTester({
   }
 }
 
-const testCases = wrapWithSkipAndOnly('describe', function (desc: string, cases: TestCase[]) {
-  this(desc, () =>
-    test.each(
-      cases.map(({ code, verboseErrors, contextMethod, chapter, variant, ...tc }, i) => {
-        chapter = chapter ?? Chapter.SOURCE_1
-        variant = variant ?? Variant.DEFAULT
-        const context = mockContext(chapter, variant)
-        if (contextMethod !== undefined) {
-          context.executionMethod = contextMethod
-        }
-
-        const chapterName = getChapterName(chapter)
-        let desc = `${i + 1}. Testing ${chapterName}, Variant: ${variant}, expected ${tc.expectedRunner} runner`
-        code = code ?? ''
-        if (verboseErrors) {
-          code = `"enable verbose";\n${code}`
-          desc += ' (verbose errors)'
-        }
-
-        return [desc, { code, chapter, variant, ...tc }]
-      })
-    )('%s', async (_, to) => caseTester(to))
-  )
-})
+const testCases = wrapWithSkipAndOnly(
+  'describe',
+  function (desc: string, cases: TestCase[], timeout?: number) {
+    this(desc, () => {
+      const testEach = test.each(
+        cases.map(({ code, verboseErrors, contextMethod, chapter, variant, ...tc }, i) => {
+          chapter = chapter ?? Chapter.SOURCE_1
+          variant = variant ?? Variant.DEFAULT
+          const context = mockContext(chapter, variant)
+          if (contextMethod !== undefined) {
+            context.executionMethod = contextMethod
+          }
+
+          const chapterName = getChapterName(chapter)
+          let desc = `${i + 1}. Testing ${chapterName}, Variant: ${variant}, expected ${tc.expectedRunner} runner`
+          code = code ?? ''
+          if (verboseErrors) {
+            code = `"enable verbose";\n${code}`
+            desc += ' (verbose errors)'
+          }
+
+          return [desc, { code, chapter, variant, ...tc }]
+        })
+      )
+      if (timeout !== undefined) {
+        return testEach('%s', { timeout }, async (_, to) => caseTester(to))
+      }
+      return testEach('%s', async (_, to) => caseTester(to))
+    })
+  }
+)
 
 describe('Ensure that the correct runner is used for the given evaluation context and settings', () => {
   testCases('Test regular source cases', sourceCases)
@@ -300,7 +308,8 @@ describe('Ensure that the correct runner is used for the given evaluation contex
       )
 
       return [fullCase, verboseErrorCase, ...variantCases]
-    })
+    }),
+    10_000
   )
 
   testCases(