Fixed issue and added TrackGetTest

Author: Gruncan

pom.xml

@@ -6,8 +6,8 @@
 
 
     <groupId>com.spotstat</groupId>
-    <artifactId>spotstat</artifactId>
-    <version>1.0-SNAPSHOT</version>
+    <artifactId>S4J</artifactId>
+    <version>1.1a</version>
 
     <properties>
         <maven.compiler.source>17</maven.compiler.source>
@@ -18,12 +18,25 @@
 
         <dependency>
             <groupId>org.junit.jupiter</groupId>
-            <artifactId>junit-jupiter-api</artifactId>
+            <artifactId>junit-jupiter</artifactId>
             <version>5.8.2</version>
             <scope>test</scope>
         </dependency>
 
     </dependencies>
 
+    <build>
+        <plugins>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-surefire-plugin</artifactId>
+                <version>2.22.2</version>
+                <configuration>
+                    <excludedGroups>SlowNetworkTest</excludedGroups>
+                </configuration>
+            </plugin>
+        </plugins>
+    </build>
+
 
 </project>

src/main/java/com/spotify/requests/AbstractRequest.java

@@ -108,7 +108,7 @@ public String buildRequestUrl() {
 
             url = spotifyRequest.value();
             StringBuilder sb = new StringBuilder(url);
-            //if (!url.endsWith("/")) sb.append("/");
+            if (!url.endsWith("/")) sb.append("/");
 
             List<Field> spotifySubRequestFields = new ArrayList<>();
             List<Field> spotifyFieldRequestFields = new ArrayList<>();
@@ -140,7 +140,10 @@ public String buildRequestUrl() {
             // Fields annotated with SpotifyRequestField
             for (Field field : spotifyFieldRequestFields) {
                 field.setAccessible(true);
-                if (!sb.toString().endsWith("&") && !sb.toString().endsWith("/")) sb.append("?");
+                if (sb.toString().endsWith("/")) {
+                    sb = new StringBuilder(sb.substring(0, sb.length() - 1));
+                    sb.append("?");
+                }
 
                 Object o = field.get(this);
                 Class<?> type = field.getType();

src/test/java/com/spotify/requests/tracks/SeveralTracksGetTest.java

@@ -4,6 +4,7 @@
 import com.spotify.SpotifyClientBuilderTester;
 import com.spotify.requests.RequestResponse;
 import com.spotify.requests.util.Market;
+import org.junit.jupiter.api.Tag;
 import org.junit.jupiter.api.Test;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
@@ -27,6 +28,7 @@ public void severalTrackGetUrlBuildTest() {
 
 
     @Test
+    @Tag("SlowNetworkTest")
     public void severalTrackGetExecuteTest() {
         String[] songs = new String[]{"1ljziaoMnRH95aPeOSGAtr", "2fOYcnUo9iPTOqIlSg26MY"};
         SeveralTracksGet tracksGet = new SeveralTracksGet(songs);

src/test/java/com/spotify/requests/tracks/TrackGetTest.java

@@ -0,0 +1,36 @@
+package com.spotify.requests.tracks;
+
+import com.spotify.SpotifyClient;
+import com.spotify.SpotifyClientBuilderTester;
+import com.spotify.requests.RequestResponse;
+import org.junit.jupiter.api.Tag;
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+class TrackGetTest {
+
+
+    @Test
+    public void trackGetUrlBuildTest() {
+        TrackGet trackGet = new TrackGet("id1");
+
+
+        String url = trackGet.buildRequestUrl();
+
+
+        assertEquals("tracks/id1", url);
+    }
+
+    @Test
+    @Tag("SlowNetworkTest")
+    public void trackGetExecuteTest() {
+        String id = "2fOYcnUo9iPTOqIlSg26MY";
+        TrackGet trackGet = new TrackGet(id);
+        SpotifyClient scbt = new SpotifyClientBuilderTester().getBuiltClient();
+        RequestResponse response = scbt.executeRequest(trackGet);
+
+        assertEquals(200, response.getCode());
+    }
+
+}