💄 style(tests): remove extra blank lines and fix formatting

Author: Jannchie

tests/unit/test_changelog.py

@@ -1160,25 +1160,19 @@ def test_tgit_commit_with_bytes_message_encoding_error(self):
         """Test TGITCommit with bytes message that has encoding issues."""
         mock_repo = Mock()
         mock_repo.git.rev_parse.return_value = "abc1234"
-        
+
         mock_commit = Mock()
         mock_commit.message = "feat: test message"  # Use valid UTF-8 string since bytes handling is in get_commits
         mock_commit.author.name = "Test Author"
         mock_commit.author.email = "test@example.com"
         mock_commit.committed_datetime = Mock()
         mock_commit.hexsha = "abcdef123456"
-        
-        message_dict = {
-            "emoji": "✨",
-            "type": "feat",
-            "scope": None,
-            "description": "test message",
-            "breaking": None
-        }
-        
+
+        message_dict = {"emoji": "✨", "type": "feat", "scope": None, "description": "test message", "breaking": None}
+
         # This should handle the commit creation properly
         tgit_commit = TGITCommit(mock_repo, mock_commit, message_dict)
-        
+
         # Should not raise an exception and should have the correct properties
         assert tgit_commit.type == "feat"
         assert tgit_commit.description == "test message"
@@ -1188,10 +1182,10 @@ def test_get_remote_uri_safe_basic(self):
         """Test _get_remote_uri_safe basic functionality."""
         mock_repo = Mock()
         mock_repo.remote.side_effect = ValueError("No remote found")
-        
+
         # Should return None when no remote is found
         result = _get_remote_uri_safe(mock_repo)
-        
+
         assert result is None
 
     def test_extract_latest_tag_from_changelog_malformed_file(self, tmp_path):
@@ -1203,9 +1197,9 @@ def test_extract_latest_tag_from_changelog_malformed_file(self, tmp_path):
 This is not a proper changelog
 No version tags here
 """)
-        
+
         result = extract_latest_tag_from_changelog(str(changelog_file))
-        
+
         # Should return None for malformed changelog
         assert result is None
 
@@ -1214,7 +1208,7 @@ def test_commit_pattern_with_unicode_message(self):
         # Test with unicode characters in commit message
         message = "feat: add 中文 支持 for internationalization"
         match = commit_pattern.match(message)
-        
+
         assert match is not None
         assert match.group("type") == "feat"
         assert "中文 支持" in match.group("description")

tests/unit/test_commit.py

@@ -561,28 +561,28 @@ class TestCommitErrorHandling:
     def test_handle_commit_no_message(self, mock_get_ai_command):
         """Test handle_commit with no message provided."""
         args = CommitArgs(message=[], emoji=False, breaking=False, ai=False)
-        
+
         # Mock get_ai_command to return None (no AI command generated)
         mock_get_ai_command.return_value = None
-        
+
         # This should call get_ai_command and return early when it returns None
         handle_commit(args)
-        
+
         # Should have called get_ai_command
         mock_get_ai_command.assert_called_once()
 
     def test_get_file_change_sizes_binary_files(self):
         """Test get_file_change_sizes with binary files."""
         mock_repo = Mock()
         mock_repo.git.diff.return_value = "-\t-\timage.png\n5\t0\tfile.py\n-\t-\tbinary.dat"
-        
+
         result = get_file_change_sizes(mock_repo)
-        
+
         # Should handle binary files (marked with -/-) as 0 size
         expected = {
             "image.png": 0,  # Binary files treated as 0 size
             "file.py": 5,
-            "binary.dat": 0
+            "binary.dat": 0,
         }
         assert result == expected
 
@@ -593,9 +593,9 @@ def test_get_changed_files_from_status_renamed_files(self, mock_repo_class):
         mock_repo.git.diff.return_value = """R100\told_name.py\tnew_name.py
 R100\tdir/old.js\tdir/new.js
 M\tmodified.py"""
-        
+
         result = get_changed_files_from_status(mock_repo)
-        
+
         # Should include both old and new names of renamed files and modified files
         expected = {"old_name.py", "new_name.py", "dir/old.js", "dir/new.js", "modified.py"}
         assert result == expected

tests/unit/test_utils.py

@@ -255,9 +255,9 @@ def test_load_settings_basic_functionality(self, tmp_path):
         tgit_dir.mkdir()
         config_file = tgit_dir / "settings.json"
         config_file.write_text(json.dumps({"apiKey": "test_key", "model": "gpt-4"}))
-        
+
         with patch("tgit.utils.Path.cwd", return_value=tmp_path):
             result = load_workspace_settings()
-            
+
             assert result["apiKey"] == "test_key"
             assert result["model"] == "gpt-4"