aboutsummaryrefslogtreecommitdiff
path: root/test/core/client_channel/client_channel_service_config_test.cc
diff options
context:
space:
mode:
Diffstat (limited to 'test/core/client_channel/client_channel_service_config_test.cc')
-rw-r--r--test/core/client_channel/client_channel_service_config_test.cc31
1 files changed, 4 insertions, 27 deletions
diff --git a/test/core/client_channel/client_channel_service_config_test.cc b/test/core/client_channel/client_channel_service_config_test.cc
index f709c8199a..e265653dae 100644
--- a/test/core/client_channel/client_channel_service_config_test.cc
+++ b/test/core/client_channel/client_channel_service_config_test.cc
@@ -83,29 +83,6 @@ TEST_F(ClientChannelParserTest, ValidLoadBalancingConfigGrpclb) {
EXPECT_EQ(lb_config->name(), "grpclb");
}
-TEST_F(ClientChannelParserTest, ValidLoadBalancingConfigXds) {
- const char* test_json =
- "{\n"
- " \"loadBalancingConfig\":[\n"
- " { \"does_not_exist\":{} },\n"
- " { \"xds_cluster_resolver_experimental\":{\n"
- " \"discoveryMechanisms\": [\n"
- " { \"clusterName\": \"foo\",\n"
- " \"type\": \"EDS\"\n"
- " } ],\n"
- " \"xdsLbPolicy\": [{\"round_robin\":{}}]\n"
- " } }\n"
- " ]\n"
- "}";
- auto service_config = ServiceConfigImpl::Create(ChannelArgs(), test_json);
- ASSERT_TRUE(service_config.ok()) << service_config.status();
- const auto* parsed_config =
- static_cast<internal::ClientChannelGlobalParsedConfig*>(
- (*service_config)->GetGlobalParsedConfig(parser_index_));
- auto lb_config = parsed_config->parsed_lb_config();
- EXPECT_EQ(lb_config->name(), "xds_cluster_resolver_experimental");
-}
-
TEST_F(ClientChannelParserTest, UnknownLoadBalancingConfig) {
const char* test_json = "{\"loadBalancingConfig\": [{\"unknown\":{}}]}";
auto service_config = ServiceConfigImpl::Create(ChannelArgs(), test_json);
@@ -163,15 +140,15 @@ TEST_F(ClientChannelParserTest, UnknownLoadBalancingPolicy) {
<< service_config.status();
}
-TEST_F(ClientChannelParserTest, LoadBalancingPolicyXdsNotAllowed) {
- const char* test_json =
- "{\"loadBalancingPolicy\":\"xds_cluster_resolver_experimental\"}";
+TEST_F(ClientChannelParserTest,
+ LegacyLoadBalancingPolicySpecifiesPolicyThatRequiresConfig) {
+ const char* test_json = "{\"loadBalancingPolicy\":\"rls_experimental\"}";
auto service_config = ServiceConfigImpl::Create(ChannelArgs(), test_json);
EXPECT_EQ(service_config.status().code(), absl::StatusCode::kInvalidArgument);
EXPECT_EQ(service_config.status().message(),
"errors validating service config: ["
"field:loadBalancingPolicy error:LB policy "
- "\"xds_cluster_resolver_experimental\" requires a config. Please "
+ "\"rls_experimental\" requires a config. Please "
"use loadBalancingConfig instead.]")
<< service_config.status();
}