Skip to content

Commit 4db6021

Browse files
committed
Fix the scheme name and introduce more assertions to make sure it gets used.
1 parent 5297086 commit 4db6021

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -100,32 +100,32 @@ public void getDefaultScheme_noProvider() {
100100
public void newNameResolver_providerReturnsNull_ioGrpcUri() {
101101
NameResolverRegistry registry = new NameResolverRegistry();
102102
registry.register(
103-
new BaseProvider(true, 5, "noScheme") {
103+
new BaseProvider(true, 5, ioGrpcUri.getScheme()) {
104104
@Override
105-
public NameResolver newNameResolver(io.grpc.Uri passedUri, NameResolver.Args passedArgs) {
105+
public NameResolver newNameResolver(Uri passedUri, NameResolver.Args passedArgs) {
106106
assertThat(passedUri).isSameInstanceAs(ioGrpcUri);
107107
assertThat(passedArgs).isSameInstanceAs(args);
108108
return null;
109109
}
110110
});
111111
assertThat(registry.asFactory().newNameResolver(ioGrpcUri, args)).isNull();
112-
assertThat(registry.asFactory().getDefaultScheme()).isEqualTo("noScheme");
112+
assertThat(registry.asFactory().getDefaultScheme()).isEqualTo(ioGrpcUri.getScheme());
113113
}
114114

115115
@Test
116116
public void newNameResolver_providerReturnsNull_javaNetUri() {
117117
NameResolverRegistry registry = new NameResolverRegistry();
118118
registry.register(
119-
new BaseProvider(true, 5, "noScheme") {
119+
new BaseProvider(true, 5, javaNetUri.getScheme()) {
120120
@Override
121-
public NameResolver newNameResolver(io.grpc.Uri passedUri, NameResolver.Args passedArgs) {
121+
public NameResolver newNameResolver(URI passedUri, NameResolver.Args passedArgs) {
122122
assertThat(passedUri).isSameInstanceAs(javaNetUri);
123123
assertThat(passedArgs).isSameInstanceAs(args);
124124
return null;
125125
}
126126
});
127127
assertThat(registry.asFactory().newNameResolver(javaNetUri, args)).isNull();
128-
assertThat(registry.asFactory().getDefaultScheme()).isEqualTo("noScheme");
128+
assertThat(registry.asFactory().getDefaultScheme()).isEqualTo(javaNetUri.getScheme());
129129
}
130130

131131
@Test

0 commit comments

Comments
 (0)