test: update unit tests to compare to original order of definitions (#…

…584) Fixes #583
climateinteractive · Dec 8, 2024 · 5549c2a · 5549c2a
1 parent 013bbf7
commit 5549c2a
Show file tree

Hide file tree

Showing 5 changed files with 3,332 additions and 2,663 deletions.
diff --git a/packages/compile/package.json b/packages/compile/package.json
@@ -11,7 +11,7 @@
     "precommit": "../../scripts/precommit",
     "type-check": "tsc --noEmit -p tsconfig-test.json",
     "test": "vitest run",
-    "test:watch": "vitest",
+    "test:watch": "vitest --hideSkippedTests",
     "test:ci": "vitest run",
     "ci:build": "run-s lint prettier:check type-check test:ci"
   },

diff --git a/packages/compile/src/_tests/test-support.ts b/packages/compile/src/_tests/test-support.ts
@@ -154,13 +154,7 @@ export function parseVensimModel(modelName: string): ParsedModel {
   const modelDir = sampleModelDir(modelName)
   const modelFile = resolve(modelDir, `${modelName}.mdl`)
   const mdlContent = readFileSync(modelFile, 'utf8')
-
-  // TODO: We currently sort the preprocessed definitions alphabetically for
-  // compatibility with tests that were written back when we used the legacy
-  // preprocessor, which sorted definitions alphabetically.  Now that we no
-  // longer rely on the legacy preprocessor, we should update the tests to
-  // use the original order and remove this option.
-  return parseModel(mdlContent, modelDir, { sort: true })
+  return parseModel(mdlContent, modelDir)
 }
 
 export function parseInlineVensimModel(mdlContent: string, modelDir?: string): ParsedModel {