Let ruff fix things

Author: jerry-git

src/pytest_split/plugin.py

@@ -76,7 +76,7 @@ def pytest_addoption(parser: "Parser") -> None:
     )
 
 
-@pytest.mark.tryfirst
+@pytest.mark.tryfirst()
 def pytest_cmdline_main(config: "Config") -> "Optional[Union[int, ExitCode]]":
     """
     Validate options.
@@ -135,9 +135,7 @@ def __init__(self, config: "Config") -> None:
         # from saving durations in a list-of-lists to a dict format
         # Remove this when bumping to v1
         if isinstance(self.cached_durations, list):
-            self.cached_durations = {
-                test_name: duration for test_name, duration in self.cached_durations
-            }
+            self.cached_durations = dict(self.cached_durations)
 
 
 class PytestSplitPlugin(Base):
@@ -182,7 +180,6 @@ def pytest_collection_modifyitems(
                 f"[pytest-split] Running group {group_idx}/{splits} (estimated duration: {group.duration:.2f}s)\n"
             )
         )
-        return None
 
 
 class PytestSplitCachePlugin(Base):

tests/test_algorithms.py

@@ -6,6 +6,7 @@
 
 if TYPE_CHECKING:
     from typing import List, Set
+
     from _pytest.nodes import Item
 
 from pytest_split.algorithms import Algorithms
@@ -17,7 +18,7 @@ class TestAlgorithms:
     @pytest.mark.parametrize("algo_name", Algorithms.names())
     def test__split_test(self, algo_name):
         durations = {"a": 1, "b": 1, "c": 1}
-        items = [item(x) for x in durations.keys()]
+        items = [item(x) for x in durations]
         algo = Algorithms[algo_name].value
         first, second, third = algo(splits=3, items=items, durations=durations)
 
@@ -70,7 +71,7 @@ def test__split_test_handles_large_duration_at_end(self):
         assert second.selected == [item(x) for x in ["a", "b", "c"]]
 
     @pytest.mark.parametrize(
-        "algo_name, expected",
+        ("algo_name", "expected"),
         [
             ("duration_based_chunks", [[item("a"), item("b")], [item("c"), item("d")]]),
             ("least_duration", [[item("a"), item("c")], [item("b"), item("d")]]),
@@ -94,7 +95,7 @@ def test__split_tests_calculates_avg_test_duration_only_on_present_tests(
         assert second.selected == expected_second
 
     @pytest.mark.parametrize(
-        "algo_name, expected",
+        ("algo_name", "expected"),
         [
             (
                 "duration_based_chunks",

tests/test_cli.py

@@ -5,11 +5,10 @@
 from unittest.mock import patch
 
 import pytest
-
 from pytest_split import cli
 
 
-@pytest.fixture
+@pytest.fixture()
 def durations_file(tmpdir):
     durations_path = str(tmpdir.join(".durations"))
     durations = {f"test_{i}": float(i) for i in range(1, 11)}

tests/test_ipynb.py

@@ -1,7 +1,6 @@
 from collections import namedtuple
 
 import pytest
-
 from pytest_split.algorithms import Algorithms
 from pytest_split.ipynb_compatibility import ensure_ipynb_compatibility
 
@@ -28,7 +27,7 @@ def test_ensure_ipynb_compatibility(self, algo_name):
             "temp/nbs/test_4.ipynb::Cell 1": 1,
             "temp/nbs/test_4.ipynb::Cell 2": 3,
         }
-        items = [item(x) for x in durations.keys()]
+        items = [item(x) for x in durations]
         algo = Algorithms[algo_name].value
         groups = algo(splits=3, items=items, durations=durations)
 

tests/test_plugin.py

@@ -10,18 +10,18 @@
 EXAMPLE_SUITE_TEST_COUNT = 10
 
 
-@pytest.fixture
+@pytest.fixture()
 def example_suite(testdir):
     testdir.makepyfile(
         "".join(
             f"def test_{num}(): pass\n"
             for num in range(1, EXAMPLE_SUITE_TEST_COUNT + 1)
         )
     )
-    yield testdir
+    return testdir
 
 
-@pytest.fixture
+@pytest.fixture()
 def durations_path(tmpdir):
     return str(tmpdir.join(".durations"))
 
@@ -74,7 +74,7 @@ def test_it_doesnt_remove_old_durations(self, example_suite, durations_path):
             durations = json.load(f)
 
         for item in old_durations:
-            assert item in durations.keys()
+            assert item in durations
         assert len(durations) == EXAMPLE_SUITE_TEST_COUNT + len(old_durations)
 
     def test_it_removes_old_when_cli_flag_used(self, example_suite, durations_path):
@@ -171,7 +171,7 @@ class TestSplitToSuites:
     enumerated_params = [(i, *param) for i, param in enumerate(all_params)]
 
     @pytest.mark.parametrize(
-        "test_idx, splits, group, algo, expected, legacy_flag", enumerated_params
+        ("test_idx", "splits", "group", "algo", "expected", "legacy_flag"), enumerated_params
     )
     def test_it_splits(  # noqa: PLR0913
         self,