Bump minor version and update ghostty-web
This commit is contained in:
GitHub Copilot
+62
-127
@@ -1,85 +1,8 @@
|
||||
"""Tests for CLI module."""
|
||||
|
||||
from pathlib import Path
|
||||
|
||||
import click
|
||||
import pytest
|
||||
from click.testing import CliRunner
|
||||
|
||||
|
||||
class TestParseAppPath:
|
||||
"""Tests for parse_app_path function."""
|
||||
|
||||
def test_parse_module_class(self):
|
||||
"""Test parsing module:class format."""
|
||||
from textual_webterm.cli import parse_app_path
|
||||
|
||||
module, cls = parse_app_path("mymodule:MyClass")
|
||||
assert module == "mymodule"
|
||||
assert cls == "MyClass"
|
||||
|
||||
def test_parse_nested_module_class(self):
|
||||
"""Test parsing nested.module:class format."""
|
||||
from textual_webterm.cli import parse_app_path
|
||||
|
||||
module, cls = parse_app_path("my.nested.module:MyClass")
|
||||
assert module == "my.nested.module"
|
||||
assert cls == "MyClass"
|
||||
|
||||
def test_parse_file_path_class(self):
|
||||
"""Test parsing file/path.py:class format."""
|
||||
from textual_webterm.cli import parse_app_path
|
||||
|
||||
module, cls = parse_app_path("path/to/file.py:MyClass")
|
||||
assert module == "path/to/file.py"
|
||||
assert cls == "MyClass"
|
||||
|
||||
def test_parse_no_colon_raises(self):
|
||||
"""Test that missing colon raises BadParameter."""
|
||||
from textual_webterm.cli import parse_app_path
|
||||
|
||||
with pytest.raises(click.BadParameter) as exc_info:
|
||||
parse_app_path("invalid_format")
|
||||
assert "Expected format" in str(exc_info.value)
|
||||
|
||||
|
||||
class TestLoadAppClass:
|
||||
"""Tests for load_app_class function."""
|
||||
|
||||
def test_load_nonexistent_module(self):
|
||||
"""Test loading from non-existent module raises."""
|
||||
from textual_webterm.cli import load_app_class
|
||||
|
||||
with pytest.raises(click.BadParameter) as exc_info:
|
||||
load_app_class("nonexistent_module_xyz:MyClass")
|
||||
assert "Could not import" in str(exc_info.value)
|
||||
|
||||
def test_load_nonexistent_class(self):
|
||||
"""Test loading non-existent class from existing module raises."""
|
||||
from textual_webterm.cli import load_app_class
|
||||
|
||||
with pytest.raises(click.BadParameter) as exc_info:
|
||||
load_app_class("os:NonExistentClass")
|
||||
assert "has no attribute" in str(exc_info.value)
|
||||
|
||||
def test_load_existing_class(self):
|
||||
"""Test loading an existing class from a module."""
|
||||
from textual_webterm.cli import load_app_class
|
||||
|
||||
# Load Path from pathlib
|
||||
cls = load_app_class("pathlib:Path")
|
||||
assert cls is Path
|
||||
|
||||
def test_load_from_file_nonexistent(self):
|
||||
"""Test loading from non-existent file raises."""
|
||||
from textual_webterm.cli import load_app_class
|
||||
|
||||
with pytest.raises(click.BadParameter) as exc_info:
|
||||
load_app_class("/nonexistent/path.py:MyClass")
|
||||
assert (
|
||||
"not found" in str(exc_info.value).lower()
|
||||
or "does not exist" in str(exc_info.value).lower()
|
||||
)
|
||||
from webterm import cli
|
||||
|
||||
|
||||
class TestCLI:
|
||||
@@ -87,7 +10,7 @@ class TestCLI:
|
||||
|
||||
def test_cli_help(self):
|
||||
"""Test CLI help output."""
|
||||
from textual_webterm.cli import app as cli_app
|
||||
cli_app = cli.app
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli_app, ["--help"])
|
||||
@@ -95,8 +18,6 @@ class TestCLI:
|
||||
assert "terminal" in result.output.lower() or "command" in result.output.lower()
|
||||
|
||||
def test_cli_runs_terminal_command(self, monkeypatch):
|
||||
from textual_webterm import cli
|
||||
|
||||
calls: dict[str, object] = {}
|
||||
|
||||
class FakeServer:
|
||||
@@ -119,9 +40,6 @@ class TestCLI:
|
||||
|
||||
def test_cli_runs_default_shell(self, monkeypatch):
|
||||
import os
|
||||
|
||||
from textual_webterm import cli
|
||||
|
||||
calls: dict[str, object] = {}
|
||||
|
||||
class FakeServer:
|
||||
@@ -143,33 +61,18 @@ class TestCLI:
|
||||
assert result.exit_code == 0
|
||||
assert calls["terminal"][1] == os.environ["SHELL"]
|
||||
|
||||
def test_cli_app_module_validation_rejects(self):
|
||||
from textual_webterm.cli import app as cli_app
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli_app, ["--app", "os;rm -rf /:Fake"])
|
||||
assert result.exit_code != 0
|
||||
|
||||
def test_cli_version(self):
|
||||
"""Test CLI version output."""
|
||||
from textual_webterm.cli import app as cli_app
|
||||
cli_app = cli.app
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli_app, ["--version"])
|
||||
assert result.exit_code == 0
|
||||
assert "version" in result.output
|
||||
|
||||
def test_cli_invalid_app_path(self):
|
||||
"""Test CLI with invalid app path."""
|
||||
from textual_webterm.cli import app as cli_app
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli_app, ["--app", "invalid"])
|
||||
assert result.exit_code != 0
|
||||
|
||||
def test_cli_port_option(self):
|
||||
"""Test CLI port option parsing."""
|
||||
from textual_webterm.cli import app as cli_app
|
||||
cli_app = cli.app
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli_app, ["--help"])
|
||||
@@ -177,51 +80,83 @@ class TestCLI:
|
||||
|
||||
def test_cli_host_option(self):
|
||||
"""Test CLI host option parsing."""
|
||||
from textual_webterm.cli import app as cli_app
|
||||
cli_app = cli.app
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli_app, ["--help"])
|
||||
assert "--host" in result.output or "-H" in result.output
|
||||
|
||||
|
||||
class TestModuleValidation:
|
||||
"""Tests for module/class name validation in CLI."""
|
||||
|
||||
def test_invalid_module_characters(self):
|
||||
"""Test that invalid module names are rejected."""
|
||||
from textual_webterm.cli import app as cli_app
|
||||
|
||||
runner = CliRunner()
|
||||
# Module with shell characters should be rejected or fail gracefully
|
||||
result = runner.invoke(cli_app, ["--app", "os; rm -rf /:Fake"])
|
||||
# Should not succeed
|
||||
assert result.exit_code != 0
|
||||
|
||||
def test_invalid_class_name(self):
|
||||
"""Test that invalid class names are rejected."""
|
||||
from textual_webterm.cli import app as cli_app
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli_app, ["--app", "os:123invalid"])
|
||||
assert result.exit_code != 0
|
||||
|
||||
|
||||
class TestCLIOptions:
|
||||
"""Tests for CLI option handling."""
|
||||
|
||||
def test_debug_option(self):
|
||||
"""Test --debug option exists."""
|
||||
from textual_webterm.cli import app as cli_app
|
||||
cli_app = cli.app
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli_app, ["--help"])
|
||||
assert "--app" in result.output
|
||||
assert "--docker-watch" in result.output
|
||||
|
||||
def test_no_run_option(self):
|
||||
"""Test --no-run option exists."""
|
||||
from textual_webterm.cli import app as cli_app
|
||||
cli_app = cli.app
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli_app, ["--help"])
|
||||
# Check that basic options are documented
|
||||
assert "port" in result.output.lower()
|
||||
|
||||
|
||||
def test_package_version_fallback(monkeypatch):
|
||||
def raise_missing(_name: str):
|
||||
raise cli.PackageNotFoundError("webterm")
|
||||
|
||||
monkeypatch.setattr(cli, "version", raise_missing)
|
||||
assert cli._package_version() == "0.0.0"
|
||||
|
||||
|
||||
def test_cli_docker_watch_mode(monkeypatch):
|
||||
calls: dict[str, object] = {}
|
||||
|
||||
class FakeServer:
|
||||
def __init__(self, *_args, **_kwargs):
|
||||
calls["init"] = True
|
||||
|
||||
def add_terminal(self, name, command, slug):
|
||||
calls["terminal"] = (name, command, slug)
|
||||
|
||||
async def run(self):
|
||||
calls["run"] = True
|
||||
|
||||
monkeypatch.setattr(cli, "LocalServer", FakeServer)
|
||||
monkeypatch.setattr(cli.asyncio, "run", lambda _coro: calls.setdefault("run", True))
|
||||
monkeypatch.setattr(cli.constants, "DEBUG", True)
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli.app, ["--docker-watch"])
|
||||
assert result.exit_code == 0
|
||||
assert "terminal" not in calls
|
||||
|
||||
|
||||
def test_cli_windows_branch(monkeypatch):
|
||||
calls: dict[str, object] = {}
|
||||
|
||||
class FakeServer:
|
||||
def __init__(self, *_args, **_kwargs):
|
||||
calls["init"] = True
|
||||
|
||||
def add_terminal(self, name, command, slug):
|
||||
calls["terminal"] = (name, command, slug)
|
||||
|
||||
async def run(self):
|
||||
calls["run"] = True
|
||||
|
||||
monkeypatch.setattr(cli, "LocalServer", FakeServer)
|
||||
monkeypatch.setattr(cli.constants, "WINDOWS", True)
|
||||
monkeypatch.setattr(cli.asyncio, "run", lambda _coro: calls.setdefault("run", True))
|
||||
|
||||
runner = CliRunner()
|
||||
result = runner.invoke(cli.app, ["--docker-watch"])
|
||||
assert result.exit_code == 0
|
||||
assert calls.get("run") is True
|
||||
|
||||
Reference in New Issue
Block a user