Skip to content

Commit f3f49b6

Browse files
committed
fix test cases
1 parent dd007ec commit f3f49b6

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

core/src/test/java/io/grpc/internal/ManagedChannelImplBuilderTest.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,7 @@
5454
import io.grpc.internal.ManagedChannelImplBuilder.ChannelBuilderDefaultPortProvider;
5555
import io.grpc.internal.ManagedChannelImplBuilder.ClientTransportFactoryBuilder;
5656
import io.grpc.internal.ManagedChannelImplBuilder.FixedPortProvider;
57+
import io.grpc.internal.ManagedChannelImplBuilder.ResolvedNameResolver;
5758
import io.grpc.internal.ManagedChannelImplBuilder.UnsupportedClientTransportFactoryBuilder;
5859
import io.grpc.testing.GrpcCleanupRule;
5960
import java.net.InetSocketAddress;
@@ -386,7 +387,7 @@ public void transportDoesNotSupportAddressTypes() {
386387
builder = new ManagedChannelImplBuilder(DUMMY_AUTHORITY_VALID,
387388
mockClientTransportFactoryBuilder, new FixedPortProvider(DUMMY_PORT));
388389
try {
389-
ManagedChannel unused = grpcCleanupRule.register(builder.build());
390+
grpcCleanupRule.register(builder.build());
390391
fail("Should fail");
391392
} catch (IllegalArgumentException e) {
392393
assertThat(e)
@@ -409,7 +410,7 @@ public void transportAddressTypeCompatibilityCheckSkipped() {
409410
builder = new ManagedChannelImplBuilder(DUMMY_AUTHORITY_VALID,
410411
mockClientTransportFactoryBuilder, new FixedPortProvider(DUMMY_PORT));
411412
// should not fail
412-
ManagedChannel unused = grpcCleanupRule.register(builder.build());
413+
grpcCleanupRule.register(builder.build());
413414
}
414415

415416
@Test
@@ -803,6 +804,7 @@ public void uriPattern() {
803804
private static class CustomSocketAddress extends SocketAddress {}
804805

805806
@Test
807+
@SuppressWarnings("deprecation")
806808
public void nameResolverFactory_notAllowedAfterRegistry() {
807809
NameResolverRegistry registry = new NameResolverRegistry();
808810
builder.nameResolverRegistry(registry);
@@ -870,7 +872,7 @@ protected int priority() {
870872
}
871873
};
872874

873-
ManagedChannelImplBuilder.ResolvedNameResolver resolved = ManagedChannelImplBuilder.getNameResolverProvider(
875+
ResolvedNameResolver resolved = ManagedChannelImplBuilder.getNameResolverProvider(
874876
target, registry, explicitProvider);
875877

876878
// Should prefer explicit provider if scheme matches?
@@ -913,7 +915,7 @@ protected int priority() {
913915
NameResolverRegistry registry = new NameResolverRegistry();
914916
registry.register(registryProvider);
915917

916-
ManagedChannelImplBuilder.ResolvedNameResolver resolved = ManagedChannelImplBuilder.getNameResolverProvider(
918+
ResolvedNameResolver resolved = ManagedChannelImplBuilder.getNameResolverProvider(
917919
target, registry, null);
918920

919921
assertThat(resolved.provider).isSameInstanceAs(registryProvider);

0 commit comments

Comments
 (0)