Do not add 8529 to the ports list, if another one is already specified

apetenchea · apetenchea · commit 7826530f515d · 2025-12-09T19:27:34.000+08:00
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -39,7 +39,7 @@ def pytest_addoption(parser):
         "--host", action="store", default="127.0.0.1", help="ArangoDB host address"
     )
     parser.addoption(
-        "--port", action="append", default=["8529"], help="ArangoDB coordinator ports"
+        "--port", action="append", default=None, help="ArangoDB coordinator ports"
     )
     parser.addoption(
         "--root", action="store", default="root", help="ArangoDB root user"
@@ -59,7 +59,7 @@ def pytest_addoption(parser):
 
 
 def pytest_configure(config):
-    ports = config.getoption("port")
+    ports = config.getoption("port") or ["8529"]
     hosts = [f"http://{config.getoption('host')}:{p}" for p in ports]
     url = hosts[0]
 

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ def pytest_addoption(parser):`
`39`	`39`	`"--host", action="store", default="127.0.0.1", help="ArangoDB host address"`
`40`	`40`	`)`
`41`	`41`	`parser.addoption(`
`42`		`- "--port", action="append", default=["8529"], help="ArangoDB coordinator ports"`
	`42`	`+ "--port", action="append", default=None, help="ArangoDB coordinator ports"`
`43`	`43`	`)`
`44`	`44`	`parser.addoption(`
`45`	`45`	`"--root", action="store", default="root", help="ArangoDB root user"`
`@@ -59,7 +59,7 @@ def pytest_addoption(parser):`
`59`	`59`
`60`	`60`
`61`	`61`	`def pytest_configure(config):`
`62`		`- ports = config.getoption("port")`
	`62`	`+ ports = config.getoption("port") or ["8529"]`
`63`	`63`	`hosts = [f"http://{config.getoption('host')}:{p}" for p in ports]`
`64`	`64`	`url = hosts[0]`
`65`	`65`