fc973f4322
* Initial CLI implementation with click package * Adding fetch command for pulling examples and deepspeed configs * Automating default options for CliArgs classes * Mimicking existing no config behavior * bugfix in choose_config * Updating fetch to sync instead of re-download * bugfix * isort fix * fixing yaml isort order * pre-commit fixes * simplifying argument parsing -- pass through kwargs to do_cli * make accelerate launch default for non-preprocess commands * fixing arg handling * testing None placeholder approach * removing hacky --use-gpu argument to preprocess command * Adding brief README documentation for CLI * remove (New) * Initial CLI pytest tests * progress on CLI pytest * adding inference CLI tests; cleanup * Refactor train CLI tests to remove various mocking * Major CLI test refator; adding remaining CLI codepath test coverage * pytest fixes * remove integration markers * parallelizing examples, deepspeed config downloads; rename test to match other CLI test naming * moving cli pytest due to isolation issues; cleanup * testing fixes; various minor improvements * fix * tests fix * Update tests/cli/conftest.py Co-authored-by: Wing Lian <wing.lian@gmail.com> --------- Co-authored-by: Dan Saunders <dan@axolotl.ai> Co-authored-by: Wing Lian <wing.lian@gmail.com>
72 lines
2.3 KiB
Python
72 lines
2.3 KiB
Python
"""pytest tests for axolotl CLI preprocess command."""
|
|
import shutil
|
|
from pathlib import Path
|
|
from unittest.mock import patch
|
|
|
|
import pytest
|
|
|
|
from axolotl.cli.main import cli
|
|
|
|
|
|
@pytest.fixture(autouse=True)
|
|
def cleanup_last_run_prepared():
|
|
yield
|
|
|
|
if Path("last_run_prepared").exists():
|
|
shutil.rmtree("last_run_prepared")
|
|
|
|
|
|
def test_preprocess_config_not_found(cli_runner):
|
|
"""Test preprocess fails when config not found"""
|
|
result = cli_runner.invoke(cli, ["preprocess", "nonexistent.yml"])
|
|
assert result.exit_code != 0
|
|
|
|
|
|
def test_preprocess_basic(cli_runner, config_path):
|
|
"""Test basic preprocessing with minimal config"""
|
|
with patch("axolotl.cli.preprocess.do_cli") as mock_do_cli:
|
|
result = cli_runner.invoke(cli, ["preprocess", str(config_path)])
|
|
assert result.exit_code == 0
|
|
|
|
mock_do_cli.assert_called_once()
|
|
assert mock_do_cli.call_args.kwargs["config"] == str(config_path)
|
|
assert mock_do_cli.call_args.kwargs["download"] is True
|
|
|
|
|
|
def test_preprocess_without_download(cli_runner, config_path):
|
|
"""Test preprocessing without model download"""
|
|
with patch("axolotl.cli.preprocess.do_cli") as mock_do_cli:
|
|
result = cli_runner.invoke(
|
|
cli, ["preprocess", str(config_path), "--no-download"]
|
|
)
|
|
assert result.exit_code == 0
|
|
|
|
mock_do_cli.assert_called_once()
|
|
assert mock_do_cli.call_args.kwargs["config"] == str(config_path)
|
|
assert mock_do_cli.call_args.kwargs["download"] is False
|
|
|
|
|
|
def test_preprocess_custom_path(cli_runner, tmp_path, valid_test_config):
|
|
"""Test preprocessing with custom dataset path"""
|
|
config_path = tmp_path / "config.yml"
|
|
custom_path = tmp_path / "custom_prepared"
|
|
config_path.write_text(valid_test_config)
|
|
|
|
with patch("axolotl.cli.preprocess.do_cli") as mock_do_cli:
|
|
result = cli_runner.invoke(
|
|
cli,
|
|
[
|
|
"preprocess",
|
|
str(config_path),
|
|
"--dataset-prepared-path",
|
|
str(custom_path.absolute()),
|
|
],
|
|
)
|
|
assert result.exit_code == 0
|
|
|
|
mock_do_cli.assert_called_once()
|
|
assert mock_do_cli.call_args.kwargs["config"] == str(config_path)
|
|
assert mock_do_cli.call_args.kwargs["dataset_prepared_path"] == str(
|
|
custom_path.absolute()
|
|
)
|