Fix linting errors in test suite

Author: Justar96

tests/conftest.py

@@ -3,9 +3,10 @@
 This module provides reusable test data and fixtures following pytest best practices.
 """
 
-import pytest
 from typing import Any, Dict, List
 
+import pytest
+
 
 # Simple test data fixtures
 @pytest.fixture

tests/test_api.py

@@ -11,8 +11,9 @@
 """
 
 import pytest
-from toon_format import encode, decode, ToonDecodeError
-from toon_format.types import EncodeOptions, DecodeOptions
+
+from toon_format import ToonDecodeError, decode, encode
+from toon_format.types import DecodeOptions, EncodeOptions
 
 
 class TestEncodeAPI:

tests/test_cli.py

@@ -1,9 +1,7 @@
 """Integration tests for the CLI module."""
 
 import json
-import sys
 from io import StringIO
-from pathlib import Path
 from unittest.mock import MagicMock, patch
 
 import pytest
@@ -239,7 +237,7 @@ def test_custom_delimiter_option(self, tmp_path):
         input_file = tmp_path / "input.json"
         input_file.write_text('{"items": [1, 2, 3]}')
 
-        with patch("sys.stdout", new_callable=StringIO) as mock_stdout:
+        with patch("sys.stdout", new_callable=StringIO):
             with patch("sys.argv", ["toon", str(input_file), "--encode", "--delimiter", "|"]):
                 result = main()
                 assert result == 0
@@ -249,7 +247,7 @@ def test_custom_indent_option(self, tmp_path):
         input_file = tmp_path / "input.json"
         input_file.write_text('{"outer": {"inner": 1}}')
 
-        with patch("sys.stdout", new_callable=StringIO) as mock_stdout:
+        with patch("sys.stdout", new_callable=StringIO):
             with patch("sys.argv", ["toon", str(input_file), "--encode", "--indent", "4"]):
                 result = main()
                 assert result == 0
@@ -259,7 +257,7 @@ def test_length_marker_option(self, tmp_path):
         input_file = tmp_path / "input.json"
         input_file.write_text('{"items": [1, 2, 3]}')
 
-        with patch("sys.stdout", new_callable=StringIO) as mock_stdout:
+        with patch("sys.stdout", new_callable=StringIO):
             with patch("sys.argv", ["toon", str(input_file), "--encode", "--length-marker"]):
                 result = main()
                 assert result == 0
@@ -269,7 +267,7 @@ def test_no_strict_option(self, tmp_path):
         input_file = tmp_path / "input.toon"
         input_file.write_text("name: Test")
 
-        with patch("sys.stdout", new_callable=StringIO) as mock_stdout:
+        with patch("sys.stdout", new_callable=StringIO):
             with patch("sys.argv", ["toon", str(input_file), "--decode", "--no-strict"]):
                 result = main()
                 assert result == 0
@@ -307,7 +305,7 @@ def test_error_during_encoding(self, tmp_path):
     def test_error_reading_input(self):
         """Test error when reading input fails."""
         mock_stdin = MagicMock()
-        mock_stdin.read.side_effect = IOError("Read failed")
+        mock_stdin.read.side_effect = OSError("Read failed")
 
         with patch("sys.stdin", mock_stdin):
             with patch("sys.stderr", new_callable=StringIO) as mock_stderr:

tests/test_decoder.py

@@ -39,14 +39,14 @@ def test_decode_returns_python_dict(self):
         toon = "id: 123"
         result = decode(toon)
         assert isinstance(result, dict)
-        assert type(result) == dict  # Not a subclass
+        assert type(result) is dict  # Not a subclass
 
     def test_decode_returns_python_list(self):
         """Ensure decode returns native Python list for arrays."""
         toon = "[3]: 1,2,3"
         result = decode(toon)
         assert isinstance(result, list)
-        assert type(result) == list  # Not a subclass
+        assert type(result) is list  # Not a subclass
 
 
 class TestPythonErrorHandling:

tests/test_encoder.py

@@ -30,7 +30,7 @@ def test_encode_returns_python_str(self):
         """Ensure encode returns native Python str, not bytes or custom type."""
         result = encode({"id": 123})
         assert isinstance(result, str)
-        assert type(result) == str  # Not a subclass
+        assert type(result) is str  # Not a subclass
 
     def test_encode_handles_none_gracefully(self):
         """Test encoding None doesn't crash (Python-specific edge case)."""

tests/test_internationalization.py

@@ -4,7 +4,6 @@
 TOON specification Section 16 (Internationalization).
 """
 
-import pytest
 
 from toon_format import decode, encode
 

tests/test_normalization.py

@@ -16,8 +16,8 @@
 official fixtures from https://github.com/toon-format/spec
 """
 
-import pytest
 from decimal import Decimal
+
 from toon_format import decode, encode
 
 

tests/test_normalize_functions.py

@@ -4,11 +4,9 @@
 full coverage of edge cases and error paths.
 """
 
-import sys
-from collections import OrderedDict, UserDict
+from collections import OrderedDict
 from datetime import date, datetime
 from decimal import Decimal
-from unittest.mock import MagicMock
 
 import pytest
 

tests/test_parsing_utils.py

@@ -327,6 +327,5 @@ def test_realistic_tabular_row_detection(self):
 
         # Row with quoted field containing colon
         row_with_quote = 'Alice,"30:manager",Engineer'
-        first_comma = find_unquoted_char(row_with_quote, ",")
         first_colon = find_unquoted_char(row_with_quote, ":")
         assert first_colon == -1  # Colon only in quotes = row

tests/test_security.py

@@ -4,8 +4,8 @@
 from the TOON specification Section 15.
 """
 
+
 import pytest
-import sys
 
 from toon_format import decode, encode
 from toon_format.types import DecodeOptions