R-112: Add --diagnostics-warehouse CLI parameter (#2362)

* Add `--diagnostics-warehouse` CLI parameter

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: nielsn

soda-core/src/soda_core/cli/cli.py

@@ -153,6 +153,15 @@ def _setup_contract_verify_command(contract_parsers) -> None:
         help="Show more detailed logs on the console.",
     )
 
+    # TODO: move into extensions
+    verify_parser.add_argument(
+        "-dw",
+        "--diagnostics-warehouse",
+        type=str,
+        nargs="?",
+        help="Specify the path to the diagnostics warehouse configuration file. ",
+    )
+
     def handle(args):
         contract_file_paths = args.contract
         dataset_identifiers = args.dataset
@@ -165,6 +174,7 @@ def handle(args):
         verbose = args.verbose
         use_agent = args.use_agent
         blocking_timeout_in_minutes = args.blocking_timeout_in_minutes
+        diagnostics_warehouse_file_path = args.diagnostics_warehouse
 
         exit_code = handle_verify_contract(
             contract_file_paths,
@@ -176,6 +186,7 @@ def handle(args):
             verbose,
             use_agent,
             blocking_timeout_in_minutes,
+            diagnostics_warehouse_file_path,
         )
 
         exit_with_code(exit_code)

soda-core/src/soda_core/cli/handlers/contract.py

@@ -25,6 +25,7 @@ def handle_verify_contract(
     verbose: bool,
     use_agent: bool,
     blocking_timeout_in_minutes: int,
+    diagnostics_warehouse_file_path: Optional[str],
 ) -> ExitCode:
     try:
         contract_verification_result = verify_contracts(
@@ -38,6 +39,7 @@ def handle_verify_contract(
             verbose=verbose,
             use_agent=use_agent,
             blocking_timeout_in_minutes=blocking_timeout_in_minutes,
+            dwh_data_source_file_path=diagnostics_warehouse_file_path,
         )
 
         return interpret_contract_verification_result(contract_verification_result)

soda-tests/tests/components/test_cli.py

@@ -45,6 +45,7 @@
                 True,
                 True,
                 42,
+                None,
             ],
         ),
         (
@@ -76,6 +77,7 @@
                 True,
                 True,
                 42,
+                None,
             ],
         ),
         (
@@ -90,7 +92,34 @@
                 "-sc",
                 "cloud.yaml",
             ],
-            [None, ["some-dataset"], ["ds.yaml"], "cloud.yaml", {}, False, False, False, 60],
+            [None, ["some-dataset"], ["ds.yaml"], "cloud.yaml", {}, False, False, False, 60, None],
+        ),
+        (
+            [
+                "soda",
+                "contract",
+                "verify",
+                "-d",
+                "some-dataset",
+                "-ds",
+                "ds.yaml",
+                "-sc",
+                "cloud.yaml",
+                "-dw",
+                "diagnostics_warehouse.yaml",
+            ],
+            [
+                None,
+                ["some-dataset"],
+                ["ds.yaml"],
+                "cloud.yaml",
+                {},
+                False,
+                False,
+                False,
+                60,
+                "diagnostics_warehouse.yaml",
+            ],
         ),
     ],
 )

soda-tests/tests/components/test_cli_contract_handlers.py

@@ -52,6 +52,7 @@ def test_handle_verify_contract_exit_codes(
         verbose=False,
         use_agent=False,
         blocking_timeout_in_minutes=10,
+        diagnostics_warehouse_file_path=None,
     )
 
     assert exit_code == expected_exit_code