Merge pull request #1385 from dbcli/RW/double-editor-command-test-timeouts

Double editor-command test timeouts to 4 seconds

Author: rolandwalker

changelog.md

@@ -5,6 +5,7 @@ Internal
 --------
 * Add mypy to Pull Request template.
 * Enable flake8-bugbear lint rules.
+* Fix flaky editor-command tests in CI.
 
 
 1.40.0 (2025/10/14)

test/features/steps/iocommands.py

@@ -14,22 +14,22 @@ def step_edit_file(context):
     if os.path.exists(context.editor_file_name):
         os.remove(context.editor_file_name)
     context.cli.sendline(f"\\e {os.path.basename(context.editor_file_name)}")
-    wrappers.expect_exact(context, 'Entering Ex mode.  Type "visual" to go to Normal mode.', timeout=2)
-    wrappers.expect_exact(context, "\r\n:", timeout=2)
+    wrappers.expect_exact(context, 'Entering Ex mode.  Type "visual" to go to Normal mode.', timeout=4)
+    wrappers.expect_exact(context, "\r\n:", timeout=4)
 
 
 @when('we type "{query}" in the editor')
 def step_edit_type_sql(context, query):
     context.cli.sendline("i")
     context.cli.sendline(query)
     context.cli.sendline(".")
-    wrappers.expect_exact(context, "\r\n:", timeout=2)
+    wrappers.expect_exact(context, "\r\n:", timeout=4)
 
 
 @when("we exit the editor")
 def step_edit_quit(context):
     context.cli.sendline("x")
-    wrappers.expect_exact(context, "written", timeout=2)
+    wrappers.expect_exact(context, "written", timeout=4)
 
 
 @then('we see "{query}" in prompt')