Merge pull request #32 from snyk/fix/marker_file_nullabliity_check

fix: check file in marker for nullability before proceed

Author: ArtsiomCh

CHANGELOG.md

@@ -1,4 +1,5 @@
 ## [2.2.2] - 2022-03
+- fix: check file in marker for nullability before proceed
 - feat: provide unique (per project) `shard` to getAnalysis call
 
 ## [2.2.1] - 2021-12-10

src/main/java/ai/deepcode/javaclient/core/AnalysisDataBase.java

@@ -681,6 +681,10 @@ private Map<Object, List<SuggestionForFile>> parseGetAnalysisResponse(
                               (it.getFile() == null || it.getFile().isEmpty())
                                   ? file
                                   : pdUtils.getFileByDeepcodedPath(it.getFile(), project);
+                          if (fileForMarker == null) {
+                            dcLogger.logWarn("File not found for marker: " + it);
+                            return null;
+                          }
                           return parsePosition2MyTextRange(
                               it, fileForMarker, Collections.emptyMap());
                         })

src/main/java/ai/deepcode/javaclient/core/PlatformDependentUtilsBase.java

@@ -53,6 +53,7 @@ public String getDeepCodedFilePath(@NotNull Object file) {
   @NotNull
   protected abstract String getProjectBasedFilePath(@NotNull Object file);
 
+  @Nullable
   public abstract Object getFileByDeepcodedPath(String path, Object project);
 
   public abstract Object[] getOpenProjects();