stream migration: do not suggest to migrate if exceptions are thrown inside the block (IDEA-129484)

java/java-analysis-impl/src/com/intellij/codeInspection/StreamApiMigrationInspection.java

@@ -111,7 +111,7 @@ public class StreamApiMigrationInspection extends BaseJavaBatchLocalInspectionTo
                     }
                   });
   
-                  if (effectivelyFinal[0]) {
+                  if (effectivelyFinal[0] && ExceptionUtil.getThrownCheckedExceptions(new PsiElement[] {body}).isEmpty()) {
                     if (isCollectCall(body)) {
                       holder.registerProblem(iteratedValue, "Can be replaced with collect call",
                                              ProblemHighlightType.GENERIC_ERROR_OR_WARNING, new ReplaceWithCollectCallFix());

java/java-tests/testData/codeInsight/daemonCodeAnalyzer/quickFix/streamApiMigration/beforeExceptionIsThrown.java

@@ -0,0 +1,14 @@
+// "Replace with collect" "false"
+import java.util.*;
+
+public class Test {
+  public void foo() throws Exception {}
+
+  public static void bar(List<Test> currentImportLine) throws Exception {
+    for (Test test : currentImp<caret>ortLine) {
+      if (true) {
+        test.foo();
+      }
+    }
+  }
+}