feat(acton): add --backtrace full option to test runner (#791)

Author: i582

modules/acton/src/org/ton/intellij/acton/cli/ActonCommand.kt

@@ -72,6 +72,7 @@ sealed class ActonCommand(val name: String) {
         var clearCache: Boolean = false,
         var useColors: Boolean = false,
         var ui: Boolean = false,
+        var backtraceFull: Boolean = false,
         var debug: Boolean = false,
         var debugPort: String = "",
     ) : ActonCommand("test") {
@@ -86,6 +87,10 @@ sealed class ActonCommand(val name: String) {
                 add("always")
             }
             if (ui) add("--ui")
+            if (backtraceFull) {
+                add("--backtrace")
+                add("full")
+            }
             if (debug) {
                 add("--debug")
                 if (debugPort.isNotBlank()) {

modules/acton/src/org/ton/intellij/acton/ide/ActonBacktraceRerunLauncher.kt

@@ -24,7 +24,7 @@ object ActonBacktraceRerunLauncher {
         )
         val configuration = settings.configuration as ActonCommandConfiguration
         configuration.copyFrom(sourceConfiguration)
-        configuration.parameters = withBacktraceFull(sourceConfiguration.parameters)
+        configuration.testBacktraceFull = true
 
         failedTestName?.trim()?.takeIf { it.isNotBlank() }?.let { testName ->
             configuration.testMode = org.ton.intellij.acton.cli.ActonCommand.Test.TestMode.FUNCTION
@@ -38,28 +38,8 @@ object ActonBacktraceRerunLauncher {
         return true
     }
 
-    internal fun withBacktraceFull(parameters: String): String {
-        val trimmed = parameters.trim()
-        if (trimmed.isBlank()) {
-            return "--backtrace full"
-        }
-        if (FULL_BACKTRACE_REGEX.containsMatchIn(trimmed)) {
-            return trimmed
-        }
-        if (BACKTRACE_REGEX.containsMatchIn(trimmed)) {
-            return BACKTRACE_REGEX.replace(trimmed) { match ->
-                val prefix = match.groups[1]?.value.orEmpty()
-                "$prefix--backtrace full"
-            }
-        }
-        return "$trimmed --backtrace full"
-    }
-
     private fun buildConfigurationName(configuration: ActonCommandConfiguration, failedTestName: String?): String {
         val suffix = failedTestName?.takeIf { it.isNotBlank() }?.let { " [$it backtrace]" } ?: " [backtrace]"
         return configuration.name + suffix
     }
-
-    private val FULL_BACKTRACE_REGEX = Regex("""(?:^|\s)--backtrace\s+full(?:\s|$)""")
-    private val BACKTRACE_REGEX = Regex("""(^|\s)--backtrace\s+\S+(?=\s|$)""")
 }

modules/acton/src/org/ton/intellij/acton/runconfig/ActonCommandConfiguration.kt

@@ -42,6 +42,7 @@ class ActonCommandConfiguration(project: Project, factory: ConfigurationFactory,
     var testFunctionName: String = ""
     var testClearCache: Boolean = false
     var testUi: Boolean = false
+    var testBacktraceFull: Boolean = false
 
     // Run command specific settings (Acton.toml scripts)
     var runScriptName: String = ""
@@ -58,7 +59,15 @@ class ActonCommandConfiguration(project: Project, factory: ConfigurationFactory,
             scriptBroadcastNet, project.actonSettings.explorer.id, scriptDebug, scriptDebugPort,
         )
 
-        "test" -> ActonCommand.Test(testMode, testTarget, testFunctionName, testClearCache, true, testUi)
+        "test" -> ActonCommand.Test(
+            testMode,
+            testTarget,
+            testFunctionName,
+            testClearCache,
+            true,
+            testUi,
+            testBacktraceFull,
+        )
         "init" -> ActonCommand.Init()
         "run" -> ActonCommand.Run(runScriptName)
         "retrace" -> ActonCommand.Retrace(retraceTransactionHash, retraceNetwork, retraceContractId)
@@ -99,6 +108,7 @@ class ActonCommandConfiguration(project: Project, factory: ConfigurationFactory,
         element.setAttribute("testFunctionName", testFunctionName)
         element.setAttribute("testClearCache", testClearCache.toString())
         element.setAttribute("testUi", testUi.toString())
+        element.setAttribute("testBacktraceFull", testBacktraceFull.toString())
 
         // Run settings
         element.setAttribute("runScriptName", runScriptName)
@@ -141,6 +151,7 @@ class ActonCommandConfiguration(project: Project, factory: ConfigurationFactory,
         testFunctionName = element.getAttributeValue("testFunctionName") ?: ""
         testClearCache = element.getAttributeValue("testClearCache")?.toBoolean() ?: false
         testUi = element.getAttributeValue("testUi")?.toBoolean() ?: false
+        testBacktraceFull = element.getAttributeValue("testBacktraceFull")?.toBoolean() ?: false
 
         // Run settings
         runScriptName = element.getAttributeValue("runScriptName") ?: ""
@@ -178,6 +189,7 @@ class ActonCommandConfiguration(project: Project, factory: ConfigurationFactory,
         testFunctionName = other.testFunctionName
         testClearCache = other.testClearCache
         testUi = other.testUi
+        testBacktraceFull = other.testBacktraceFull
 
         runScriptName = other.runScriptName
 

modules/acton/src/org/ton/intellij/acton/runconfig/ActonCommandConfigurationEditor.kt

@@ -56,6 +56,7 @@ class ActonCommandConfigurationEditor(private val project: Project) : SettingsEd
     private val testFunctionNameField = JBTextField()
     private val testClearCacheCheckBox = JBCheckBox("Clear compilation cache before testing", false)
     private val testUiCheckBox = JBCheckBox("Open browser UI", false)
+    private val testBacktraceFullCheckBox = JBCheckBox("Run tests with full execution backtrace", false)
     private var testMode: ActonCommand.Test.TestMode = ActonCommand.Test.TestMode.DIRECTORY
 
     // Run specific
@@ -227,6 +228,7 @@ class ActonCommandConfigurationEditor(private val project: Project) : SettingsEd
         testFunctionNameField.text = configuration.testFunctionName
         testClearCacheCheckBox.isSelected = configuration.testClearCache
         testUiCheckBox.isSelected = configuration.testUi
+        testBacktraceFullCheckBox.isSelected = configuration.testBacktraceFull
 
         runScriptNameField.text = configuration.runScriptName
 
@@ -264,6 +266,7 @@ class ActonCommandConfigurationEditor(private val project: Project) : SettingsEd
         configuration.testFunctionName = testFunctionNameField.text.trim()
         configuration.testClearCache = testClearCacheCheckBox.isSelected
         configuration.testUi = testUiCheckBox.isSelected
+        configuration.testBacktraceFull = testBacktraceFullCheckBox.isSelected
 
         configuration.runScriptName = runScriptNameField.text.trim()
 
@@ -338,6 +341,10 @@ class ActonCommandConfigurationEditor(private val project: Project) : SettingsEd
                 row {
                     cell(testUiCheckBox)
                 }.topGap(TopGap.NONE)
+
+                row {
+                    cell(testBacktraceFullCheckBox)
+                }.topGap(TopGap.NONE)
             }.topGap(TopGap.NONE)
 
             runGroup = group("Run arguments") {

modules/acton/test/org/ton/intellij/acton/cli/ActonCommandTest.kt

@@ -27,4 +27,15 @@ class ActonCommandTest {
             ActonCommand.from("init", "--create-dapp"),
         )
     }
+
+    @Test
+    fun `test command includes full backtrace flag`() {
+        assertEquals(
+            listOf("--reporter", "console,teamcity", "--backtrace", "full", "."),
+            ActonCommand.Test(
+                target = ".",
+                backtraceFull = true,
+            ).getArguments(),
+        )
+    }
 }

modules/tolk/src/org/ton/intellij/tolk/ide/fixes/TolkCreateLocalVariableQuickfix.kt

@@ -1,6 +1,5 @@
 package org.ton.intellij.tolk.ide.fixes
 
-import com.intellij.codeInsight.intention.HighPriorityAction
 import com.intellij.codeInsight.template.TemplateManager
 import com.intellij.codeInsight.template.impl.ConstantNode
 import com.intellij.codeInspection.LocalQuickFixAndIntentionActionOnPsiElement