refactor: rename dest param name

Author: darwintree

src/xdist/plugin.py

@@ -129,13 +129,16 @@ def pytest_addoption(parser: pytest.Parser) -> None:
     )
     group.addoption(
         "--no-loadscope-reorder",
-        action="store_true",
-        dest="noloadscopenoreorder",
-        default=False,
+        dest="loadscopereorder",
+        action="store_false",
+        default=True,
         help=(
-            "Reorders tests when used in conjunction with loadscope.\n"
-            "Will order tests by number of tests per scope as a best-effort"
-            " attempt to evenly distribute scopes across all workers."
+            "Pytest-xdist will default reorder tests by number of tests per scope "
+            "when used in conjunction with loadscope.\n"
+            "This option will disable loadscope reorder, "
+            "and the partial order of tests can be retained.\n"
+            "This is useful when pytest-xdist is used together with "
+            "other plugins that specify tests in a specific order."
         ),
     )
     group.addoption(

src/xdist/scheduler/loadscope.py

@@ -371,15 +371,15 @@ def schedule(self) -> None:
             work_unit = unsorted_workqueue.setdefault(scope, {})
             work_unit[nodeid] = False
 
-        if self.config.option.noloadscopenoreorder:
-            for scope, nodeids in unsorted_workqueue.items():
-                self.workqueue[scope] = nodeids
-        else:
+        if self.config.option.loadscopereorder:
             # Insert tests scopes into work queue ordered by number of tests.
             for scope, nodeids in sorted(
                 unsorted_workqueue.items(), key=lambda item: -len(item[1])
             ):
                 self.workqueue[scope] = nodeids
+        else:
+            for scope, nodeids in unsorted_workqueue.items():
+                self.workqueue[scope] = nodeids
 
         # Avoid having more workers than work
         extra_nodes = len(self.nodes) - len(self.workqueue)