Merge pull request #3 from avimanyum/W-14925234

Catch json exception
salesforce · Feb 13, 2024 · 3eb7b54 · 3eb7b54
2 parents c2e5362 + 53a890a
commit 3eb7b54
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/...e-image-update/src/main/java/com/salesforce/dockerfileimageupdate/utils/PullRequests.java b/...e-image-update/src/main/java/com/salesforce/dockerfileimageupdate/utils/PullRequests.java
@@ -6,6 +6,7 @@
 import net.sourceforge.argparse4j.inf.*;
 import org.json.JSONObject;
 import org.json.JSONTokener;
+import org.json.JSONException;
 import org.kohsuke.github.*;
 import org.slf4j.*;
 
@@ -67,6 +68,8 @@ protected boolean isRenovateEnabled(List<String> filePaths, GitHubContentToProce
                 log.debug("The file with name {} not found in the repository. Exception: {}", filePath, e.getMessage());
             } catch (IOException e) {
                 log.debug("Exception while trying to close a resource. Exception: {}", e.getMessage());
+            } catch (JSONException e) {
+                log.debug("Exception while trying to read the renovate configuration file. Exception: {}", e.getMessage());
             }
         }
         return false;

diff --git a/...age-update/src/test/java/com/salesforce/dockerfileimageupdate/utils/PullRequestsTest.java b/...age-update/src/test/java/com/salesforce/dockerfileimageupdate/utils/PullRequestsTest.java
@@ -4,6 +4,7 @@
 import com.salesforce.dockerfileimageupdate.model.*;
 import com.salesforce.dockerfileimageupdate.process.*;
 import net.sourceforge.argparse4j.inf.*;
+import org.json.JSONException;
 import org.kohsuke.github.*;
 import org.mockito.*;
 import org.testng.*;
@@ -199,7 +200,7 @@ public void testisRenovateEnabledReturnsTrueIfRenovateConfigFileFoundButEnabledK
     }
 
     @Test
-    public void testisRenovateEnabledReturnsTrueIfRenovateConfigFileFoundAndResourcesThrowAnException() throws IOException {
+    public void testisRenovateEnabledReturnsFalseIfRenovateConfigFileFoundAndResourcesThrowAnException() throws IOException {
         PullRequests pullRequests = new PullRequests();
         List<String> filePaths = Collections.singletonList("renovate.json");
         GitHubContentToProcess gitHubContentToProcess = mock(GitHubContentToProcess.class);
@@ -211,6 +212,19 @@ public void testisRenovateEnabledReturnsTrueIfRenovateConfigFileFoundAndResource
         Assert.assertFalse(pullRequests.isRenovateEnabled(filePaths, gitHubContentToProcess));
     }
 
+    @Test
+    public void testisRenovateEnabledReturnsFalseIfRenovateConfigFileFoundAndJSONParsingThrowsAnException() throws IOException {
+        PullRequests pullRequests = new PullRequests();
+        List<String> filePaths = Collections.singletonList("renovate.json");
+        GitHubContentToProcess gitHubContentToProcess = mock(GitHubContentToProcess.class);
+        GHRepository ghRepository = mock(GHRepository.class);
+        GHContent content = mock(GHContent.class);
+        when(gitHubContentToProcess.getParent()).thenReturn(ghRepository);
+        when(ghRepository.getFileContent(anyString())).thenReturn(content);
+        when(content.read()).thenThrow(new JSONException(""));
+        Assert.assertFalse(pullRequests.isRenovateEnabled(filePaths, gitHubContentToProcess));
+    }
+
     @Test
     public void testisRenovateEnabledReturnsTrueIfRenovateConfigFileFoundAndEnabledKeySetToTrue() throws IOException {
         PullRequests pullRequests = new PullRequests();