Added a missing test case

percy · Mar 31, 2023 · 53aa109 · 53aa109
1 parent cc62b1f
commit 53aa109
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/src/main/java/io/percy/appium/providers/AppAutomate.java b/src/main/java/io/percy/appium/providers/AppAutomate.java
@@ -20,6 +20,8 @@ public AppAutomate(AppiumDriver driver) {
     }
 
     public String getDebugUrl(JSONObject result) {
+        if (result == null) return null;
+
         String buildHash = result.getString("buildHash");
         String sessionHash = result.getString("sessionHash");
         return "https://app-automate.browserstack.com/dashboard/v2/builds/" + buildHash + "/sessions/" + sessionHash;

diff --git a/src/test/java/io/percy/appium/providers/AppAutomateTest.java b/src/test/java/io/percy/appium/providers/AppAutomateTest.java
@@ -68,6 +68,11 @@ public void testGetDebugUrl() {
         Assert.assertEquals(appAutomate.getDebugUrl(result), "https://app-automate.browserstack.com/dashboard/v2/builds/abc/sessions/def");
     }
 
+    @Test
+    public void testGetDebugUrlWhenResultIsNull() {
+        Assert.assertEquals(appAutomate.getDebugUrl(null), null);
+    }
+
     @Test
     public void testSupports() {
         try {