Skip to content

Commit 80b1de2

Browse files
committed
Add nullability annotations to module/spring-boot-restclient-test
See gh-46587
1 parent d0dd8c8 commit 80b1de2

File tree

4 files changed

+10
-3
lines changed

4 files changed

+10
-3
lines changed

module/spring-boot-restclient-test/src/main/java/org/springframework/boot/restclient/test/MockServerRestClientCustomizer.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@
2121
import java.util.concurrent.ConcurrentHashMap;
2222
import java.util.function.Supplier;
2323

24+
import org.jspecify.annotations.Nullable;
25+
2426
import org.springframework.beans.BeanUtils;
2527
import org.springframework.boot.restclient.RestClientCustomizer;
2628
import org.springframework.http.client.BufferingClientHttpRequestFactory;
@@ -133,7 +135,7 @@ public Map<RestClient.Builder, RequestExpectationManager> getExpectationManagers
133135
return this.expectationManagers;
134136
}
135137

136-
public MockRestServiceServer getServer(RestClient.Builder restClientBuilder) {
138+
public @Nullable MockRestServiceServer getServer(RestClient.Builder restClientBuilder) {
137139
return this.servers.get(restClientBuilder);
138140
}
139141

module/spring-boot-restclient-test/src/main/java/org/springframework/boot/restclient/test/MockServerRestTemplateCustomizer.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@
2121
import java.util.concurrent.ConcurrentHashMap;
2222
import java.util.function.Supplier;
2323

24+
import org.jspecify.annotations.Nullable;
25+
2426
import org.springframework.beans.BeanUtils;
2527
import org.springframework.boot.restclient.RestTemplateBuilder;
2628
import org.springframework.boot.restclient.RestTemplateCustomizer;
@@ -147,7 +149,7 @@ public Map<RestTemplate, RequestExpectationManager> getExpectationManagers() {
147149
return this.expectationManagers;
148150
}
149151

150-
public MockRestServiceServer getServer(RestTemplate restTemplate) {
152+
public @Nullable MockRestServiceServer getServer(RestTemplate restTemplate) {
151153
return this.servers.get(restTemplate);
152154
}
153155

module/spring-boot-restclient-test/src/main/java/org/springframework/boot/restclient/test/RootUriRequestExpectationManager.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ public static RequestExpectationManager forRestTemplate(RestTemplate restTemplat
158158
RequestExpectationManager expectationManager) {
159159
Assert.notNull(restTemplate, "'restTemplate' must not be null");
160160
UriTemplateHandler templateHandler = restTemplate.getUriTemplateHandler();
161-
if (templateHandler instanceof RootUriTemplateHandler rootHandler) {
161+
if (templateHandler instanceof RootUriTemplateHandler rootHandler && rootHandler.getRootUri() != null) {
162162
return new RootUriRequestExpectationManager(rootHandler.getRootUri(), expectationManager);
163163
}
164164
return expectationManager;

module/spring-boot-restclient-test/src/main/java/org/springframework/boot/restclient/test/package-info.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,7 @@
1717
/**
1818
* RestClient and RestTemplate test utilities.
1919
*/
20+
@NullMarked
2021
package org.springframework.boot.restclient.test;
22+
23+
import org.jspecify.annotations.NullMarked;

0 commit comments

Comments
 (0)