Trying color flag to false for ASCII error

Author: Asanto32

src/wristpy/core/cli.py

@@ -84,7 +84,7 @@ def main(
         ActivityMetric.enmo,
         "-a",
         "--activity-metric",
-        help="Metric should be used for physical activity categorization."
+        help="Metric used for physical activity categorization."
         "Choose from 'enmo', 'mad', or 'ag_count'.",
         case_sensitive=False,
     ),
@@ -94,7 +94,7 @@ def main(
         "--thresholds",
         help="Provide three thresholds for light, moderate, and vigorous activity. "
         "Exactly three values must be >= 0, given in ascending order,"
-        " and separated by a space.",
+        " and separated by a space. (e.g. '-t 0.1 1.0 1.5').",
         min=0,
     ),
     nonwear_algorithm: list[NonwearAlgorithms] = typer.Option(

tests/unit/test_cli.py

@@ -5,21 +5,21 @@
 
 import pytest
 import pytest_mock
-from typer.testing import CliRunner
+from typer import testing
 
 from wristpy.core import cli, orchestrator
 
 
 @pytest.fixture
-def create_typer_cli_runner() -> CliRunner:
+def create_typer_cli_runner() -> testing.CliRunner:
     """Create a Typer CLI runner."""
-    return CliRunner()
+    return testing.CliRunner()
 
 
 def test_main_default(
     mocker: pytest_mock.MockerFixture,
     sample_data_gt3x: pathlib.Path,
-    create_typer_cli_runner: CliRunner,
+    create_typer_cli_runner: testing.CliRunner,
 ) -> None:
     """Test cli with only necessary arguments."""
     mock_run = mocker.patch.object(orchestrator, "run")
@@ -44,7 +44,7 @@ def test_main_default(
 def test_main_enmo_default(
     mocker: pytest_mock.MockerFixture,
     sample_data_gt3x: pathlib.Path,
-    create_typer_cli_runner: CliRunner,
+    create_typer_cli_runner: testing.CliRunner,
 ) -> None:
     """Test that correct enmo default thresholds are pulled."""
     mock_run = mocker.patch.object(orchestrator, "_run_file")
@@ -67,7 +67,7 @@ def test_main_enmo_default(
 def test_main_mad_default(
     mocker: pytest_mock.MockerFixture,
     sample_data_gt3x: pathlib.Path,
-    create_typer_cli_runner: CliRunner,
+    create_typer_cli_runner: testing.CliRunner,
 ) -> None:
     """Test that correct mad default thresholds are pulled."""
     mock_run = mocker.patch.object(orchestrator, "_run_file")
@@ -90,7 +90,7 @@ def test_main_mad_default(
 def test_main_agcount_default(
     mocker: pytest_mock.MockerFixture,
     sample_data_gt3x: pathlib.Path,
-    create_typer_cli_runner: CliRunner,
+    create_typer_cli_runner: testing.CliRunner,
 ) -> None:
     """Test that correct ag_count default thresholds are pulled."""
     mock_run = mocker.patch.object(orchestrator, "_run_file")
@@ -114,7 +114,7 @@ def test_main_with_options(
     mocker: pytest_mock.MockerFixture,
     sample_data_gt3x: pathlib.Path,
     tmp_path: pathlib.Path,
-    create_typer_cli_runner: CliRunner,
+    create_typer_cli_runner: testing.CliRunner,
 ) -> None:
     """Test cli with optional arguments."""
     test_output = tmp_path / "test.csv"
@@ -160,31 +160,31 @@ def test_main_with_options(
 
 def test_main_with_bad_thresholds(
     sample_data_gt3x: pathlib.Path,
-    create_typer_cli_runner: CliRunner,
+    create_typer_cli_runner: testing.CliRunner,
 ) -> None:
     """Test cli with bad thresholds."""
     result = create_typer_cli_runner.invoke(
-        cli.app, [str(sample_data_gt3x), "-t", "3.0"]
+        cli.app, [str(sample_data_gt3x), "-t", "3.0"], color=False
     )
 
     assert result.exit_code != 0
-    # partial matching due to ANSI escape sequences in Github Actions
-    assert "requires 3 arguments." in result.output
+    assert "Option '-t' requires 3 arguments." in result.output
 
 
 def test_main_with_bad_epoch(
     sample_data_gt3x: pathlib.Path,
-    create_typer_cli_runner: CliRunner,
+    create_typer_cli_runner: testing.CliRunner,
 ) -> None:
     """Test cli with invalid epoch length."""
     result = create_typer_cli_runner.invoke(
-        cli.app, [str(sample_data_gt3x), "-e", "-5"]
+        cli.app, [str(sample_data_gt3x), "-e", "-5"], color=False
     )
 
     assert result.exit_code != 0
-    # partial matching due to ANSI escape sequences in Github Actions
-    assert "Invalid value for" in result.output
-    assert "is not in the range x>=1." in result.output
+    assert (
+        "Invalid value for '-e' / '--epoch-length': -5 is not in the range x>=1."
+        in result.output
+    )
 
 
 @pytest.mark.parametrize(
@@ -198,7 +198,7 @@ def test_main_with_bad_epoch(
 def test_main_verbosity(
     mocker: pytest_mock.MockerFixture,
     sample_data_gt3x: pathlib.Path,
-    create_typer_cli_runner: CliRunner,
+    create_typer_cli_runner: testing.CliRunner,
     verbosity: str,
     expected_log_level: int,
 ) -> None: