Skip to content

emrg: 启用 Windows CI 纯逻辑 TUI 测试（移除 65 个过时 skipif） #406

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
argszero merged 1 commit into from
Aug 5, 2026
Merged

emrg: 启用 Windows CI 纯逻辑 TUI 测试（移除 65 个过时 skipif） #406

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions tests/test_app_widgets.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,9 @@
import sys
import pytest

# TUI (python_tui) 依赖 POSIX-only fcntl/termios/tty——Windows 跳过（Windows 冒烟不跑 TUI）
pytestmark = pytest.mark.skipif(sys.platform == "win32", reason="TUI is POSIX-only (fcntl/termios)")
# R123 (#401) 后 Windows 已有原生 TUI（Win32Console），但 app.py 交互测试依赖
# 终端输入/信号行为（SIGWINCH/raw mode），Windows CI 冒烟阶段不可靠 → 仍跳过。
pytestmark = pytest.mark.skipif(sys.platform == "win32", reason="app 交互测试依赖 POSIX 终端行为（SIGWINCH/raw mode）")

from emrg.client.python_tui.widgets.base import Line, RenderContext, Span, Widget
from emrg.client.app import ProjectSelector, ModelSelector
Expand Down
6 changes: 0 additions & 6 deletions tests/test_buffer.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,6 @@

from __future__ import annotations

import sys
import pytest

# TUI (python_tui) 依赖 POSIX-only fcntl/termios/tty——Windows 跳过（Windows 冒烟不跑 TUI）
pytestmark = pytest.mark.skipif(sys.platform == "win32", reason="TUI is POSIX-only (fcntl/termios)")

from rich.style import Style

from emrg.client.python_tui.buffer import Buffer, Cell, CellWidth, write_lines_to_buffer
Expand Down
6 changes: 4 additions & 2 deletions tests/test_daemon_manager.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,10 @@

import sys

# TUI (python_tui) 依赖 POSIX-only fcntl/termios/tty——Windows 跳过（Windows 冒烟不跑 TUI）
pytestmark = pytest.mark.skipif(sys.platform == "win32", reason="TUI is POSIX-only (fcntl/termios)")
# R123 (#401) 后 Windows 已有原生 TUI，但 daemon_manager 测试覆盖 daemon 生命周期
# （spawn/信号/超时语义），Windows 上 CREATE_NEW_PROCESS_GROUP 等行为与 POSIX 不同
# → Windows CI 冒烟阶段仍跳过（纯逻辑测试见 test_buffer/test_output/test_input_parser）。
pytestmark = pytest.mark.skipif(sys.platform == "win32", reason="daemon 生命周期测试依赖 POSIX 进程语义（Windows CI 冒烟不跑）")

from emrg.client import daemon_manager

Expand Down
6 changes: 4 additions & 2 deletions tests/test_daemon_manager_e2e.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,10 @@

import sys

# TUI (python_tui) 依赖 POSIX-only fcntl/termios/tty——Windows 跳过（Windows 冒烟不跑 TUI）
pytestmark = pytest.mark.skipif(sys.platform == "win32", reason="TUI is POSIX-only (fcntl/termios)")
# R123 (#401) 后 Windows 已有原生 TUI，但 e2e 测试 spawn 真实 daemon + asyncio
# 事件循环行为（ProactorEventLoop 无 add_reader）与 POSIX 差异大 → Windows CI
# 冒烟阶段仍跳过（纯逻辑测试见 test_buffer/test_output/test_input_parser）。
pytestmark = pytest.mark.skipif(sys.platform == "win32", reason="e2e 测试依赖 POSIX daemon spawn/事件循环语义（Windows CI 冒烟不跑）")

from emrg.client import daemon_manager
from tests.test_ws_e2e import _boot_server, _make_config, _make_fake_chat_stream
Expand Down
6 changes: 0 additions & 6 deletions tests/test_input_parser.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,5 @@
"""Tests for InputParser — byte-stream → complete key sequences, including UTF-8."""

import pytest

import sys

# TUI (python_tui) 依赖 POSIX-only fcntl/termios/tty——Windows 跳过（Windows 冒烟不跑 TUI）
pytestmark = pytest.mark.skipif(sys.platform == "win32", reason="TUI is POSIX-only (fcntl/termios)")
from emrg.client.python_tui.events import InputParser, _utf8_len


Expand Down
6 changes: 0 additions & 6 deletions tests/test_output.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,6 @@

from __future__ import annotations

import sys
import pytest

# TUI (python_tui) 依赖 POSIX-only fcntl/termios/tty——Windows 跳过（Windows 冒烟不跑 TUI）
pytestmark = pytest.mark.skipif(sys.platform == "win32", reason="TUI is POSIX-only (fcntl/termios)")

from rich.style import Style

from emrg.client.python_tui.output import style_diff_sgr, style_to_sgr
Expand Down
Loading