feat: fix whitespace formatting and adding test for the functionality

Author: eleonora2687

src/main/java/spotify/api/impl/MarketApiRetrofit.java

@@ -4,7 +4,6 @@
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-
 import retrofit2.Call;
 import retrofit2.Response;
 import spotify.api.enums.HttpStatusCode;
@@ -18,12 +17,10 @@
 
 
 public class MarketApiRetrofit implements MarketApi {
-
     private final Logger logger = LoggerFactory.getLogger(MarketApiRetrofit.class);
     private final String accessToken;
     private final MarketService marketService;
 
-
     public MarketApiRetrofit(final String accessToken) {
         this(accessToken, RetrofitHttpServiceFactory.getMarketService());
     }
@@ -33,15 +30,13 @@ public MarketApiRetrofit(final String accessToken, final MarketService marketSer
         this.marketService = marketService;
     }
 
-
     @Override
     public MarketFull getMarkets(){
         logger.trace("Constructing HTTP call to fetch markets.");
         Call<MarketFull> httpCall = marketService.getMarkets("Bearer " + this.accessToken);
 
         try {
-
-            logger.info("Executing HTTP call to fetch markets");
+            logger.info("Executing HTTP call to fetch markets.");
             logger.debug("Fetching markets....");
             LoggingUtil.logHttpCall(logger, httpCall);
             Response<MarketFull> response = httpCall.execute();

src/main/java/spotify/api/spotify/SpotifyApi.java

@@ -105,7 +105,7 @@ public AlbumFull getAlbum(String albumId, Map<String, String> options) {
     }
 
     public MarketFull getMarket() {
-        logger.info("Requesting all available markets");
+        logger.info("Requesting all available markets.");
         return marketApi.getMarkets();
     }
 

src/main/java/spotify/models/markets/MarketFull.java

@@ -1,12 +1,10 @@
 package spotify.models.markets;
 
-import java.util.ArrayList;
 import java.util.List;
 
 public class MarketFull {
-
-    private List<String> markets = new ArrayList<String>() ;
-    public List<String> getAllMarkets() {
+    private List<String> markets;
+    public List<String> getMarkets() {
         return markets;
 
     }

src/test/java/spotify/api/impl/MarketApiRetrofitTest.java

@@ -0,0 +1,85 @@
+package spotify.api.impl;
+
+import okhttp3.MediaType;
+import okhttp3.Request;
+import okhttp3.ResponseBody;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.mockito.Mock;
+import org.mockito.MockitoAnnotations;
+import retrofit2.Call;
+import retrofit2.Response;
+import spotify.exceptions.HttpRequestFailedException;
+import spotify.exceptions.SpotifyActionFailedException;
+import spotify.models.markets.MarketFull;
+import spotify.retrofit.services.MarketService;
+
+import java.io.IOException;
+
+import static org.mockito.ArgumentMatchers.anyString;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+
+
+public class MarketApiRetrofitTest extends AbstractApiRetrofitTest {
+    private MarketApiRetrofit sut;
+    @Mock
+    private MarketService mockedMarketService;
+    @Mock
+    private Call<MarketFull> mockedMarketFullCall;
+
+    @BeforeEach
+    void setUp() {
+        MockitoAnnotations.openMocks(this);
+
+        sut = new MarketApiRetrofit(fakeAccessToken, mockedMarketService);
+
+        when(mockedMarketService.getMarkets(anyString())).thenReturn(mockedMarketFullCall);
+
+        when(mockedMarketFullCall.request()).thenReturn(new Request.Builder().url(fakeUrl).build());
+    }
+
+    @Test
+    void getMarketsUsesCorrectValuesToCreatHttpCall() throws IOException {
+        when(mockedMarketFullCall.execute()).thenReturn(Response.success(new MarketFull()));
+
+        sut.getMarkets();
+        verify(mockedMarketService).getMarkets(fakeAccessTokenWithBearer);
+    }
+
+    @Test
+    void getMarketsExecutesHttpCall() throws IOException {
+        when(mockedMarketFullCall.execute()).thenReturn(Response.success(new MarketFull()));
+
+        sut.getMarkets();
+        verify(mockedMarketFullCall).execute();
+    }
+
+    @Test
+    void getMarketsThrowsSpotifyActionFailedExceptionWhenError() throws IOException {
+        when(mockedMarketFullCall.execute())
+                .thenReturn(
+                        Response.error(
+                                400,
+                                ResponseBody.create(MediaType.get("application/json"), getJson("error.json"))
+                        )
+                );
+
+        Assertions.assertThrows(SpotifyActionFailedException.class, () -> sut.getMarkets());
+    }
+
+    @Test
+    void getMarketsThrowsHttpRequestFailedWhenHttpFails() throws IOException {
+        when(mockedMarketFullCall.execute()).thenThrow(IOException.class);
+
+        Assertions.assertThrows(HttpRequestFailedException.class, () -> sut.getMarkets());
+    }
+
+    @Test
+    void getMarketsReturnsMarketFullWhenSuccessful() throws IOException {
+        when(mockedMarketFullCall.execute()).thenReturn(Response.success(new MarketFull()));
+
+        Assertions.assertNotNull(sut.getMarkets());
+    }
+}