Skip to content

Commit 5297086

Browse files
committed
Override the newNameResolver overload that takes io.grpc.Uri instead of java.net.URI to avoid error-prone warnings.
1 parent c662a18 commit 5297086

1 file changed

Lines changed: 6 additions & 2 deletions

File tree

api/src/test/java/io/grpc/NameResolverRegistryTest.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,9 @@ public void newNameResolver_providerReturnsNull_ioGrpcUri() {
102102
registry.register(
103103
new BaseProvider(true, 5, "noScheme") {
104104
@Override
105-
public NameResolver newNameResolver(URI passedUri, NameResolver.Args passedArgs) {
105+
public NameResolver newNameResolver(io.grpc.Uri passedUri, NameResolver.Args passedArgs) {
106+
assertThat(passedUri).isSameInstanceAs(ioGrpcUri);
107+
assertThat(passedArgs).isSameInstanceAs(args);
106108
return null;
107109
}
108110
});
@@ -116,7 +118,9 @@ public void newNameResolver_providerReturnsNull_javaNetUri() {
116118
registry.register(
117119
new BaseProvider(true, 5, "noScheme") {
118120
@Override
119-
public NameResolver newNameResolver(URI passedUri, NameResolver.Args passedArgs) {
121+
public NameResolver newNameResolver(io.grpc.Uri passedUri, NameResolver.Args passedArgs) {
122+
assertThat(passedUri).isSameInstanceAs(javaNetUri);
123+
assertThat(passedArgs).isSameInstanceAs(args);
120124
return null;
121125
}
122126
});

0 commit comments

Comments
 (0)