Add unit test for renamer.new_name_for

.github/workflows/run-tests.yml

@@ -0,0 +1,17 @@
+on:
+  push:
+    branches:
+      - "*"
+
+jobs:
+  test:
+    steps:
+    - uses: actions/checkout@v4
+    - uses: actions/setup-python@v5
+      with:
+        python-version: 3.11
+        cache: pip
+    - name: Install shroudstone
+      run: pip install -e .[test]
+    - name: Run tests
+      run: python -m pytest

tests/conftest.py

@@ -0,0 +1,26 @@
+from pathlib import Path
+from typing import NamedTuple
+
+import pytest
+
+data_dir = Path(__file__).parent / "replays"
+
+
+def pytest_addoption(parser):
+    parser.addoption("--update-golden", default=False, action="store_true")
+
+
+class ReplayCase(NamedTuple):
+    replay_file: Path
+    summary_file: Path
+    expected_name_file: Path
+
+
+@pytest.fixture(params=list(data_dir.glob("**/*.SGReplay")))
+def replay_case(request):
+    path = request.param
+    yield ReplayCase(
+        replay_file=path,
+        summary_file=path.with_suffix(".json"),
+        expected_name_file=path.with_suffix(".txt"),
+    )

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL44821-2024.02.03-18.48.txt

@@ -0,0 +1 @@
+2024-02-03 07.48 L 00m10s Pox IvV PeacefulBot - Boneyard.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL44821-2024.02.13-22.25.txt

@@ -0,0 +1 @@
+2024-02-13 11.25 L 04m16s Pox IvV PeacefulBot - JaggedMaw.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.02.22-16.42.txt

@@ -0,0 +1 @@
+2024-02-22 05.42 L 10m31s Pox IvV justasloth - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.02.29-21.25.txt

@@ -0,0 +1 @@
+2024-02-29 10.25 W 15m04s Pox IvV Frizy - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.02.29-21.53.txt

@@ -0,0 +1 @@
+2024-02-29 10.53 L 02m13s Pox IvI 你们怎么办，只有天知道 - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-00.02.txt

@@ -0,0 +1 @@
+2024-03-06 13.02 W 08m55s Pox IvV itsMerikh - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-00.12.txt

@@ -0,0 +1 @@
+2024-03-06 13.12 L 12m29s Pox IvV 물망초 - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-00.24.txt

@@ -0,0 +1 @@
+2024-03-06 13.24 W 14m49s Pox IvV ελευθερία - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-21.49.txt

@@ -0,0 +1 @@
+2024-03-07 10.49 L 10m33s Pox IvV TOP - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-21.59.txt

@@ -0,0 +1 @@
+2024-03-07 10.59 W 17m49s Pox IvV Sturgeon - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-22.20.txt

@@ -0,0 +1 @@
+2024-03-07 11.20 W 03m39s Pox IvI thatguy - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-22.41.txt

@@ -0,0 +1 @@
+2024-03-07 11.41 W 08m14s Pox IvI indyrem - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-23.02.txt

@@ -0,0 +1 @@
+2024-03-07 12.02 W 12m13s Pox IvI SkWizZ_SSSylas - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-23.15.txt

@@ -0,0 +1 @@
+2024-03-07 12.15 W 04m15s Pox IvI leviotus - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-23.21.txt

@@ -0,0 +1 @@
+2024-03-07 12.21 L 15m13s Pox IvV albino - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.07-23.57.txt

@@ -0,0 +1 @@
+2024-03-07 12.57 L 10m41s Pox IvV PrimeScout - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.10-17.05.txt

@@ -0,0 +1 @@
+2024-03-10 06.05 W 06m29s Pox IvV Dutchy - BrokenCrown.SGReplay

tests/replays/c6b4eb4e-4994-4e96-b098-3e1953a02033/CL45737-2024.03.10-17.12.txt

@@ -0,0 +1 @@
+2024-03-10 06.12 W 03m39s Pox IvI Warm Milk - BrokenCrown.SGReplay

tests/test_renamer.py

@@ -0,0 +1,18 @@
+from shroudstone.renamer import Replay, new_name_for
+from tests.conftest import ReplayCase
+
+f1v1 = "{time:%Y-%m-%d %H.%M} {result:.1} {duration} {us} {f1:.1}v{f2:.1} {them} - {map_name}.SGReplay"
+fgeneric = (
+    "{time:%Y-%m-%d %H.%M} {duration} {players_with_factions} - {map_name}.SGReplay"
+)
+
+
+def test_replay_renaming(replay_case: ReplayCase, request):
+    replay = Replay.from_path(replay_case.replay_file)
+    assert replay is not None
+    new_name = new_name_for(replay, format_1v1=f1v1, format_generic=fgeneric)
+    assert isinstance(new_name, str)
+    if request.config.getoption("--update-golden"):
+        replay_case.expected_name_file.write_text(new_name)
+    else:
+        assert new_name == replay_case.expected_name_file.read_text()