summaryrefslogtreecommitdiff
path: root/pkgs/development/python-modules/beetcamp/remove-git-pytest-option.diff
blob: e61a8e690687529a73a81b11edcebfbb4f7af562 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
The test suite has support for comparing results against a base revision of the
repository.

This requires that we run the tests from a Git checkout of the `beetcamp` repo,
which we do not do. We don't want to compare against a base revision, so we
just remove the option entirely.

diff --git a/tests/conftest.py b/tests/conftest.py
index 04d81f66f0..018d9e3c0c 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -9,7 +9,6 @@
 
 import pytest
 from beets.autotag.hooks import AlbumInfo, TrackInfo
-from git import Repo
 from rich_tables.diff import pretty_diff
 from rich_tables.utils import make_console
 
@@ -17,10 +16,8 @@
 from beetsplug.bandcamp.helpers import Helpers
 
 if TYPE_CHECKING:
-    from _pytest.config import Config
     from _pytest.config.argparsing import Parser
     from _pytest.fixtures import SubRequest
-    from _pytest.terminal import TerminalReporter
     from rich.console import Console
 
 
@@ -29,28 +26,6 @@
 
 
 def pytest_addoption(parser: Parser) -> None:
-    newest_folders = sorted(
-        (p for p in Path("lib_tests").glob("*") if p.is_dir()),
-        key=lambda p: p.stat().st_ctime,
-        reverse=True,
-    )
-    all_names = [f.name for f in newest_folders]
-    names = [n for n in all_names if n != "dev"]
-    names_set = set(names)
-
-    base_name = ""
-    for commit in Repo(".").iter_commits(paths=["./beetsplug"]):
-        short_commit = str(commit)[:8]
-        if short_commit in names_set:
-            base_name = short_commit
-            break
-
-    parser.addoption(
-        "--base",
-        choices=all_names,
-        default=base_name or "dev",
-        help="base directory / comparing against",
-    )
     parser.addoption(
         "--target",
         default="dev",
@@ -64,16 +39,6 @@
     )
 
 
-def pytest_terminal_summary(
-    terminalreporter: TerminalReporter,
-    exitstatus: int,  # noqa: ARG001
-    config: Config,
-) -> None:
-    base = config.getoption("base")
-    target = config.getoption("target")
-    terminalreporter.write(f"--- Compared {target} against {base} ---\n")
-
-
 def pytest_assertrepr_compare(op: str, left: Any, right: Any):  # noqa: ARG001
     """Pretty print the difference between dict objects."""
     actual, expected = left, right