From 6faffc1ea8b58f00afc91f1ecb3cf3925611cbd8 Mon Sep 17 00:00:00 2001 From: Simone Scarduzio Date: Mon, 29 Sep 2025 16:10:06 +0200 Subject: [PATCH] style: Fix linting issues in recursive delete test file MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Fix import ordering - Remove boolean equality comparison - Add missing newline at end of file All ruff and mypy checks now pass. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- .gitignore | 1 + .../integration/test_recursive_delete_reference_cleanup.py | 7 +++++-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/.gitignore b/.gitignore index d89c05f..5e89eef 100644 --- a/.gitignore +++ b/.gitignore @@ -1,4 +1,5 @@ # Python +ror-data-importer/ __pycache__/ *.py[cod] *$py.class diff --git a/tests/integration/test_recursive_delete_reference_cleanup.py b/tests/integration/test_recursive_delete_reference_cleanup.py index af5081e..9e80f26 100644 --- a/tests/integration/test_recursive_delete_reference_cleanup.py +++ b/tests/integration/test_recursive_delete_reference_cleanup.py @@ -1,6 +1,7 @@ """Focused tests for recursive delete reference cleanup functionality.""" from unittest.mock import Mock, patch + import pytest from deltaglider.app.cli.main import create_service @@ -161,6 +162,7 @@ class TestRecursiveDeleteReferenceCleanup: def test_cli_uses_core_service_method(self): """Test that CLI rm -r command uses the core service delete_recursive method.""" from click.testing import CliRunner + from deltaglider.app.cli.main import cli runner = CliRunner() @@ -188,6 +190,7 @@ class TestRecursiveDeleteReferenceCleanup: def test_cli_dryrun_does_not_call_delete_recursive(self): """Test that CLI dryrun does not call the actual delete_recursive method.""" from click.testing import CliRunner + from deltaglider.app.cli.main import cli runner = CliRunner() @@ -232,7 +235,7 @@ class TestRecursiveDeleteReferenceCleanup: from deltaglider.core import ObjectKey result = service.delete(ObjectKey(bucket="test-bucket", key="test/file.zip.delta")) - assert result["deleted"] == True + assert result["deleted"] assert result["type"] == "delta" def test_reference_cleanup_intelligence_basic(self): @@ -307,4 +310,4 @@ class TestRecursiveDeleteReferenceCleanup: if __name__ == "__main__": - pytest.main([__file__, "-v"]) \ No newline at end of file + pytest.main([__file__, "-v"])