Add tests

Signed-off-by: Dhi Aurrahman <[email protected]>
envoyproxy · Aug 13, 2018 · 6c263d3 · 6c263d3
1 parent 80b5498
commit 6c263d3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/test/integration/ads_integration_test.cc b/test/integration/ads_integration_test.cc
@@ -132,6 +132,10 @@ class AdsIntegrationTest : public AdsIntegrationBaseTest,
     envoy::api::v2::DiscoveryRequest discovery_request;
     VERIFY_ASSERTION(ads_stream_->waitForGrpcMessage(*dispatcher_, discovery_request));
 
+    ASSERT(discovery_request.has_node());
+    EXPECT_FALSE(discovery_request.node().id().empty());
+    EXPECT_FALSE(discovery_request.node().cluster().empty());
+
     // TODO(PiotrSikora): Remove this hack once fixed internally.
     if (!(expected_type_url == discovery_request.type_url())) {
       return AssertionFailure() << fmt::format("type_url {} does not match expected {}",

diff --git a/test/integration/load_stats_integration_test.cc b/test/integration/load_stats_integration_test.cc
@@ -164,6 +164,9 @@ class LoadStatsIntegrationTest : public HttpIntegrationTest,
       return;
     } else if (loadstats_request.cluster_stats_size() == 0) {
       loadstats_request.CopyFrom(local_loadstats_request);
+      ASSERT(loadstats_request.has_node());
+      EXPECT_FALSE(loadstats_request.node().id().empty());
+      EXPECT_FALSE(loadstats_request.node().cluster().empty());
       return;
     }