Skip to content

Commit

Permalink
static_route_test queue empty check not correct
Browse files Browse the repository at this point in the history
After enabling the processing of queue, check should be for the queue to
become empty, not the otherway around.

Change-Id: I77cdaf5942ae53e5cfb6a6f5e01048091238b62f
Closes-bug:#1560287
  • Loading branch information
ananth-at-camphor-networks authored and Nischal Sheth committed Mar 22, 2016
1 parent 64ad777 commit 03e480c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/bgp/test/static_route_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2670,7 +2670,7 @@ TYPED_TEST(StaticRouteTest, MultipleVpnRoutes) {
TASK_UTIL_EXPECT_TRUE(this->IsQueueEmpty("nat-2"));
for (int i = 0; i < DB::PartitionCount(); i++) {
DBPartition *partition = this->bgp_server_->database()->GetPartition(i);
TASK_UTIL_EXPECT_FALSE(partition->IsDBQueueEmpty());
TASK_UTIL_EXPECT_TRUE(partition->IsDBQueueEmpty());
}

// Check for Static route
Expand Down Expand Up @@ -2723,7 +2723,7 @@ TYPED_TEST(StaticRouteTest, MultipleVpnRoutes) {
TASK_UTIL_EXPECT_TRUE(this->IsQueueEmpty("nat-2"));
for (int i = 0; i < DB::PartitionCount(); i++) {
DBPartition *partition = this->bgp_server_->database()->GetPartition(i);
TASK_UTIL_EXPECT_FALSE(partition->IsDBQueueEmpty());
TASK_UTIL_EXPECT_TRUE(partition->IsDBQueueEmpty());
}

// Check for Static route
Expand Down

0 comments on commit 03e480c

Please sign in to comment.