Relax the timeouts in the register tests

OpenCyphal · Jan 24, 2024 · 3da3aea · 3da3aea
1 parent 813bc0c
commit 3da3aea
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 0 deletions.
diff --git a/tests/cmd/register_access.py b/tests/cmd/register_access.py
@@ -155,6 +155,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "-j",
             "register-access",
+            "--timeout=10",
             "10,",
             expect_register,
             environment_variables={
@@ -172,6 +173,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "-j",
             "register-access",
+            "--timeout=10",
             "10",
             expect_register,
             environment_variables={
@@ -188,6 +190,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "-j",
             "register-access",
+            "--timeout=10",
             "10,",
             expect_register,
             "Reference value",

diff --git a/tests/cmd/register_batch.py b/tests/cmd/register_batch.py
@@ -91,6 +91,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "register-batch",
             f"--file={file}",
+            "--timeout=10",
             environment_variables={
                 **transport_factory(100).environment,
                 "YAKUT_PATH": str(OUTPUT_DIR),
@@ -109,6 +110,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "register-batch",
             f"--file={file}",
+            "--timeout=10",
             environment_variables={
                 **transport_factory(100).environment,
                 "YAKUT_PATH": str(OUTPUT_DIR),
@@ -126,6 +128,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "register-batch",
             f"--file={file}",
+            "--timeout=10",
             "10",
             environment_variables={
                 **transport_factory(100).environment,
@@ -144,6 +147,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "register-batch",
             f"--file={file}",
+            "--timeout=10",
             "10,11",
             environment_variables={
                 **transport_factory(100).environment,
@@ -162,6 +166,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "register-batch",
             f"--file={file}",
+            "--timeout=3",  # Shorter timeout here because we know one is going to time out.
             "10-13",
             environment_variables={
                 **transport_factory(100).environment,
@@ -182,6 +187,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "register-batch",
             f"--file={file}",
+            "--timeout=10",
             "10,11",
             environment_variables={
                 **transport_factory(100).environment,
@@ -201,6 +207,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "register-batch",
             f"--file={file}",
+            "--timeout=10",
             "10,11",
             "--optional-register",
             environment_variables={
@@ -220,6 +227,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "register-batch",
             f"--file={file}",
+            "--timeout=10",
             "10",
             "--detailed",
             environment_variables={
@@ -238,6 +246,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "register-batch",
             f"--file={file}",
+            "--timeout=10",
             "10",
             "--only=iv",  # The requested register is not immutable-volatile so it will be skipped.
             environment_variables={

diff --git a/tests/cmd/register_list.py b/tests/cmd/register_list.py
@@ -93,6 +93,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "-j",
             "register-list",
+            "--timeout=10",
             "10",
             environment_variables={
                 "YAKUT_TRANSPORT": transport_factory(100).expression,
@@ -109,6 +110,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "-j",
             "register-list",
+            "--timeout=10",
             "10,",  # Mind the comma!
             environment_variables={
                 "YAKUT_TRANSPORT": transport_factory(100).expression,
@@ -125,6 +127,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "-j",
             "register-list",
+            "--timeout=3",  # Shorter timeout because some nodes are expected to not respond.
             "10..13",
             environment_variables={
                 "YAKUT_TRANSPORT": transport_factory(100).expression,
@@ -144,6 +147,7 @@ def _unittest_cmd(compiled_dsdl: Any, transport_factory: TransportFactory) -> No
         status, stdout, _ = execute_cli(
             "-j",
             "register-list",
+            "--timeout=3",  # Shorter timeout because some nodes are expected to not respond.
             "10..13",
             "--optional-service",
             environment_variables={