Imply --parallel when --parallel-no-spinner is passed (#3159)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: tusharsadhwani

docs/changelog/3158.bugfix.rst

@@ -0,0 +1 @@
+``--parallel-no-spinner`` flag now implies ``--parallel``

docs/user_guide.rst

@@ -394,8 +394,8 @@ Parallel mode
   - ``auto`` to limit it to CPU count,
   - or pass an integer to set that limit.
 - Parallel mode displays a progress spinner while running tox environments in parallel, and reports outcome of these as
-  soon as they have been completed with a human readable duration timing attached. This spinner can be disabled via the
-  ``--parallel-no-spinner`` flag.
+  soon as they have been completed with a human readable duration timing attached. To run parallelly without the spinner,
+  you can use the ``--parallel-no-spinner`` flag.
 - Parallel mode by default shows output only of failed environments and ones marked as :ref:`parallel_show_output`
   ``=True``.
 - There's now a concept of dependency between environments (specified via :ref:`depends`), tox will re-order the

src/tox/session/cmd/legacy.py

@@ -110,7 +110,7 @@ def legacy(state: State) -> int:
             option.env = CliEnv(["py"])
         option.devenv_path = Path(option.devenv_path)
         return devenv(state)
-    if option.parallel != 0:  # only 0 means sequential
+    if option.parallel_no_spinner is True or option.parallel != 0:  # only 0 means sequential
         return run_parallel(state)
     return run_sequential(state)
 

src/tox/session/cmd/run/parallel.py

@@ -74,7 +74,7 @@ def parallel_flags(
         "--parallel-no-spinner",
         action="store_true",
         dest="parallel_no_spinner",
-        help="do not show the spinner",
+        help="run tox environments in parallel, but don't show the spinner, implies --parallel",
     )
 
 
@@ -83,7 +83,7 @@ def run_parallel(state: State) -> int:
     option = state.conf.options
     return execute(
         state,
-        max_workers=option.parallel,
+        max_workers=None if option.parallel_no_spinner is True else option.parallel,
         has_spinner=option.parallel_no_spinner is False and option.parallel_live is False,
         live=option.parallel_live,
     )

tests/session/cmd/test_parallel.py

@@ -6,9 +6,11 @@
 from subprocess import PIPE, Popen
 from time import sleep
 from typing import TYPE_CHECKING
+from unittest import mock
 
 import pytest
 
+from tox.session.cmd.run import parallel
 from tox.session.cmd.run.parallel import parse_num_processes
 from tox.tox_env.api import ToxEnv
 from tox.tox_env.errors import Fail
@@ -169,3 +171,28 @@ def test_parallel_requires_arg(tox_project: ToxProjectCreator) -> None:
     outcome = tox_project({"tox.ini": ""}).run("p", "-p", "-h")
     outcome.assert_failed()
     assert "argument -p/--parallel: expected one argument" in outcome.err
+
+
+def test_parallel_no_spinner(tox_project: ToxProjectCreator) -> None:
+    """Ensure passing `--parallel-no-spinner` implies `--parallel`."""
+    with mock.patch.object(parallel, "execute") as mocked:
+        tox_project({"tox.ini": ""}).run("p", "--parallel-no-spinner")
+
+    mocked.assert_called_once_with(
+        mock.ANY,
+        max_workers=None,
+        has_spinner=False,
+        live=False,
+    )
+
+
+def test_parallel_no_spinner_legacy(tox_project: ToxProjectCreator) -> None:
+    with mock.patch.object(parallel, "execute") as mocked:
+        tox_project({"tox.ini": ""}).run("--parallel-no-spinner")
+
+    mocked.assert_called_once_with(
+        mock.ANY,
+        max_workers=None,
+        has_spinner=False,
+        live=False,
+    )