8000 test(cli): add coverage for maximize/list-solvers/version/managers commands by dominicusin · Pull Request #59 · dominicusin/package-maximizer · GitHub
[go: up one dir, main page]

Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
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
19 changes: 19 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,3 +32,22 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Z3Solver `Solver` → `Optimize` critical bug.
- `pulp_solver` import compatibility.
- Makefile `PYTHON` override for venv compatibility.

## [0.6.0] - 2026-09-02

### Added
- 12 new package managers: apk, zypper, yum, pip, gem, yarn, composer, vcpkg, nuget, winget, scoop, choco.
- Registered CondaParser and PortageParser (were implemented but not registered).
- Total registered package managers: **8 → 22**.
- CLI `list-managers` command shows all supported package managers.
- Updated README with full list of 22 package managers.

### Changed
- Updated `--manager` help text with all 22 options.
- Updated package description to reflect 22+ package manager support.

### Tests
- Added 24 tests for all new parsers.
- Added tests for `list-managers` CLI command.
- Total: 492 passed, 2 skipped.
- Coverage: **85%**.
76 changes: 76 additions & 0 deletions tests/test_cli_commands.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,10 @@
config_command,
init_config_command,
tui_command,
list_managers,
maximize,
list_solvers,
version,
)


Expand Down Expand Up @@ -112,3 +116,75 @@ def test_tui_missing_textual_exits(self, monkeypatch):
runner = CliRunner()
result = runner.invoke(tui_command, [])
assert result.exit_code != 0


class TestListManagersCommand:
"""list_managers shows all 22 registered package managers."""

def test_list_managers_text(self):
runner = CliRunner()
result = runner.invoke(list_managers, [])
assert result.exit_code == 0
assert "apt" in result.output
assert "pacman" in result.output
assert "brew" in result.output
assert "pip" in result.output
assert "conda" in result.output
assert "choco" in result.output


class TestMaximizeCommand:
"""maximize covers solver + error branches."""

def test_maximize_basic(self):
runner = CliRunner()
result = runner.invoke(maximize, ["alpha", "beta", "--solver", "greedy", "--output", "text"])
assert result.exit_code == 0
assert "alpha" in result.output or "beta" in result.output

def test_maximize_json(self):
runner = CliRunner()
result = runner.invoke(maximize, ["alpha", "beta", "--solver", "greedy", "--output", "json"])
assert result.exit_code == 0
data = json.loads(result.output)
93B6 assert "output" in data

def test_maximize_unknown_manager(self):
runner = CliRunner()
result = runner.invoke(maximize, ["alpha", "--manager", "unknown_xyz"])
assert result.exit_code != 0

def test_maximize_unknown_solver(self):
runner = CliRunner()
result = runner.invoke(maximize, ["alpha", "--solver", "unknown_xyz"])
assert result.exit_code != 0

def test_maximize_with_conflicts(self):
runner = CliRunner()
result = runner.invoke(maximize, ["alpha", "beta", "--solver", "greedy", "-c", "alpha", "beta"])
assert result.exit_code == 0

def test_maximize_with_weights(self):
runner = CliRunner()
result = runner.invoke(maximize, ["alpha", "beta", "--solver", "greedy", "-w", "alpha", "2.0", "-w", "beta", "1.0"])
assert result.exit_code == 0


class TestListSolversCommand:
"""list_solvers shows all registered solvers."""

def test_list_solvers(self):
runner = CliRunner()
result = runner.invoke(list_solvers, [])
assert result.exit_code == 0
assert "greedy" in result.output


class TestVersionCommand:
"""version prints the version string."""

def test_version(self):
runner = CliRunner()
result = runner.invoke(version, [])
assert result.exit_code == 0
assert "version" in result.output.lower()
0