Clean up overlays and remove obsolete dependencies

2026-09-19 21:13:08 -04:00
parent b1f6de04c2
commit fc91e226e6
15 changed files with 435 additions and 149 deletions
@@ -0,0 +1,36 @@
+Subject: [PATCH] tests: finish parsing before inspecting editor state
+
+EditorState creation has a 20 ms parsing budget. A descheduled test can
+therefore observe an incomplete tree. Finish these small test documents
+without an interactive deadline and publish the result with a transaction.
+Keep the original completion and vector-matching assertions enabled.
+
+--- a/module/codemirror-promql/src/test/utils-test.ts
+++ b/module/codemirror-promql/src/test/utils-test.ts
+@@ -13,7 +13,7 @@
+ 
+ import { parser } from '@prometheus-io/lezer-promql';
+ import { EditorState } from '@codemirror/state';
+-import { LRLanguage } from '@codemirror/language';
+import { ensureSyntaxTree, LRLanguage } from '@codemirror/language';
+ import nock from 'nock';
+ import path from 'path';
+ import { fileURLToPath } from 'url';
+@@ -23,10 +23,16 @@
+ const __dirname = path.dirname(fileURLToPath(import.meta.url));
+ 
+ export function createEditorState(expr: string): EditorState {
+-  return EditorState.create({
+  const state = EditorState.create({
+     doc: expr,
+     extensions: lightPromQLSyntax,
+   });
+  // These tests need a complete tree, independent of the editor's time budget.
+  if (!ensureSyntaxTree(state, state.doc.length, Infinity)) {
+    throw new Error('Unable to parse the test expression');
+  }
+  // Publish the completed parse so syntaxTree(state) sees it too.
+  return state.update({}).state;
+ }
+ 
+ export function mockPrometheusServer(): void {