deflake

Author: AaronPlave

e2e-tests/fixtures/Workspace.ts

@@ -155,8 +155,11 @@ export class Workspace {
   }
 
   async fillSequenceContent(content: string): Promise<void> {
-    await this.sequenceEditor.click();
-    await this.sequenceEditor.fill(content);
+    // Target the contenteditable host (.cm-content) rather than a line div (.cm-activeLine):
+    // CodeMirror remounts when navigating between files, and a line div can briefly resolve
+    // before its contenteditable ancestor is ready, making fill() throw "not editable".
+    await this.sequenceEditorContent.click();
+    await this.sequenceEditorContent.fill(content);
   }
 
   private async fillSequenceName(name: string, path?: string): Promise<void> {
@@ -222,7 +225,14 @@ export class Workspace {
     }
     const moreActionsButton = row.getByLabel('More actions');
     await hoverRowAndWaitForButton(this.page, row, moreActionsButton);
-    await moreActionsButton.click();
+    // Open the menu via a DOM click event instead of a synthesized pointer click. The button
+    // sits at the row's right edge under AG Grid's invisible vertical scrollbar overlay and
+    // is only revealed on hover, so a real click is fragile: its actionability scroll summons
+    // the overlay (which intercepts the click) and moves the row off the cursor (hiding the
+    // button, so the click retries forever). The handler positions the menu from the button's
+    // own getBoundingClientRect (event.currentTarget), so dispatching 'click' opens the menu
+    // correctly without depending on pointer coordinates, scroll position, or hover.
+    await moreActionsButton.dispatchEvent('click');
     await this.workspaceFileContextMenu.waitFor({ state: 'visible' });
   }