Skip to content

Commit

Permalink
Refs #22658: corrected failing CI
Browse files Browse the repository at this point in the history
Signed-off-by: Juanjo Garcia <juanjosegarcia@eprosima.com>
  • Loading branch information
juanjo4936 committed Jan 29, 2025
1 parent c1e2ee7 commit 92c93c3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
3 changes: 2 additions & 1 deletion test/blackbox/api/dds-pim/PubSubWriter.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -893,7 +893,8 @@ class PubSubWriter
}

PubSubWriter& reliability(
const eprosima::fastdds::dds::ReliabilityQosPolicyKind kind, int max_blocking_time)
const eprosima::fastdds::dds::ReliabilityQosPolicyKind kind,
int max_blocking_time)
{
datawriter_qos_.reliability().kind = kind;
datawriter_qos_.reliability().max_blocking_time.seconds = max_blocking_time;
Expand Down
3 changes: 1 addition & 2 deletions test/blackbox/common/DDSBlackboxTestsListeners.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3439,7 +3439,7 @@ TEST(DDSStatus, keyed_reliable_positive_acks_disabled_on_unack_sample_removed)
/*!
* Regression Test for 22658: when he entire history is acked in volatile, given that the entries are deleted from the
* history, check_acked_status satisfies min_low_mark >= get_seq_num_min() because seq_num_min is unknown. This makes
* try_remove to fail, because it tries to remove changes but there were none. This causes prepare_change to not
* try_remove to fail, because it tries to remove changes but there were none. This causes prepare_change to not
* perform the changes, since the history was full and could not delete any changes.
*/

Expand Down Expand Up @@ -3467,7 +3467,6 @@ TEST(DDSStatus, entire_history_acked_volatile_unknown_pointer)
reader.wait_discovery();

auto data = default_helloworld_data_generator(2);

for (auto sample : data)
{
EXPECT_TRUE(writer.send_sample(sample));
Expand Down

0 comments on commit 92c93c3

Please sign in to comment.