mirror of
https://github.com/actionbook/actionbook.git
synced 2026-09-19 03:28:06 +08:00
s1mba
893315f4aa
* [packages/cli]test: improve CLI E2E test coverage to 93% Add 56 new test cases covering previously untested commands, subcommands, and parameter combinations. New test files for setup, extension, help, and global options. Enhanced existing browser, config, search, and get tests with parameter-level coverage. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * [packages/cli]test: add real assertions to browser E2E tests Replace exit-code-only checks with actual value assertions: - scroll tests verify window.scrollY changes before/after - select/focus verify element value and activeElement - type verifies input field value contains typed text - cookies delete/clear verify cookie is actually removed - navigate goto verifies location.href - inspect verifies output contains element info Move restart/close tests to end to prevent cascade failures. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * [packages/actionbook-rs]test: expand Rust CLI integration tests to match TypeScript E2E coverage Add ~70 new tests bringing total to 123, covering all CLI commands and options: help, browser (57 tests), config, profile, extension, setup, search, get, sources, and global flags. Mirrors the TypeScript E2E test suite structure. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * [packages/cli]fix: relax assertions for offline/extension-mode-only tests - extension install: accept exit code 0 or 1 (network failures in CI) - cookies set --domain: tolerate graceful failure outside extension mode Addresses PR #167 review feedback. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * [packages/actionbook-rs]test: add browser E2E tests mirroring TypeScript cli-browser-e2e.test.ts 57 individual #[test] functions covering navigation, page content, form interaction, screenshot/pdf, cookies, scroll, type, select/hover/focus/press, inspect, tab management, and restart/close. Shares browser session via OnceLock, gated by RUN_BROWSER_TESTS=true, runs with --test-threads=1. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * [packages/cli]fix: address PR #167 review comments from Codex - cli_test.rs: fail setup tests explicitly when process is killed by signal (timeout) instead of silently passing - cli_test.rs: isolate extension stop test with temp HOME/XDG dirs to avoid killing real user bridge processes - cli-browser-e2e.test.ts: clarify that cookie --dry-run/--domain tests require extension mode and rename describe block Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>