Merge pull request #2 from kintsugi-tax/bug/no-assume-transaction

stephen-kintsugi · web-flow · commit 5fd544a5c0ef · 2026-02-27T14:03:30.000-06:00
Don't assume in transaction
diff --git a/squawk_alembic/hook.py b/squawk_alembic/hook.py
@@ -220,7 +220,7 @@ def main():
 
         try:
             result = subprocess.run(
-                ["squawk", "--assume-in-transaction", tmp_path],
+                ["squawk", tmp_path],
                 capture_output=True,
                 text=True,
             )
diff --git a/tests/test_main.py b/tests/test_main.py
@@ -99,7 +99,6 @@ def upgrade():
         assert "def456:abc123" in alembic_call
         squawk_call = mock_run.call_args_list[1][0][0]
         assert squawk_call[0] == "squawk"
-        assert "--assume-in-transaction" in squawk_call
 
 
 def test_squawk_failure(repo, capsys):

Original file line number	Diff line number	Diff line change
`@@ -220,7 +220,7 @@ def main():`
`220`	`220`
`221`	`221`	`try:`
`222`	`222`	`result = subprocess.run(`
`223`		`- ["squawk", "--assume-in-transaction", tmp_path],`
	`223`	`+ ["squawk", tmp_path],`
`224`	`224`	`capture_output=True,`
`225`	`225`	`text=True,`
`226`	`226`	`)`