feat: add --use-desc-for-id to deactivate_jobs; add CLI tests for all commands

Add the missing @option_use_desc_for_id decorator and use_desc_for_id kwarg to
DBTCloud() in deactivate_jobs, matching the pattern used by all other commands.
Add five new tests (validate, import_jobs, link, unlink, deactivate_jobs) that
verify DBTCloud is constructed with use_desc_for_id=True when the flag is passed.

Author: b-per

src/dbt_jobs_as_code/main.py

@@ -645,8 +645,15 @@ def unlink(
     multiple=True,
     help="The ID of the job to deactivate.",
 )
+@option_use_desc_for_id
 def deactivate_jobs(
-    config, account_id, project_id, environment_id, job_id, disable_ssl_verification
+    config,
+    account_id,
+    project_id,
+    environment_id,
+    job_id,
+    disable_ssl_verification,
+    use_desc_for_id,
 ):
     """
     Deactivate jobs triggers in dbt Cloud (schedule and CI/CI triggers) without remoing the jobs.
@@ -669,6 +676,7 @@ def deactivate_jobs(
         api_key=os.environ.get("DBT_API_KEY"),
         base_url=os.environ.get("DBT_BASE_URL", "https://cloud.getdbt.com"),
         disable_ssl_verification=disable_ssl_verification,
+        use_desc_for_id=use_desc_for_id,
     )
     cloud_jobs = dbt_cloud.get_jobs()
 

tests/test_main.py

@@ -534,3 +534,131 @@ def test_use_desc_for_id_default_false(mock_build_change_set, mock_empty_change_
     mock_build_change_set.assert_called_once()
     call_args = mock_build_change_set.call_args
     assert call_args.kwargs.get("use_desc_for_id") is False
+
+
+@patch("dbt_jobs_as_code.main.DBTCloud")
+@patch("dbt_jobs_as_code.main.load_job_configuration")
+@patch("dbt_jobs_as_code.main.resolve_file_paths")
+def test_use_desc_for_id_option_validate(
+    mock_resolve_file_paths, mock_load_job_configuration, mock_DBTCloud
+):
+    """Test that validate --online passes use_desc_for_id=True to DBTCloud"""
+    from dbt_jobs_as_code.schemas.common_types import Settings, Triggers
+    from dbt_jobs_as_code.schemas.job import JobDefinition
+
+    mock_resolve_file_paths.return_value = (["config.yml"], [])
+
+    job = JobDefinition(
+        project_id=123,
+        environment_id=456,
+        account_id=789,
+        name="Test Job",
+        settings=Settings(threads=4),
+        run_generate_sources=False,
+        execute_steps=["dbt run"],
+        generate_docs=False,
+        schedule={"cron": "0 * * * *"},
+        triggers=Triggers(schedule=True),
+    )
+    mock_config = Mock()
+    mock_config.jobs = {"test-job": job}
+    mock_load_job_configuration.return_value = mock_config
+
+    instance = mock_DBTCloud.return_value
+    instance.get_environments.return_value = [{"id": 456, "project_id": 123}]
+    instance.get_jobs.return_value = []
+
+    runner = CliRunner()
+    result = runner.invoke(cli, ["validate", "--online", "--use-desc-for-id", "config.yml"])
+
+    assert result.exit_code == 0
+    mock_DBTCloud.assert_called_once()
+    assert mock_DBTCloud.call_args.kwargs["use_desc_for_id"] is True
+
+
+@patch("dbt_jobs_as_code.main.DBTCloud")
+def test_use_desc_for_id_option_import_jobs(mock_DBTCloud):
+    """Test that import-jobs passes use_desc_for_id=True to DBTCloud"""
+    instance = mock_DBTCloud.return_value
+    instance.get_jobs.return_value = []
+    instance.get_env_vars.return_value = {}
+
+    runner = CliRunner()
+    result = runner.invoke(
+        cli,
+        ["import-jobs", "--account-id", "789", "--use-desc-for-id"],
+    )
+
+    assert result.exit_code == 0
+    mock_DBTCloud.assert_called_once()
+    assert mock_DBTCloud.call_args.kwargs["use_desc_for_id"] is True
+
+
+@patch("dbt_jobs_as_code.main.DBTCloud")
+@patch("dbt_jobs_as_code.main.load_job_configuration")
+@patch("dbt_jobs_as_code.main.resolve_file_paths")
+def test_use_desc_for_id_option_link(
+    mock_resolve_file_paths, mock_load_job_configuration, mock_DBTCloud
+):
+    """Test that link passes use_desc_for_id=True to DBTCloud"""
+    from dbt_jobs_as_code.schemas.common_types import Settings, Triggers
+    from dbt_jobs_as_code.schemas.job import JobDefinition
+
+    mock_resolve_file_paths.return_value = (["config.yml"], [])
+
+    job = JobDefinition(
+        project_id=123,
+        environment_id=456,
+        account_id=789,
+        name="Test Job",
+        settings=Settings(threads=4),
+        run_generate_sources=False,
+        execute_steps=["dbt run"],
+        generate_docs=False,
+        schedule={"cron": "0 * * * *"},
+        triggers=Triggers(schedule=True),
+    )
+    mock_config = Mock()
+    mock_config.jobs = {"test-job": job}
+    mock_load_job_configuration.return_value = mock_config
+
+    runner = CliRunner()
+    result = runner.invoke(cli, ["link", "--dry-run", "--use-desc-for-id", "config.yml"])
+
+    assert result.exit_code == 0
+    mock_DBTCloud.assert_called_once()
+    assert mock_DBTCloud.call_args.kwargs["use_desc_for_id"] is True
+
+
+@patch("dbt_jobs_as_code.main.DBTCloud")
+def test_use_desc_for_id_option_unlink(mock_DBTCloud):
+    """Test that unlink passes use_desc_for_id=True to DBTCloud"""
+    instance = mock_DBTCloud.return_value
+    instance.get_jobs.return_value = []
+
+    runner = CliRunner()
+    result = runner.invoke(
+        cli,
+        ["unlink", "--account-id", "789", "--use-desc-for-id"],
+    )
+
+    assert result.exit_code == 0
+    mock_DBTCloud.assert_called_once()
+    assert mock_DBTCloud.call_args.kwargs["use_desc_for_id"] is True
+
+
+@patch("dbt_jobs_as_code.main.DBTCloud")
+def test_use_desc_for_id_option_deactivate_jobs(mock_DBTCloud):
+    """Test that deactivate-jobs passes use_desc_for_id=True to DBTCloud"""
+    instance = mock_DBTCloud.return_value
+    instance.get_jobs.return_value = []
+
+    runner = CliRunner()
+    result = runner.invoke(
+        cli,
+        ["deactivate-jobs", "--account-id", "789", "--use-desc-for-id"],
+    )
+
+    assert result.exit_code == 0
+    mock_DBTCloud.assert_called_once()
+    assert mock_DBTCloud.call_args.kwargs["use_desc_for_id"] is True