From 56695275026e5b72d2bc627ebbd6f0a5a66d55d4 Mon Sep 17 00:00:00 2001 From: JesusPoderoso Date: Wed, 12 Jun 2024 13:14:46 +0200 Subject: [PATCH] Refs #20650: Apply rev suggestions and fix test Signed-off-by: JesusPoderoso --- examples/cpp/delivery_mechanisms/PubSubApp.cpp | 2 +- test/examples/test_delivery_mechanisms.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/examples/cpp/delivery_mechanisms/PubSubApp.cpp b/examples/cpp/delivery_mechanisms/PubSubApp.cpp index 66019640c9d..3d064672329 100644 --- 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 index b2575f17eaf..27d45c6354a 100644 --- 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)