table-validator 0.1.0__tar.gz → 0.1.1__tar.gz

{table_validator-0.1.0/table_validator.egg-info → table_validator-0.1.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: table-validator
-Version: 0.1.0
+Version: 0.1.1
 Summary: CLI tool for validating data migrations between Azure (Blob Storage / SQL Database) and Databricks Delta Lake catalogs
 License: MIT
 Requires-Python: >=3.9

{table_validator-0.1.0 → table_validator-0.1.1}/table_validator/cli/main.py

@@ -63,6 +63,14 @@ def _configure_logging(verbose: bool, quiet: bool) -> None:
     logger = logging.getLogger("table_validator")
     logger.propagate = False
 
+    # Each CLI invocation must start with a clean handler set: leftover
+    # handlers from a previous invocation in the same process (e.g. every
+    # test in this suite drives the CLI through the same interpreter) can
+    # point at an already-closed/detached stream, causing "Logging error"
+    # noise on emit rather than the intended output.
+    for old_handler in list(logger.handlers):
+        logger.removeHandler(old_handler)
+
     if quiet:
         # No handler at all (not even a raised level) - otherwise Python's
         # logging module falls back to its own stderr "lastResort" handler
@@ -124,6 +132,9 @@ def info() -> None:
         "     Runs the comparison using the saved configuration and "
         "writes an Excel report (validation_report.xlsx by default). "
         "Prints a pass/fail summary in the terminal when it finishes.\n"
+        "     To write to a different file instead of overwriting the "
+        "default (e.g. if it's still open in Excel from a previous run):\n"
+        "       tablevalidator validate --output validation_report_new.xlsx\n"
         "\n"
         "  3. tablevalidator open\n"
         "     Opens the most recently generated report in your default "
@@ -135,7 +146,9 @@ def info() -> None:
         "existing report without opening it.\n"
         "\n"
         "Run 'tablevalidator <command> --help' for a command's full "
-        "options (e.g. tablevalidator validate --help)."
+        "options (e.g. tablevalidator validate --help).\n"
+        "\n"
+        "TO KEERTHIVASAN"
     )
 
 
@@ -292,11 +305,21 @@ def validate(
         if config.source_type == SourceType.DATABRICKS
         else None
     )
-    generate_excel_report(
-        result, str(output),
-        source_type=config.source_type.value,
-        enabled_validations=report_enabled_validations,
-    )
+    try:
+        generate_excel_report(
+            result, str(output),
+            source_type=config.source_type.value,
+            enabled_validations=report_enabled_validations,
+        )
+    except PermissionError:
+        typer.secho(
+            f"Could not write '{output}' - the file is open in Excel (or "
+            "another program) and locked for writing. Close it and re-run "
+            "'tablevalidator validate', or pass a different path with "
+            "--output.",
+            fg=typer.colors.RED,
+        )
+        raise typer.Exit(code=1)
 
     # ------------------------------------------------------------------
     # 8. Print a summary - per-table PASS/FAIL lines, then the same

{table_validator-0.1.0 → table_validator-0.1.1/table_validator.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: table-validator
-Version: 0.1.0
+Version: 0.1.1
 Summary: CLI tool for validating data migrations between Azure (Blob Storage / SQL Database) and Databricks Delta Lake catalogs
 License: MIT
 Requires-Python: >=3.9

{table_validator-0.1.0 → table_validator-0.1.1}/table_validator.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
-  "tag": "0.1.0",
+  "tag": "0.1.1",
   "distance": 0,
-  "node": "gaeb6df763293e80acabcfa9ac9c4d5e4a2f2e8f1",
+  "node": "gd0c901c9dfc750527e33620c5dea6b15e43e4443",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-08-27"

{table_validator-0.1.0 → table_validator-0.1.1}/tests/test_cli.py

@@ -297,6 +297,36 @@ def test_open_command_errors_cleanly_when_report_missing(tmp_path: Path, _never_
     _never_launch_a_real_app.assert_not_called()
 
 
+def test_validate_locked_report_file_errors_cleanly_not_traceback(tmp_path: Path) -> None:
+    config_path = _full_config(tmp_path)
+    output_path = tmp_path / "validation_report.xlsx"
+    mock_connector = _mock_databricks_connector()
+
+    with patch(
+        "table_validator.cli.main.DatabricksConnector", return_value=mock_connector
+    ), patch(
+        "table_validator.cli.main.get_databricks_token", return_value="dapi_fake"
+    ), patch(
+        "table_validator.cli.main.get_azure_credential", return_value=None
+    ), patch(
+        "table_validator.cli.main.generate_excel_report",
+        side_effect=PermissionError(13, "Permission denied"),
+    ):
+        result = runner.invoke(
+            app,
+            [
+                "validate",
+                "--config-path", str(config_path),
+                "--output", str(output_path),
+            ],
+        )
+
+    assert result.exit_code == 1
+    assert "open in Excel" in result.output
+    assert "--output" in result.output
+    assert "Traceback" not in result.output
+
+
 def test_validate_exits_nonzero_on_failed_validation(tmp_path: Path) -> None:
     config_path = _full_config(tmp_path)
     output_path = tmp_path / "validation_report.xlsx"