mirror of
https://github.com/hanzoai/insights.git
synced 2026-10-30 16:15:31 +00:00
zeekayandHanzo Dev
70ff9c7236
tools/hogbox-preview drives posthog-hogland — PostHog's own internal Firecracker CLI and SDK, the one their engineers install with `brew install posthog/tap/hogland`. It is not a library we can point somewhere else: its default host is https://box.mascot-kitefin.ts.net, a machine on PostHog's tailnet, and that is the only .ts.net address in this entire tree. So it could never have run here. Nothing outside its own directory imports it — 13 files, 2,867 lines, self-contained and unreachable. insights-hogland and pixelhog leave pyproject's exclude-newer-package with it, since nothing asks for either now. Preview environments belong on our own compute. The Go cloud already serves /v1/compute/*; a backend against that is a thing to write, not a thing to rename PostHog's into. Also finishes the rename in packages/quill, tools and common/scriptvm — 74 files, plus renderHogChart and the scriptvm python objects. Co-authored-by: Hanzo Dev <[email protected]>
574 lines
24 KiB
Python
574 lines
24 KiB
Python
"""Tests for auto-detecting test runner."""
|
|
|
|
from __future__ import annotations
|
|
|
|
from pathlib import Path
|
|
|
|
import pytest
|
|
from unittest.mock import MagicMock, patch
|
|
|
|
import click
|
|
from click.testing import CliRunner
|
|
from insightscli.cli import cli
|
|
from insightscli_commands.test_runner import (
|
|
_batch_find_rs_cfg_test,
|
|
_detect_all,
|
|
_find_test_files_for_source,
|
|
_get_changed_files,
|
|
_is_test_file,
|
|
_resolve_to_repo_relative,
|
|
_run_changed,
|
|
_run_grouped,
|
|
detect_test_type,
|
|
)
|
|
from parameterized import parameterized
|
|
|
|
runner = CliRunner()
|
|
|
|
|
|
class TestDetectTestType:
|
|
"""Tests hit real files on disk — they validate that detection works
|
|
end-to-end against the actual repo layout (package.json, Cargo.toml, go.mod).
|
|
"""
|
|
|
|
@parameterized.expand(
|
|
[
|
|
("insights/api/test/test_user.py", "python", ["pytest", "-s", "insights/api/test/test_user.py"]),
|
|
("insights/models/test/test_team.py", "python", ["pytest", "-s", "insights/models/test/test_team.py"]),
|
|
("ee/datastore/test/test_client.py", "python", ["pytest", "-s", "ee/datastore/test/test_client.py"]),
|
|
(
|
|
"products/alerts/backend/test/test_api.py",
|
|
"python",
|
|
["pytest", "-s", "products/alerts/backend/test/test_api.py"],
|
|
),
|
|
("tools/insightscli/tests/test_cli.py", "python", ["pytest", "-s", "tools/insightscli/tests/test_cli.py"]),
|
|
("dags/tests/test_dag.py", "python", ["pytest", "-s", "dags/tests/test_dag.py"]),
|
|
]
|
|
)
|
|
def test_python_tests(self, file_path: str, expected_type: str, expected_command: list[str]) -> None:
|
|
config = detect_test_type(file_path)
|
|
assert config.test_type == expected_type
|
|
assert config.command == expected_command
|
|
assert "REDIS_URL" in config.env
|
|
|
|
def test_python_with_node_id(self) -> None:
|
|
config = detect_test_type("insights/api/test/test_user.py::TestUserAPI::test_retrieve")
|
|
assert config.test_type == "python"
|
|
assert config.command == ["pytest", "-s", "insights/api/test/test_user.py::TestUserAPI::test_retrieve"]
|
|
|
|
def test_python_eval_uses_special_config(self) -> None:
|
|
config = detect_test_type("ee/scriptai/eval/eval_router.py")
|
|
assert config.test_type == "python-eval"
|
|
assert config.command == ["pytest", "-c", "ee/scriptai/eval/pytest.ini", "-s", "ee/scriptai/eval/eval_router.py"]
|
|
assert "REDIS_URL" in config.env
|
|
|
|
# -- Jest tests: these hit real package.json files on disk --
|
|
|
|
@parameterized.expand(
|
|
[
|
|
("frontend/src/scenes/dashboard/Dashboard.test.tsx", "@hanzo/frontend"),
|
|
("frontend/src/lib/utils.test.ts", "@hanzo/frontend"),
|
|
]
|
|
)
|
|
def test_frontend_jest(self, file_path: str, expected_filter: str) -> None:
|
|
config = detect_test_type(file_path)
|
|
assert config.test_type == "jest"
|
|
assert config.command == ["pnpm", f"--filter={expected_filter}", "exec", "jest", file_path]
|
|
|
|
@parameterized.expand(
|
|
[
|
|
"products/ai_observability/frontend/utils.test.ts",
|
|
"products/llm_analytics/frontend/scenes/Trace.test.tsx",
|
|
]
|
|
)
|
|
def test_product_frontend_jest_routes_to_frontend(self, file_path: str) -> None:
|
|
config = detect_test_type(file_path)
|
|
assert config.test_type == "jest"
|
|
assert config.command == ["pnpm", "--filter=@hanzo/frontend", "exec", "jest", file_path]
|
|
|
|
@parameterized.expand(
|
|
[
|
|
(
|
|
"products/desktop/packages/core/src/archive/archiveListView.test.ts",
|
|
"products/desktop/packages/core",
|
|
"src/archive/archiveListView.test.ts",
|
|
),
|
|
(
|
|
"products/desktop/packages/ui/src/shell/logCapture.test.ts",
|
|
"products/desktop/packages/ui",
|
|
"src/shell/logCapture.test.ts",
|
|
),
|
|
]
|
|
)
|
|
def test_desktop_test_file_routes_to_package_vitest(self, file_path: str, pkg_dir: str, rel: str) -> None:
|
|
config = detect_test_type(file_path)
|
|
assert config.test_type == "vitest"
|
|
assert config.command == ["pnpm", "--dir", pkg_dir, "exec", "vitest", "run", rel]
|
|
|
|
def test_desktop_node_id_adds_test_name_filter(self) -> None:
|
|
config = detect_test_type("products/desktop/packages/core/src/archive/archiveListView.test.ts::renders rows")
|
|
assert config.test_type == "vitest"
|
|
assert config.command == [
|
|
"pnpm",
|
|
"--dir",
|
|
"products/desktop/packages/core",
|
|
"exec",
|
|
"vitest",
|
|
"run",
|
|
"src/archive/archiveListView.test.ts",
|
|
"-t",
|
|
"renders rows",
|
|
]
|
|
|
|
@parameterized.expand(
|
|
[
|
|
(
|
|
"products/desktop/packages/core",
|
|
["pnpm", "--dir", "products/desktop/packages/core", "exec", "vitest", "run"],
|
|
),
|
|
(
|
|
"products/desktop/packages/core/src/archive",
|
|
["pnpm", "--dir", "products/desktop/packages/core", "exec", "vitest", "run", "src/archive"],
|
|
),
|
|
("products/desktop", ["pnpm", "--dir", "products/desktop", "test"]),
|
|
("products/desktop/packages", ["pnpm", "--dir", "products/desktop", "test"]),
|
|
]
|
|
)
|
|
def test_desktop_directories_route_to_vitest_or_turbo(self, dir_path: str, expected_command: list[str]) -> None:
|
|
config = detect_test_type(dir_path)
|
|
assert config.test_type == "vitest"
|
|
assert config.command == expected_command
|
|
|
|
def test_jest_node_id_adds_test_name_pattern(self) -> None:
|
|
config = detect_test_type("frontend/src/lib/utils.test.ts::some test name")
|
|
assert config.test_type == "jest"
|
|
assert config.command == [
|
|
"pnpm",
|
|
"--filter=@hanzo/frontend",
|
|
"exec",
|
|
"jest",
|
|
"frontend/src/lib/utils.test.ts",
|
|
"--testNamePattern",
|
|
"some test name",
|
|
]
|
|
|
|
def test_nodejs_jest(self) -> None:
|
|
config = detect_test_type("nodejs/tests/cdp/cdp-api.test.ts")
|
|
assert config.test_type == "jest"
|
|
assert config.command[:4] == ["pnpm", "--filter=@hanzo/nodejs", "exec", "jest"]
|
|
|
|
@parameterized.expand(
|
|
[
|
|
("common/scriptvm/typescript/src/__tests__/execute.test.ts", "@hanzo/scriptvm"),
|
|
("common/replay-shared/src/replay.test.ts", "@hanzo/replay-shared"),
|
|
("common/replay-headless/src/render.test.ts", "@hanzo/replay-headless"),
|
|
]
|
|
)
|
|
def test_common_jest_packages(self, file_path: str, expected_filter: str) -> None:
|
|
config = detect_test_type(file_path)
|
|
assert config.test_type == "jest"
|
|
assert config.command[:4] == ["pnpm", f"--filter={expected_filter}", "exec", "jest"]
|
|
|
|
def test_playwright(self) -> None:
|
|
config = detect_test_type("playwright/e2e/dashboards.spec.ts")
|
|
assert config.test_type == "playwright"
|
|
assert config.command == ["pnpm", "exec", "playwright", "test", "playwright/e2e/dashboards.spec.ts"]
|
|
|
|
# -- Rust tests: these hit real Cargo.toml files on disk --
|
|
|
|
def test_rust_workspace_crate(self) -> None:
|
|
config = detect_test_type("rust/capture/tests/events.rs")
|
|
assert config.test_type == "rust"
|
|
assert "--manifest-path=rust/Cargo.toml" in config.command
|
|
assert "-p" in config.command
|
|
assert "capture" in config.command
|
|
assert "--test" in config.command
|
|
assert "events" in config.command
|
|
|
|
def test_rust_standalone_cli(self) -> None:
|
|
config = detect_test_type("cli/src/main.rs")
|
|
assert config.test_type == "rust"
|
|
assert "--manifest-path=cli/Cargo.toml" in config.command
|
|
assert "--" not in config.command # main.rs has no module filter
|
|
|
|
def test_rust_standalone_funnel_udf(self) -> None:
|
|
config = detect_test_type("funnel-udf/src/lib.rs")
|
|
assert config.test_type == "rust"
|
|
assert "--manifest-path=funnel-udf/Cargo.toml" in config.command
|
|
assert "--" not in config.command # lib.rs has no module filter
|
|
|
|
def test_rust_single_file_filters_to_module(self) -> None:
|
|
config = detect_test_type("cli/src/utils/throttler.rs")
|
|
assert config.test_type == "rust"
|
|
assert "--manifest-path=cli/Cargo.toml" in config.command
|
|
assert config.command[-2:] == ["--", "utils::throttler"]
|
|
|
|
def test_rust_directory_filters_to_module(self) -> None:
|
|
config = detect_test_type("cli/src/utils")
|
|
assert config.test_type == "rust"
|
|
assert "--manifest-path=cli/Cargo.toml" in config.command
|
|
assert config.command[-2:] == ["--", "utils"]
|
|
|
|
def test_rust_node_id_filters_to_test(self) -> None:
|
|
config = detect_test_type("cli/src/utils/throttler.rs::test_new_creates_empty_throttler")
|
|
assert config.test_type == "rust"
|
|
assert "--manifest-path=cli/Cargo.toml" in config.command
|
|
assert config.command[-2:] == ["--", "utils::throttler::test_new_creates_empty_throttler"]
|
|
|
|
def test_rust_no_cargo_toml_raises(self) -> None:
|
|
with pytest.raises(click.UsageError, match="No Cargo.toml found"):
|
|
detect_test_type("random/thing.rs")
|
|
|
|
# -- Go tests: these hit real go.mod files on disk --
|
|
# Any .go file or go.mod should run tests from the module root
|
|
|
|
@parameterized.expand(
|
|
[
|
|
("livestream/main_test.go", "livestream", "./..."),
|
|
("livestream/main.go", "livestream", "./..."),
|
|
("livestream/handlers/handler.go", "livestream", "./handlers/..."),
|
|
("livestream/go.mod", "livestream", "./..."),
|
|
("tools/phrocs/internal/tui/app_test.go", "tools/phrocs", "./internal/tui/..."),
|
|
("bin/hobby-installer/installer_test.go", "bin/hobby-installer", "./..."),
|
|
]
|
|
)
|
|
def test_go_tests(self, file_path: str, expected_mod_root: str, expected_target: str) -> None:
|
|
config = detect_test_type(file_path)
|
|
assert config.test_type == "go"
|
|
assert config.command == ["go", "test", expected_target]
|
|
assert config.cwd == _get_repo_root() / expected_mod_root
|
|
|
|
def test_go_no_go_mod_raises(self) -> None:
|
|
with pytest.raises(click.UsageError, match="No go.mod found"):
|
|
detect_test_type("random/foo_test.go")
|
|
|
|
# -- Directory tests: run all tests in a directory --
|
|
|
|
@parameterized.expand(
|
|
[
|
|
("insights/api/test", "python", ["pytest", "-s", "insights/api/test"]),
|
|
("ee/scriptai", "python", ["pytest", "-s", "ee/scriptai"]),
|
|
(
|
|
"tools/insightscli-commands/insightscli_commands/tests",
|
|
"python",
|
|
["pytest", "-s", "tools/insightscli-commands/insightscli_commands/tests"],
|
|
),
|
|
]
|
|
)
|
|
def test_python_directory(self, dir_path: str, expected_type: str, expected_command: list[str]) -> None:
|
|
config = detect_test_type(dir_path)
|
|
assert config.test_type == expected_type
|
|
assert config.command == expected_command
|
|
|
|
def test_go_directory(self) -> None:
|
|
config = detect_test_type("livestream")
|
|
assert config.test_type == "go"
|
|
assert config.command == ["go", "test", "./..."]
|
|
assert config.cwd == _get_repo_root() / "livestream"
|
|
|
|
def test_go_subdirectory(self) -> None:
|
|
config = detect_test_type("livestream/events")
|
|
assert config.test_type == "go"
|
|
assert config.command == ["go", "test", "./events/..."]
|
|
assert config.cwd == _get_repo_root() / "livestream"
|
|
|
|
def test_rust_directory(self) -> None:
|
|
config = detect_test_type("rust/capture/tests")
|
|
assert config.test_type == "rust"
|
|
assert "--manifest-path=rust/Cargo.toml" in config.command
|
|
|
|
def test_jest_directory(self) -> None:
|
|
config = detect_test_type("frontend/src/scenes/dashboard")
|
|
assert config.test_type == "jest"
|
|
assert config.command[:4] == ["pnpm", "--filter=@hanzo/frontend", "exec", "jest"]
|
|
|
|
def test_playwright_directory(self) -> None:
|
|
config = detect_test_type("playwright/e2e")
|
|
assert config.test_type == "playwright"
|
|
assert config.command == ["pnpm", "exec", "playwright", "test", "playwright/e2e"]
|
|
|
|
# -- Product directory (Turbo) tests --
|
|
|
|
def test_product_root_uses_turbo(self) -> None:
|
|
config = detect_test_type("products/alerts")
|
|
assert config.test_type == "turbo"
|
|
assert config.command == ["pnpm", "turbo", "run", "backend:test", "--filter=@hanzo/products-alerts"]
|
|
|
|
def test_product_root_trailing_slash_uses_turbo(self) -> None:
|
|
config = detect_test_type("products/alerts/")
|
|
assert config.test_type == "turbo"
|
|
assert config.command == ["pnpm", "turbo", "run", "backend:test", "--filter=@hanzo/products-alerts"]
|
|
|
|
def test_product_subdirectory_uses_pytest(self) -> None:
|
|
config = detect_test_type("products/error_tracking/backend/test")
|
|
assert config.test_type == "python"
|
|
assert config.command == ["pytest", "-s", "products/error_tracking/backend/test"]
|
|
|
|
# -- Edge cases --
|
|
|
|
def test_unknown_file_raises(self) -> None:
|
|
with pytest.raises(click.UsageError, match="Could not detect test type"):
|
|
detect_test_type("random/file.txt")
|
|
|
|
def test_unknown_python_location_raises(self) -> None:
|
|
with pytest.raises(click.UsageError, match="Could not detect test type"):
|
|
detect_test_type("somewhere/test_foo.py")
|
|
|
|
@patch("insightscli_commands.test_runner.platform")
|
|
def test_macos_env_var(self, mock_platform) -> None:
|
|
mock_platform.system.return_value = "Darwin"
|
|
config = detect_test_type("insights/api/test/test_user.py")
|
|
assert config.env["OBJC_DISABLE_INITIALIZE_FORK_SAFETY"] == "YES"
|
|
|
|
@patch("insightscli_commands.test_runner.platform")
|
|
def test_linux_no_objc_env_var(self, mock_platform) -> None:
|
|
mock_platform.system.return_value = "Linux"
|
|
config = detect_test_type("insights/api/test/test_user.py")
|
|
assert "OBJC_DISABLE_INITIALIZE_FORK_SAFETY" not in config.env
|
|
|
|
|
|
class TestResolveToRepoRelative:
|
|
def test_relative_path_unchanged(self) -> None:
|
|
with patch("insightscli_commands.test_runner.Path.cwd", return_value=Path(str(_get_repo_root()))):
|
|
result = _resolve_to_repo_relative("insights/api/test/test_user.py")
|
|
assert result == "insights/api/test/test_user.py"
|
|
|
|
def test_absolute_path_made_relative(self) -> None:
|
|
repo_root = str(_get_repo_root())
|
|
result = _resolve_to_repo_relative(f"{repo_root}/insights/api/test/test_user.py")
|
|
assert result == "insights/api/test/test_user.py"
|
|
|
|
def test_node_id_preserved(self) -> None:
|
|
repo_root = str(_get_repo_root())
|
|
result = _resolve_to_repo_relative(f"{repo_root}/insights/test_foo.py::TestBar::test_baz")
|
|
assert result == "insights/test_foo.py::TestBar::test_baz"
|
|
|
|
def test_relative_with_node_id(self) -> None:
|
|
with patch("insightscli_commands.test_runner.Path.cwd", return_value=Path(str(_get_repo_root()))):
|
|
result = _resolve_to_repo_relative("insights/test_foo.py::TestBar")
|
|
assert result == "insights/test_foo.py::TestBar"
|
|
|
|
|
|
class TestIsTestFile:
|
|
@parameterized.expand(
|
|
[
|
|
("insights/api/test/test_user.py", True),
|
|
("ee/scriptai/eval/eval_router.py", True),
|
|
("ee/scriptai/eval_router.py", False),
|
|
("insights/eval_something.py", False),
|
|
("ee/scriptai/router.py", False),
|
|
("insights/models/team.py", False),
|
|
("frontend/src/scenes/dashboard/Dashboard.test.tsx", True),
|
|
("frontend/src/scenes/dashboard/Dashboard.tsx", False),
|
|
("playwright/e2e/dashboards.spec.ts", True),
|
|
("playwright/e2e/helpers.ts", False),
|
|
("rust/capture/tests/events.rs", True),
|
|
("rust/capture/src/api_test.rs", True),
|
|
("livestream/main_test.go", True),
|
|
("livestream/main.go", False),
|
|
]
|
|
)
|
|
def test_is_test_file(self, path: str, expected: bool) -> None:
|
|
assert _is_test_file(path) == expected
|
|
|
|
@parameterized.expand(
|
|
[
|
|
("rust/capture/src/api.rs",),
|
|
("rust/capture/src/v1/util.rs",),
|
|
]
|
|
)
|
|
def test_rs_inline_cfg_test_detected_with_batch(self, path: str) -> None:
|
|
rs_cfg_test = _batch_find_rs_cfg_test("rust/capture/src")
|
|
assert _is_test_file(path, rs_cfg_test=rs_cfg_test)
|
|
|
|
def test_rs_inline_cfg_test_not_detected_without_batch(self) -> None:
|
|
assert not _is_test_file("rust/capture/src/api.rs")
|
|
|
|
def test_rs_inline_cfg_test_absent_from_batch(self) -> None:
|
|
assert not _is_test_file("rust/capture/src/api.rs", rs_cfg_test=set())
|
|
|
|
|
|
class TestGetChangedFiles:
|
|
@patch("insightscli_commands.test_runner.changed_files")
|
|
@patch("insightscli_commands.test_runner.subprocess.run")
|
|
def test_drops_paths_missing_from_disk(self, mock_run: MagicMock, mock_changed: MagicMock) -> None:
|
|
mock_run.return_value = MagicMock(stdout="feature-branch\n")
|
|
mock_changed.return_value = [
|
|
"tools/insightscli-commands/insightscli_commands/test_runner.py",
|
|
"renamed_away/test_gone.py",
|
|
]
|
|
assert _get_changed_files() == ["tools/insightscli-commands/insightscli_commands/test_runner.py"]
|
|
|
|
|
|
class TestFindTestFilesForSource:
|
|
@parameterized.expand(
|
|
[
|
|
("insights/api/comments.py", "insights/api/test/test_comments.py"),
|
|
(
|
|
"tools/insightscli-commands/insightscli_commands/test_runner.py",
|
|
"tools/insightscli-commands/insightscli_commands/tests/test_test_runner.py",
|
|
),
|
|
(
|
|
"frontend/src/scenes/dashboard/DashboardHeader.tsx",
|
|
"frontend/src/scenes/dashboard/DashboardHeader.test.tsx",
|
|
),
|
|
("livestream/events/filter.go", "livestream/events/filter_test.go"),
|
|
]
|
|
)
|
|
def test_finds_test_for_source(self, source: str, expected_test: str) -> None:
|
|
results = _find_test_files_for_source(source)
|
|
assert expected_test in results
|
|
|
|
def test_non_source_file_returns_empty(self) -> None:
|
|
assert _find_test_files_for_source("README.md") == []
|
|
|
|
def test_nonexistent_file_returns_empty(self) -> None:
|
|
assert _find_test_files_for_source("insights/api/nonexistent_module.py") == []
|
|
|
|
|
|
class TestRunChanged:
|
|
@patch("insightscli_commands.test_runner._run")
|
|
@patch("insightscli_commands.test_runner._get_changed_files")
|
|
def test_runs_changed_python_files(self, mock_changed, mock_run) -> None:
|
|
mock_changed.return_value = [
|
|
"insights/api/test/test_user.py",
|
|
"insights/api/test/test_comments.py",
|
|
"insights/api/views.py", # not a test file
|
|
]
|
|
_run_changed([])
|
|
|
|
mock_run.assert_called_once()
|
|
command = mock_run.call_args[0][0]
|
|
assert command[0] == "pytest"
|
|
assert "insights/api/test/test_user.py" in command
|
|
assert "insights/api/test/test_comments.py" in command
|
|
assert "insights/api/views.py" not in command
|
|
|
|
@patch("insightscli_commands.test_runner._run")
|
|
@patch("insightscli_commands.test_runner._get_changed_files")
|
|
def test_jest_files_grouped_by_package(self, mock_changed, mock_run) -> None:
|
|
mock_changed.return_value = [
|
|
"frontend/src/scenes/dashboard/Dashboard.test.tsx",
|
|
"frontend/src/lib/utils.test.ts",
|
|
"nodejs/tests/cdp/cdp-api.test.ts",
|
|
]
|
|
_run_changed([])
|
|
|
|
assert mock_run.call_count == 2
|
|
commands = [call[0][0] for call in mock_run.call_args_list]
|
|
# One call for @hanzo/frontend, one for @hanzo/nodejs
|
|
frontend_cmd = next(c for c in commands if "--filter=@hanzo/frontend" in c)
|
|
nodejs_cmd = next(c for c in commands if "--filter=@hanzo/nodejs" in c)
|
|
assert "frontend/src/scenes/dashboard/Dashboard.test.tsx" in frontend_cmd
|
|
assert "frontend/src/lib/utils.test.ts" in frontend_cmd
|
|
assert "nodejs/tests/cdp/cdp-api.test.ts" in nodejs_cmd
|
|
|
|
@patch("insightscli_commands.test_runner._run")
|
|
@patch("insightscli_commands.test_runner._get_changed_files")
|
|
def test_discovers_tests_for_changed_source_files(self, mock_changed, mock_run) -> None:
|
|
mock_changed.return_value = ["insights/api/comments.py"]
|
|
_run_changed([])
|
|
|
|
mock_run.assert_called_once()
|
|
command = mock_run.call_args[0][0]
|
|
assert "insights/api/test/test_comments.py" in command
|
|
|
|
@patch("insightscli_commands.test_runner._run")
|
|
@patch("insightscli_commands.test_runner._get_changed_files")
|
|
def test_deduplicates_direct_and_discovered_tests(self, mock_changed, mock_run) -> None:
|
|
mock_changed.return_value = [
|
|
"insights/api/comments.py",
|
|
"insights/api/test/test_comments.py",
|
|
]
|
|
_run_changed([])
|
|
|
|
mock_run.assert_called_once()
|
|
command = mock_run.call_args[0][0]
|
|
assert command.count("insights/api/test/test_comments.py") == 1
|
|
|
|
@patch("insightscli_commands.test_runner._get_changed_files")
|
|
def test_no_changed_files_exits(self, mock_changed) -> None:
|
|
mock_changed.return_value = ["insights/api/views.py"]
|
|
with pytest.raises(SystemExit):
|
|
_run_changed([])
|
|
|
|
@patch("insightscli_commands.test_runner._get_changed_files")
|
|
def test_changed_on_master_raises(self, mock_changed) -> None:
|
|
mock_changed.side_effect = click.UsageError("Cannot use --changed on the master branch.")
|
|
with pytest.raises(click.UsageError, match="master"):
|
|
_run_changed([])
|
|
|
|
|
|
class TestRunGrouped:
|
|
@patch("insightscli_commands.test_runner._run")
|
|
def test_mixed_python_and_jest(self, mock_run) -> None:
|
|
detected = _detect_all(
|
|
[
|
|
"insights/api/test/test_user.py",
|
|
"frontend/src/scenes/dashboard/Dashboard.test.tsx",
|
|
]
|
|
)
|
|
_run_grouped(detected, [])
|
|
assert mock_run.call_count == 2
|
|
commands = [call[0][0] for call in mock_run.call_args_list]
|
|
python_cmd = next(c for c in commands if c[0] == "pytest")
|
|
jest_cmd = next(c for c in commands if c[0] == "pnpm")
|
|
assert "insights/api/test/test_user.py" in python_cmd
|
|
assert "frontend/src/scenes/dashboard/Dashboard.test.tsx" in jest_cmd
|
|
|
|
@patch("insightscli_commands.test_runner._run")
|
|
def test_passes_extra_args(self, mock_run) -> None:
|
|
detected = _detect_all(["insights/api/test/test_user.py"])
|
|
_run_grouped(detected, ["-v", "--tb=short"])
|
|
command = mock_run.call_args[0][0]
|
|
assert "-v" in command
|
|
assert "--tb=short" in command
|
|
|
|
@patch("insightscli_commands.test_runner._run")
|
|
def test_jest_grouped_by_package(self, mock_run) -> None:
|
|
detected = _detect_all(
|
|
[
|
|
"frontend/src/scenes/dashboard/Dashboard.test.tsx",
|
|
"frontend/src/lib/utils.test.ts",
|
|
"nodejs/tests/cdp/cdp-api.test.ts",
|
|
]
|
|
)
|
|
_run_grouped(detected, [])
|
|
assert mock_run.call_count == 2
|
|
commands = [call[0][0] for call in mock_run.call_args_list]
|
|
frontend_cmd = next(c for c in commands if "--filter=@hanzo/frontend" in c)
|
|
nodejs_cmd = next(c for c in commands if "--filter=@hanzo/nodejs" in c)
|
|
assert "frontend/src/scenes/dashboard/Dashboard.test.tsx" in frontend_cmd
|
|
assert "frontend/src/lib/utils.test.ts" in frontend_cmd
|
|
assert "nodejs/tests/cdp/cdp-api.test.ts" in nodejs_cmd
|
|
|
|
|
|
class TestCliPassthrough:
|
|
@patch("insightscli_commands.test_runner._run")
|
|
def test_insightscli_test_passes_unknown_options_to_runner(self, mock_run: MagicMock) -> None:
|
|
result = runner.invoke(
|
|
cli,
|
|
[
|
|
"test",
|
|
"tools/insightscli-commands/insightscli_commands/tests/test_commands.py",
|
|
"-k",
|
|
"cache",
|
|
],
|
|
)
|
|
|
|
assert result.exit_code == 0, result.output
|
|
command = mock_run.call_args[0][0]
|
|
assert command == [
|
|
"pytest",
|
|
"-s",
|
|
"tools/insightscli-commands/insightscli_commands/tests/test_commands.py",
|
|
"-k",
|
|
"cache",
|
|
]
|
|
|
|
|
|
def _get_repo_root():
|
|
from insightscli.manifest import REPO_ROOT
|
|
|
|
return REPO_ROOT
|