PathProviderTest.getImageUrl covered with tests

Author: pashidlos

src/main/java/io/visual_regression_tracker/sdk_java/PathProvider.java

@@ -23,7 +23,7 @@ public String getTestRunPath() {
     }
 
     public String getImageUrl(String name) {
-        if(name.isEmpty()){
+        if(name == null || name.isEmpty()){
             return null;
         }
         return baseApiUrl.concat("/").concat(name);

src/test/java/io/visual_regression_tracker/sdk_java/PathProviderTest.java

@@ -0,0 +1,33 @@
+package io.visual_regression_tracker.sdk_java;
+
+import org.testng.annotations.BeforeMethod;
+import org.testng.annotations.DataProvider;
+import org.testng.annotations.Test;
+
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.MatcherAssert.assertThat;
+
+public class PathProviderTest {
+
+  String apiUrl = "http://localhost:4200";
+  PathProvider pathProvider;
+
+  @BeforeMethod()
+  public void setUp() {
+    pathProvider = new PathProvider(apiUrl);
+  }
+
+  @DataProvider(name = "shouldGetImageUrlCases")
+  public Object[][] shouldGetImageUrlCases() {
+    return new Object[][] {
+      {null, null}, {"", null}, {"some", apiUrl.concat("/some")},
+    };
+  }
+
+  @Test(dataProvider = "shouldGetImageUrlCases")
+  public void shouldGetImageUrl(String imageName, String expectedResult) {
+    String result = pathProvider.getImageUrl(imageName);
+
+    assertThat(result, is(expectedResult));
+  }
+}