ensure --batch and --logfile are distinct files

Motivation: to avoid some kind of infinite loop, in which mycli reads
from a file which is constantly being written to.

Author: rolandwalker

changelog.md

@@ -12,6 +12,11 @@ Breaking Changes
 * Remove support for `.myclirc` files in the current working directory.
 
 
+Bug Fixes
+---------
+* Ensure that `--batch` and `--logfile` files are distinct.
+
+
 Documentation
 ---------
 * Show section when recommending `default_character_set` setting.

mycli/main.py

@@ -327,6 +327,17 @@ def preprocess_cli_args(
             click.secho('Error: --batch and --checkpoint must be different files.', err=True, fg='red')
             sys.exit(1)
 
+    if (
+        cli_args.logfile
+        and os.path.exists(cli_args.logfile.name)
+        and cli_args.batch
+        and cli_args.batch != '-'
+        and os.path.exists(cli_args.batch)
+    ):
+        if os.stat(cli_args.batch) == os.stat(cli_args.logfile.name):
+            click.secho('Error: --batch and --logfile must be different files.', err=True, fg='red')
+            sys.exit(1)
+
     if cli_args.verbose and cli_args.quiet:
         click.secho('Error: --verbose and --quiet are incompatible.', err=True, fg='red')
         sys.exit(1)

test/pytests/test_main.py

@@ -2449,6 +2449,26 @@ def test_preprocess_cli_args_rejects_same_batch_and_checkpoint_file(
     assert 'Error: --batch and --checkpoint must be different files.' in capsys.readouterr().err
 
 
+def test_preprocess_cli_args_rejects_same_batch_and_logfile(
+    capsys: pytest.CaptureFixture[str],
+    tmp_path: Path,
+) -> None:
+    batch_path = tmp_path / 'batch.sql'
+    batch_path.write_text('select 1;\n', encoding='utf-8')
+    cli_args = CliArgs()
+    cli_args.batch = str(batch_path)
+    cli_args.logfile = batch_path.open('a', encoding='utf-8')  # type: ignore[assignment]
+
+    try:
+        with pytest.raises(SystemExit) as excinfo:
+            preprocess_cli_args(cli_args, valid_connection_scheme)
+    finally:
+        cli_args.logfile.close()
+
+    assert excinfo.value.code == 1
+    assert 'Error: --batch and --logfile must be different files.' in capsys.readouterr().err
+
+
 def test_preprocess_cli_args_rejects_verbose_and_quiet(capsys: pytest.CaptureFixture[str]) -> None:
     cli_args = CliArgs()
     cli_args.verbose = 1