Setting to disable the SARIF reports visualizer (#1184)

Add a setting to disable built-in SARIF reports visualizer (#1167)

Author: mmvpm

utbot-intellij/src/main/kotlin/org/utbot/intellij/plugin/generator/CodeGenerationController.kt

@@ -160,7 +160,7 @@ object CodeGenerationController {
                                 UtTestsDialogProcessor.updateIndicator(indicator, UtTestsDialogProcessor.ProgressRange.SARIF, "Start tests with coverage", 1.0)
 
                                 invokeLater {
-                                    runInspectionsIfNeeded(model.project, srcClassPathToSarifReport)
+                                    runInspectionsIfNeeded(model, srcClassPathToSarifReport)
                                 }
                             }
                         }
@@ -174,19 +174,22 @@ object CodeGenerationController {
      * Runs the UTBot inspection if there are detected errors.
      */
     private fun runInspectionsIfNeeded(
-        project: Project,
+        model: GenerateTestsModel,
         srcClassPathToSarifReport: MutableMap<Path, Sarif>
     ) {
+        if (!model.runInspectionAfterTestGeneration) {
+            return
+        }
         val sarifHasResults = srcClassPathToSarifReport.any { (_, sarif) ->
             sarif.getAllResults().isNotEmpty()
         }
         if (!sarifHasResults) {
             return
         }
         UnitTestBotInspectionManager
-            .getInstance(project, srcClassPathToSarifReport)
+            .getInstance(model.project, srcClassPathToSarifReport)
             .createNewGlobalContext()
-            .doInspections(AnalysisScope(project))
+            .doInspections(AnalysisScope(model.project))
     }
 
     private fun proceedTestReport(proc: EngineProcess, model: GenerateTestsModel) {

utbot-intellij/src/main/kotlin/org/utbot/intellij/plugin/models/GenerateTestsModel.kt

@@ -67,6 +67,7 @@ data class GenerateTestsModel(
     lateinit var parametrizedTestSource: ParametrizedTestSource
     lateinit var runtimeExceptionTestsBehaviour: RuntimeExceptionTestsBehaviour
     lateinit var hangingTestsTimeout: HangingTestsTimeout
+    var runInspectionAfterTestGeneration: Boolean = false
     lateinit var forceStaticMocking: ForceStaticMocking
     lateinit var chosenClassesToMockAlways: Set<ClassId>
     lateinit var commentStyle: JavaDocCommentStyle

utbot-intellij/src/main/kotlin/org/utbot/intellij/plugin/settings/Settings.kt

@@ -49,6 +49,7 @@ class Settings(val project: Project) : PersistentStateComponent<Settings.State>
         var runtimeExceptionTestsBehaviour: RuntimeExceptionTestsBehaviour = RuntimeExceptionTestsBehaviour.defaultItem,
         @OptionTag(converter = HangingTestsTimeoutConverter::class)
         var hangingTestsTimeout: HangingTestsTimeout = HangingTestsTimeout(),
+        var runInspectionAfterTestGeneration: Boolean = false,
         var forceStaticMocking: ForceStaticMocking = ForceStaticMocking.defaultItem,
         var treatOverflowAsError: TreatOverflowAsError = TreatOverflowAsError.defaultItem,
         var parametrizedTestSource: ParametrizedTestSource = ParametrizedTestSource.defaultItem,
@@ -66,6 +67,7 @@ class Settings(val project: Project) : PersistentStateComponent<Settings.State>
             staticsMocking = model.staticsMocking,
             runtimeExceptionTestsBehaviour = model.runtimeExceptionTestsBehaviour,
             hangingTestsTimeout = model.hangingTestsTimeout,
+            runInspectionAfterTestGeneration = model.runInspectionAfterTestGeneration,
             forceStaticMocking = model.forceStaticMocking,
             parametrizedTestSource = model.parametrizedTestSource,
             classesToMockAlways = model.chosenClassesToMockAlways.mapTo(mutableSetOf()) { it.name }.toTypedArray(),
@@ -88,6 +90,7 @@ class Settings(val project: Project) : PersistentStateComponent<Settings.State>
             if (staticsMocking != other.staticsMocking) return false
             if (runtimeExceptionTestsBehaviour != other.runtimeExceptionTestsBehaviour) return false
             if (hangingTestsTimeout != other.hangingTestsTimeout) return false
+            if (runInspectionAfterTestGeneration != other.runInspectionAfterTestGeneration) return false
             if (forceStaticMocking != other.forceStaticMocking) return false
             if (treatOverflowAsError != other.treatOverflowAsError) return false
             if (parametrizedTestSource != other.parametrizedTestSource) return false
@@ -107,6 +110,7 @@ class Settings(val project: Project) : PersistentStateComponent<Settings.State>
             result = 31 * result + staticsMocking.hashCode()
             result = 31 * result + runtimeExceptionTestsBehaviour.hashCode()
             result = 31 * result + hangingTestsTimeout.hashCode()
+            result = 31 * result + runInspectionAfterTestGeneration.hashCode()
             result = 31 * result + forceStaticMocking.hashCode()
             result = 31 * result + treatOverflowAsError.hashCode()
             result = 31 * result + parametrizedTestSource.hashCode()
@@ -137,6 +141,8 @@ class Settings(val project: Project) : PersistentStateComponent<Settings.State>
 
     val staticsMocking: StaticsMocking get() = state.staticsMocking
 
+    val runInspectionAfterTestGeneration: Boolean get() = state.runInspectionAfterTestGeneration
+
     val forceStaticMocking: ForceStaticMocking get() = state.forceStaticMocking
 
     val treatOverflowAsError: TreatOverflowAsError get() = state.treatOverflowAsError

utbot-intellij/src/main/kotlin/org/utbot/intellij/plugin/settings/SettingsWindow.kt

@@ -35,6 +35,7 @@ class SettingsWindow(val project: Project) {
 
     // TODO it is better to use something like SearchEverywhere for classes but it is complicated to implement
     private val excludeTable = MockAlwaysClassesTable(project)
+    private lateinit var runInspectionAfterTestGenerationCheckBox: JCheckBox
     private lateinit var forceMockCheckBox: JCheckBox
     private lateinit var enableSummarizationGenerationCheckBox: JCheckBox
 
@@ -99,6 +100,23 @@ class SettingsWindow(val project: Project) {
             valuesComboBox(loader, values)
         }
 
+        row {
+            cell {
+                runInspectionAfterTestGenerationCheckBox = checkBox("Display detected errors on the Problems tool window")
+                    .onApply {
+                        settings.state.runInspectionAfterTestGeneration = runInspectionAfterTestGenerationCheckBox.isSelected
+                    }
+                    .onReset {
+                        runInspectionAfterTestGenerationCheckBox.isSelected = settings.state.runInspectionAfterTestGeneration
+                    }
+                    .onIsModified {
+                        runInspectionAfterTestGenerationCheckBox.isSelected xor settings.state.runInspectionAfterTestGeneration
+                    }
+                    // .apply { ContextHelpLabel.create("Automatically run code inspection after test generation")() }
+                    .component
+            }
+        }
+
         row {
             cell {
                 enableSummarizationGenerationCheckBox = checkBox("Enable Summaries Generation")

utbot-intellij/src/main/kotlin/org/utbot/intellij/plugin/ui/GenerateTestsDialogWindow.kt

@@ -510,6 +510,7 @@ class GenerateTestsDialogWindow(val model: GenerateTestsModel) : DialogWrapper(m
         with(settings) {
             model.runtimeExceptionTestsBehaviour = runtimeExceptionTestsBehaviour
             model.hangingTestsTimeout = hangingTestsTimeout
+            model.runInspectionAfterTestGeneration = runInspectionAfterTestGeneration
             model.forceStaticMocking = forceStaticMocking
             model.chosenClassesToMockAlways = chosenClassesToMockAlways()
             model.fuzzingValue = fuzzingValue