Refs #20650: Apply rev suggestions and fix test

Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com>
eProsima · Jun 12, 2024 · 5669527 · 5669527
1 parent d387880
commit 5669527
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/examples/cpp/delivery_mechanisms/PubSubApp.cpp b/examples/cpp/delivery_mechanisms/PubSubApp.cpp
@@ -90,7 +90,7 @@ PubSubApp::PubSubApp(
 
     switch (config.delivery_mechanism)
     {
-        case CLIParser::DeliveryMechanismKind::INTRA_PROCESS:   // (It should never reach this section
+        case CLIParser::DeliveryMechanismKind::INTRA_PROCESS:
         {
             // No transport needed, but at least a transport needs to be declared to avoid participant creation failure
             pqos.transport().use_builtin_transports = true;

diff --git a/test/examples/test_delivery_mechanisms.py b/test/examples/test_delivery_mechanisms.py
@@ -77,10 +77,10 @@ def test_delivery_mechanisms_timeout(pub_args, sub_args):
         print (e.output)
     except subprocess.TimeoutExpired:
         ret = True
-        subprocess.check_output('@DOCKER_EXECUTABLE@ compose -f configuration.compose.yml down',
+        subprocess.check_output('@DOCKER_EXECUTABLE@ compose -f delivery_mechanisms.compose.yml down',
             stderr=subprocess.STDOUT,
             shell=True,
-            timeout=30
+            timeout=15
         )
 
     assert(ret)