Skip to content

Commit 5f76927

Browse files
committed
Fix OptionalDecoderTests
1 parent d264baf commit 5f76927

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

‎java8/src/test/java/feign/optionals/OptionalDecoderTests.java‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,11 @@ interface OptionalInterface {
3434

3535
@Test
3636
public void simple404OptionalTest() throws IOException, InterruptedException {
37-
MockWebServer server = new MockWebServer();
37+
final MockWebServer server = new MockWebServer();
3838
server.enqueue(new MockResponse().setResponseCode(404));
3939
server.enqueue(new MockResponse().setBody("foo"));
4040

41-
OptionalInterface api = Feign.builder()
41+
final OptionalInterface api = Feign.builder()
4242
.decode404()
4343
.decoder(new OptionalDecoder(new Decoder.Default()))
4444
.target(OptionalInterface.class, server.url("/").toString());
@@ -49,14 +49,13 @@ public void simple404OptionalTest() throws IOException, InterruptedException {
4949

5050
@Test
5151
public void simple204OptionalTest() throws IOException, InterruptedException {
52-
MockWebServer server = new MockWebServer();
52+
final MockWebServer server = new MockWebServer();
5353
server.enqueue(new MockResponse().setResponseCode(204));
5454

55-
OptionalInterface api = Feign.builder()
55+
final OptionalInterface api = Feign.builder()
5656
.decoder(new OptionalDecoder(new Decoder.Default()))
5757
.target(OptionalInterface.class, server.url("/").toString());
5858

5959
assertThat(api.get().isPresent()).isFalse();
60-
assertThat(api.get().get()).isNull();
6160
}
6261
}

0 commit comments

Comments
 (0)