Skip to content

Commit da63e51

Browse files
authored
Merge pull request #2769 from opentensor/fix/roman/legacy_runner
Fix e2e `conftest.py` for legacy runner Merge it bc doesn't affect tests with docker. Just with local run with legacy runner
2 parents fc47114 + 427f530 commit da63e51

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

tests/e2e_tests/conftest.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ def local_chain(request):
7777
logging.warning("Docker not found in the operating system!")
7878
logging.warning(docker_command)
7979
logging.warning("Tests are run in legacy mode.")
80-
yield from legacy_runner(request)
80+
yield from legacy_runner(params)
8181

8282

8383
def legacy_runner(params):
@@ -95,7 +95,6 @@ def legacy_runner(params):
9595

9696
# Compile commands to send to process
9797
cmds = shlex.split(f"{script_path} {args}")
98-
9998
with subprocess.Popen(
10099
cmds,
101100
start_new_session=True,

0 commit comments

Comments
 (0)