Skip to content

Commit 78dddea

Browse files
authored
Merge pull request #1920 from dbcli/RW/skip-flaky-external-editor-test
Skip flaky external-editor test in CI
2 parents a166fcf + d1223b1 commit 78dddea

4 files changed

Lines changed: 9 additions & 2 deletions

File tree

changelog.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,13 @@ Features
55
--------
66
* Detect Apache Doris and display its real version instead of the MySQL-compat placeholder.
77

8+
89
Bug Fixes
910
---------
1011
* Keep completion-menu Escape cancellation eager only in Vi mode so Emacs Alt-key bindings keep working while completions are open.
1112
* Keep identifiers that start with `set` highlighted as names.
1213
* Fix version display for MySQL distributions that return a plain `X.Y.Z` version string with no suffix (e.g. Homebrew MySQL).
14+
* Speed up startup by skipping LLM imports when `MYCLI_LLM_OFF` is set.
1315

1416

1517
Internal
@@ -20,7 +22,7 @@ Internal
2022
* Add CI on macOS.
2123
* Add limited CI on Windows.
2224
* Add limited CI on Windows WSL.
23-
* Speed up startup by skipping LLM imports when `MYCLI_LLM_OFF` is set.
25+
* Skip flaky external-editor test in CI.
2426

2527

2628
1.73.1 (2026/05/29)

pyproject.toml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,8 @@ passenv = ['PYTEST_HOST',
136136
'PYTEST_USER',
137137
'PYTEST_PASSWORD',
138138
'PYTEST_PORT',
139-
'PYTEST_CHARSET']
139+
'PYTEST_CHARSET',
140+
'GITHUB_ACTION']
140141
commands = [['uv', 'pip', 'install', '-e', '.[dev,ssh,llm]'],
141142
['coverage', 'run', '-m', 'pytest', '-v', 'test'],
142143
['coverage', 'report', '-m', '--sort=Miss'],

test/features/environment.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,9 @@ def before_scenario(context, arg):
105105
# Skip scenarios marked skip_py312 when running on Python 3.12
106106
if sys.version_info[:2] == (3, 12) and "skip_py312" in arg.tags:
107107
arg.skip("Skipped on Python 3.12")
108+
# Skip flaky editor test in CI
109+
if os.getenv('GITHUB_ACTION') and 'skip_ci' in arg.tags:
110+
arg.skip('Skipped in CI')
108111
with open(test_log_file, "w") as f:
109112
f.write("")
110113
if arg.location.filename not in SELF_CONNECTING_FEATURES:

test/features/iocommands.feature

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
Feature: I/O commands
22

3+
@skip_ci
34
Scenario: edit sql in file with external editor
45
When we start external editor providing a file name
56
and we type "select * from abc" in the editor

0 commit comments

Comments
 (0)