Fix broken cse-machine tests

Author: leeyi45

eslint.config.mjs

@@ -32,6 +32,7 @@ export default tseslint.config(
       'import/order': 'warn',
       '@typescript-eslint/no-base-to-string': 'off', // TODO: Remove
       'prefer-const': 'off', // TODO: Remove
+      'no-constant-condition': ['warn', { checkLoops: false }],
       'no-var': 'off', // TODO: Remove
       '@typescript-eslint/ban-ts-comment': 'warn',
       '@typescript-eslint/ban-types': 'off',

src/runner/__tests__/execMethod.ts

@@ -1,7 +1,7 @@
 import runners, { type RunnerTypes } from '../sourceRunner'
 import { Chapter, type ExecutionMethod, Variant } from '../../types'
 import type { Runner } from '../types'
-import { DEFAULT_SOURCE_OPTIONS, runCodeInSource } from '..'
+import { runCodeInSource } from '..'
 import { mockContext } from '../../utils/testing/mocks'
 import { getChapterName, objectKeys, objectValues } from '../../utils/misc'
 import { asMockedFunc } from '../../utils/testing/misc'
@@ -90,7 +90,7 @@ const sourceCases: FullTestCase[] = [
   {
     contextMethod: 'native',
     variant: Variant.NATIVE,
-    expectedRunner: 'native',
+    expectedRunner: 'fulljs',
     expectedPrelude: false
   }
 ]
@@ -155,11 +155,12 @@ async function testCase({
   // Check if the prelude is null before execution
   // because the prelude gets set to null if it wasn't before
   const shouldPrelude = expectedPrelude && context.prelude !== null
-  const options = { ...DEFAULT_SOURCE_OPTIONS }
-
-  if (optionMethod !== undefined) {
-    options.executionMethod = optionMethod
-  }
+  const options =
+    optionMethod === undefined
+      ? undefined
+      : {
+          executionMethod: optionMethod
+        }
 
   await runCodeInSource(code, context, options)
 
@@ -287,6 +288,15 @@ describe('Ensure that the correct runner is used for the given evaluation contex
       expectedRunner: 'native'
     }))
 
+  // testCases('runner correctly respects optionMethod', objectKeys(runners).map(runner => ({
+  //   code: '"enable verbose"; 0;',
+  //   optionMethod: runner,
+  //   chapter: Chapter.SOURCE_4,
+  //   variant: Variant.DEFAULT,
+  //   expectedPrelude: true,
+  //   expectedRunner: runner
+  // })))
+
   test('if optionMethod is specified, debubger statements are ignored', () =>
     testCase({
       code: 'debugger; 0;',

src/utils/ast/helpers.ts

@@ -83,9 +83,11 @@ export function getSourceVariableDeclaration(decl: es.VariableDeclaration) {
     'Variable Declarations in Source should be declared using an Identifier!'
   )
 
+  assert(!!declaration.init, 'Variable declarations in Source must be initialized!')
+
   return {
     id: declaration.id,
-    init: declaration.init!,
+    init: declaration.init,
     loc: declaration.loc
   }
 }