diff options
| author | Christophe Besson <cbesson@gmail.com> | 2026-10-15 17:42:30 +0200 |
|---|---|---|
| committer | Christophe Besson <cbesson@gmail.com> | 2026-10-15 17:45:41 +0200 |
| commit | 41d03f0a35a95daa7d54e7f29ab924776bedf6e4 (patch) | |
| tree | 2e631013746f913413b9811562b9c9a4734a0e41 /packages/meshbay-cli/tests/console.py | |
| parent | 39cac2878de059992cb09a1ccf09b6bcece99fae (diff) | |
| download | meshbay-41d03f0a35a95daa7d54e7f29ab924776bedf6e4.tar.gz | |
fix(cli): make the command-line client work on Windows
Its suite had only run on Linux. On Windows, against a real hub and node and
from the installed meshbay-cli.exe:
- secrets.enc was written in text mode (os.open without O_BINARY): every 0x0A
of the ciphertext became CR LF and the saved session "does not open with
its key" on the next run.
- isatty() is true for NUL, so a script run with `< NUL` or by the Task
Scheduler was asked questions instead of told to pass --yes;
output.terminal() checks for a real console.
- Ctrl+C waited for the selector loop's next timer, up to half a minute in
`chat --follow`; the loop now points the signal wakeup at its self-pipe,
as the proactor does.
- Right-click copy and paste raised: asyncio cannot start a process on the
selector loop; the clipboard tool runs in a thread.
- Tab completion of remote paths ran on the default (proactor) loop, which
never opens a node, and wrote in the locale encoding; the PowerShell script
now reads it as UTF-8.
- `get -R` into a folder past MAX_PATH died with a traceback; get, put, the
resume file and the interactive mode use the extended form at each disk
call, as the node does.
- `groups` showed a SQLite hub's naive UTC time as local time.
The opt-in end-to-end suite now runs on Windows (16 tests: Chrome found under
Program Files and started with the real environment, VLC as the player, Ctrl+C
through a console of its own, PowerShell 5.1 completion of an accented name).
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Diffstat (limited to 'packages/meshbay-cli/tests/console.py')
| -rw-r--r-- | packages/meshbay-cli/tests/console.py | 48 |
1 files changed, 48 insertions, 0 deletions
diff --git a/packages/meshbay-cli/tests/console.py b/packages/meshbay-cli/tests/console.py new file mode 100644 index 0000000..d3ea6c9 --- /dev/null +++ b/packages/meshbay-cli/tests/console.py @@ -0,0 +1,48 @@ +""" +A person's Ctrl+C, delivered to a client run started by a test. + +On Windows Ctrl+C is a console event, not a signal, so the run gets a hidden +console of its own and a helper attached to that console raises the event. The +run goes through a launcher that turns Ctrl+C handling back on first: a process +inherits "ignore Ctrl+C" from its parent, and whatever started the test suite +may have set it (an IDE, a CI runner, an agent's shell) where a person's console +never does — measured, the event then reached nothing at all. +""" + +import subprocess +import sys + +_LAUNCHER = ("import ctypes, signal, subprocess, sys\n" + "ctypes.windll.kernel32.SetConsoleCtrlHandler(None, 0)\n" + "signal.signal(signal.SIGINT, signal.SIG_IGN)\n" + "sys.exit(subprocess.call(sys.argv[1:]))\n") + +# os._exit: with the console swapped, flushing stdout at exit fails (code 120). +_SENDER = ("import ctypes, os, sys\n" + "k = ctypes.windll.kernel32\n" + "k.FreeConsole()\n" + "ok = k.AttachConsole(int(sys.argv[1])) and k.SetConsoleCtrlHandler(None, 1)\n" + "os._exit(0 if ok and k.GenerateConsoleCtrlEvent(0, 0) else 1)\n") + + +def interruptible(argv: list[str], env: dict | None) -> subprocess.Popen: + """`argv` started so that `ctrl_c` reaches it, its output read as UTF-8.""" + extra: dict = {} + if sys.platform == "win32": + si = subprocess.STARTUPINFO() + si.dwFlags |= subprocess.STARTF_USESHOWWINDOW + si.wShowWindow = 0 # SW_HIDE + extra = {"creationflags": subprocess.CREATE_NEW_CONSOLE, "startupinfo": si} + argv = [sys.executable, "-c", _LAUNCHER, *argv] + return subprocess.Popen(argv, env=env, stdout=subprocess.PIPE, stderr=subprocess.PIPE, + text=True, encoding="utf-8", **extra) + + +def ctrl_c(proc: subprocess.Popen) -> None: + if sys.platform != "win32": + import signal + proc.send_signal(signal.SIGINT) + return + subprocess.run([sys.executable, "-c", _SENDER, str(proc.pid)], check=True, timeout=30, + stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL, + creationflags=subprocess.CREATE_NO_WINDOW) |