Don't check history depth if RMW_QOS_POLICY_HISTORY_KEEP_ALL (#593)
Signed-off-by: Dan Rose <dan@digilabs.io>
This commit is contained in:
parent
ec43777891
commit
4835a947a5
1 changed files with 6 additions and 6 deletions
|
@ -171,9 +171,9 @@ TEST_P_RMW(TestPublisherGetActualQoS, test_publisher_get_qos_settings)
|
||||||
EXPECT_EQ(
|
EXPECT_EQ(
|
||||||
qos->history,
|
qos->history,
|
||||||
parameters.qos_expected.history);
|
parameters.qos_expected.history);
|
||||||
EXPECT_EQ(
|
if (parameters.qos_expected.history == RMW_QOS_POLICY_HISTORY_KEEP_LAST) {
|
||||||
qos->depth,
|
EXPECT_EQ(qos->depth, parameters.qos_expected.depth);
|
||||||
parameters.qos_expected.depth);
|
}
|
||||||
EXPECT_EQ(
|
EXPECT_EQ(
|
||||||
qos->reliability,
|
qos->reliability,
|
||||||
parameters.qos_expected.reliability);
|
parameters.qos_expected.reliability);
|
||||||
|
@ -226,9 +226,9 @@ TEST_P_RMW(TestSubscriptionGetActualQoS, test_subscription_get_qos_settings)
|
||||||
EXPECT_EQ(
|
EXPECT_EQ(
|
||||||
qos->history,
|
qos->history,
|
||||||
parameters.qos_expected.history);
|
parameters.qos_expected.history);
|
||||||
EXPECT_EQ(
|
if (parameters.qos_expected.history == RMW_QOS_POLICY_HISTORY_KEEP_LAST) {
|
||||||
qos->depth,
|
EXPECT_EQ(qos->depth, parameters.qos_expected.depth);
|
||||||
parameters.qos_expected.depth);
|
}
|
||||||
EXPECT_EQ(
|
EXPECT_EQ(
|
||||||
qos->reliability,
|
qos->reliability,
|
||||||
parameters.qos_expected.reliability);
|
parameters.qos_expected.reliability);
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue