/**
 * Copyright 2012-2018 The Feign Authors
 *
 * Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except
 * in compliance with the License. You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software distributed under the License
 * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express
 * or implied. See the License for the specific language governing permissions and limitations under
 * the License.
 */
package feign.optionals;

import feign.Feign;
import feign.RequestLine;
import feign.codec.Decoder;
import okhttp3.mockwebserver.MockResponse;
import okhttp3.mockwebserver.MockWebServer;
import org.junit.Test;

import java.io.IOException;
import java.util.Optional;

import static org.assertj.core.api.Assertions.assertThat;

public class OptionalDecoderTests {

    interface OptionalInterface {
        @RequestLine("GET /")
        Optional<String> get();
    }

    @Test
    public void simple404OptionalTest() throws IOException, InterruptedException {
        MockWebServer server = new MockWebServer();
        server.enqueue(new MockResponse().setResponseCode(404));
        server.enqueue(new MockResponse().setBody("foo"));

        OptionalInterface api = Feign.builder()
                .decode404()
                .decoder(new OptionalDecoder(new Decoder.Default()))
                .target(OptionalInterface.class, server.url("/").toString());

        assertThat(api.get().isPresent()).isFalse();
        assertThat(api.get().get()).isEqualTo("foo");
    }

    @Test
    public void simple204OptionalTest() throws IOException, InterruptedException {
        MockWebServer server = new MockWebServer();
        server.enqueue(new MockResponse().setResponseCode(204));

        OptionalInterface api = Feign.builder()
                .decoder(new OptionalDecoder(new Decoder.Default()))
                .target(OptionalInterface.class, server.url("/").toString());

        assertThat(api.get().isPresent()).isFalse();
        assertThat(api.get().get()).isNull();
    }
}

@@ -1149,32 +1149,38 @@
 ption %7B%0A        
+final 
 MockWebServer se
@@ -1199,32 +1199,32 @@
 ockWebServer();%0A
-
         server.e
@@ -1332,32 +1332,38 @@
 oo%22));%0A%0A        
+final 
 OptionalInterfac
@@ -1778,16 +1778,22 @@
         
+final 
 MockWebS
@@ -1894,32 +1894,38 @@
 204));%0A%0A        
+final 
 OptionalInterfac
@@ -2088,32 +2088,32 @@
 ).toString());%0A%0A
+
         assertTh
@@ -2153,54 +2153,8 @@
 ();%0A
-        assertThat(api.get().get()).isNull();%0A
     

Fix OptionalDecoderTests