remove useless monkeypatch parameter

Author: SandrineP

test/test_commit.py

@@ -4,9 +4,7 @@
 
 
 @pytest.mark.parametrize("all_flag", ["", "-A", "--all", "--no-ignore-removal"])
-def test_commit(
-    xtl_clone, commit_env_config, git2cpp_path, tmp_path, monkeypatch, all_flag
-):
+def test_commit(xtl_clone, commit_env_config, git2cpp_path, tmp_path, all_flag):
     assert (tmp_path / "xtl").exists()
     xtl_path = tmp_path / "xtl"
 

test/test_log.py

@@ -4,9 +4,7 @@
 
 
 @pytest.mark.parametrize("format_flag", ["", "--format=full", "--format=fuller"])
-def test_log(
-    xtl_clone, commit_env_config, git2cpp_path, tmp_path, monkeypatch, format_flag
-):
+def test_log(xtl_clone, commit_env_config, git2cpp_path, tmp_path, format_flag):
     assert (tmp_path / "xtl").exists()
     xtl_path = tmp_path / "xtl"
 
@@ -47,7 +45,7 @@ def test_log_nogit(commit_env_config, git2cpp_path, tmp_path):
 
 @pytest.mark.parametrize("max_count_flag", ["", "-n", "--max-count"])
 def test_max_count(
-    xtl_clone, commit_env_config, git2cpp_path, tmp_path, monkeypatch, max_count_flag
+    xtl_clone, commit_env_config, git2cpp_path, tmp_path, max_count_flag
 ):
     assert (tmp_path / "xtl").exists()
     xtl_path = tmp_path / "xtl"

test/test_merge.py

@@ -5,9 +5,7 @@
 
 # TODO: Have a different "person" for the commit and for the merge
 # TODO: Test "unborn" case, but how ?
-def test_merge_fast_forward(
-    xtl_clone, commit_env_config, git2cpp_path, tmp_path, monkeypatch
-):
+def test_merge_fast_forward(xtl_clone, commit_env_config, git2cpp_path, tmp_path):
     assert (tmp_path / "xtl").exists()
     xtl_path = tmp_path / "xtl"
 
@@ -54,9 +52,7 @@ def test_merge_fast_forward(
     assert p_merge_2.stdout == "Already up-to-date\n"
 
 
-def test_merge_commit(
-    xtl_clone, commit_env_config, git2cpp_path, tmp_path, monkeypatch
-):
+def test_merge_commit(xtl_clone, commit_env_config, git2cpp_path, tmp_path):
     assert (tmp_path / "xtl").exists()
     xtl_path = tmp_path / "xtl"
 
@@ -118,9 +114,7 @@ def test_merge_commit(
 
 
 @pytest.mark.parametrize("flag", ["--abort", "--quit", "--continue"])
-def test_merge_conflict(
-    xtl_clone, commit_env_config, git2cpp_path, tmp_path, monkeypatch, flag
-):
+def test_merge_conflict(xtl_clone, commit_env_config, git2cpp_path, tmp_path, flag):
     assert (tmp_path / "xtl").exists()
     xtl_path = tmp_path / "xtl"
 

test/test_remote.py

@@ -321,7 +321,9 @@ def test_fetch_depth(git2cpp_path, tmp_path, run_in_tmp_path):
     url = "https://github.com/xtensor-stack/xtl.git"
 
     invalid_clone_cmd = [git2cpp_path, "clone", "--depth", "0", url]
-    p_invalid_clone = subprocess.run(invalid_clone_cmd, capture_output=True, cwd=tmp_path, text=True)
+    p_invalid_clone = subprocess.run(
+        invalid_clone_cmd, capture_output=True, cwd=tmp_path, text=True
+    )
     assert p_invalid_clone.returncode == 0
     assert p_invalid_clone.stdout.startswith("fatal: depth 0 is not a positive number")
 
@@ -418,7 +420,7 @@ def test_remote_show_in_cloned_repo(xtl_clone, git2cpp_path, tmp_path):
     assert "http" in p.stdout or "git" in p.stdout or "https" in p.stdout
 
 
-def test_push_local(xtl_clone, commit_env_config, git2cpp_path, tmp_path, monkeypatch):
+def test_push_local(xtl_clone, commit_env_config, git2cpp_path, tmp_path):
     """Test setting push on a local remote."""
     assert (tmp_path / "xtl").exists()
     xtl_path = tmp_path / "xtl"

test/test_reset.py

@@ -3,7 +3,7 @@
 import pytest
 
 
-def test_reset(xtl_clone, commit_env_config, git2cpp_path, tmp_path, monkeypatch):
+def test_reset(xtl_clone, commit_env_config, git2cpp_path, tmp_path):
     assert (tmp_path / "xtl").exists()
     xtl_path = tmp_path / "xtl"
 

test/test_revlist.py

@@ -3,7 +3,7 @@
 import pytest
 
 
-def test_revlist(xtl_clone, commit_env_config, git2cpp_path, tmp_path, monkeypatch):
+def test_revlist(xtl_clone, commit_env_config, git2cpp_path, tmp_path):
     assert (tmp_path / "xtl").exists()
     xtl_path = tmp_path / "xtl"