Merge pull request #90 from isaacharrisholt/clear-flag

Ignore `!Clear:AutoSwitch` lines

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "quiffen"
-version = "2.0.10"
+version = "2.0.11"
 description = "Quiffen"
 authors = ["Isaac Harris-Holt <[email protected]>"]
 license = "GPL-3.0-or-later"

quiffen/core/qif.py

@@ -160,12 +160,20 @@ def parse(
                 continue
 
             # Allow for comments and blank lines at the top of sections
+            # Also skip `!Clear:` lines to ignore flags such as `!Clear:AutoSwitch`
             for i, line in enumerate(section_lines):
+                if (
+                    not line.strip()
+                    or line.strip().startswith("!Clear:")
+                    or line[0] == "#"
+                ):
+                    continue
+
+                # Found the header line
                 header_line = line
-                if line.strip() and line[0] != "#":
-                    line_number += i
-                    section_lines = section_lines[i:]
-                    break
+                line_number += i
+                section_lines = section_lines[i:]
+                break
             else:
                 # Empty section
                 continue

tests/test_files/test_clear_autoswitch.qif

@@ -0,0 +1,6 @@
+!Clear:AutoSwitch
+!Account
+NMy Bank Account
+D
+TBank
+^

tests/test_qif.py

@@ -39,6 +39,11 @@ def nonexistent_file():
     return Path(__file__).parent / "test_files" / "nonexistent.qif"
 
 
+@pytest.fixture
+def qif_file_with_clear_autoswitch():
+    return Path(__file__).parent / "test_files" / "test_clear_autoswitch.qif"
+
+
 def test_create_qif():
     """Test creating a Qif instance"""
     qif = Qif()
@@ -1101,3 +1106,14 @@ def test_transaction_account_type_qif():
 ^
 """
     )
+
+
+def test_clear_autoswitch_ignored(qif_file_with_clear_autoswitch):
+    """Tests that `!Clear:AutoSwitch` flag exported by Quicken and GNUCash
+    is ignored.
+
+    Relates to discussion #89.
+    """
+    qif = Qif.parse(qif_file_with_clear_autoswitch)
+    assert len(qif.accounts) == 1
+    assert list(qif.accounts.keys()) == ["My Bank Account"]

tox.ini

@@ -1,7 +1,7 @@
 [tox]
 requires =
     tox>=4
-envlist = py{38,39,310,311}, black, ruff, pyright, isort
+envlist = py{38,39,310,311,312}, black, ruff, pyright, isort
 isolated_build = true
 
 [gh-actions]
@@ -10,6 +10,7 @@ python =
     3.9: py39
     3.10: py310
     3.11: py311
+    3.12: py312
 
 [testenv]
 setenv =