Skip to content

Commit 5f139d9

Browse files
committed
feat(intentions): simplify TestCodeGenTask and fix getElementToAction logic.
1 parent 7c0e41d commit 5f139d9

File tree

3 files changed

+8
-9
lines changed

3 files changed

+8
-9
lines changed

src/main/kotlin/cc/unitmesh/devti/intentions/action/AutoTestThisBaseIntention.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ class AutoTestThisBaseIntention : ChatBaseIntention() {
2929
val element = getElementToAction(project, editor) ?: return
3030
selectElement(element, editor)
3131

32-
val task = TestCodeGenTask(TestCodeGenRequest(file, element, project, editor, element.text))
32+
val task = TestCodeGenTask(TestCodeGenRequest(file, element, project, editor))
3333

3434
ProgressManager.getInstance()
3535
.runProcessWithProgressAsynchronously(task, BackgroundableProcessIndicator(task))

src/main/kotlin/cc/unitmesh/devti/intentions/action/task/TestCodeGenTask.kt

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,16 +4,16 @@ import cc.unitmesh.devti.AutoDevBundle
44
import cc.unitmesh.devti.AutoDevNotifications
55
import cc.unitmesh.devti.context.modifier.CodeModifierProvider
66
import cc.unitmesh.devti.gui.chat.ChatActionType
7-
import cc.unitmesh.devti.intentions.action.test.TestCodeGenRequest
87
import cc.unitmesh.devti.intentions.action.test.TestCodeGenContext
8+
import cc.unitmesh.devti.intentions.action.test.TestCodeGenRequest
99
import cc.unitmesh.devti.llms.LlmFactory
10-
import cc.unitmesh.devti.util.parser.parseCodeFromString
1110
import cc.unitmesh.devti.provider.AutoTestService
1211
import cc.unitmesh.devti.provider.context.*
13-
import cc.unitmesh.devti.template.GENIUS_CODE
1412
import cc.unitmesh.devti.statusbar.AutoDevStatus
1513
import cc.unitmesh.devti.statusbar.AutoDevStatusService
14+
import cc.unitmesh.devti.template.GENIUS_CODE
1615
import cc.unitmesh.devti.template.TemplateRender
16+
import cc.unitmesh.devti.util.parser.parseCodeFromString
1717
import com.intellij.lang.LanguageCommenters
1818
import com.intellij.openapi.application.ApplicationManager
1919
import com.intellij.openapi.application.ReadAction
@@ -26,7 +26,7 @@ import com.intellij.openapi.project.DumbService
2626
import com.intellij.openapi.project.Project
2727
import com.intellij.openapi.vfs.VirtualFile
2828
import com.intellij.psi.PsiNameIdentifierOwner
29-
import kotlinx.coroutines.flow.*
29+
import kotlinx.coroutines.flow.Flow
3030
import kotlinx.coroutines.runBlocking
3131

3232
class TestCodeGenTask(val request: TestCodeGenRequest) :
@@ -43,7 +43,7 @@ class TestCodeGenTask(val request: TestCodeGenRequest) :
4343
private val template = templateRender.getTemplate("test-gen.vm")
4444

4545
override fun run(indicator: ProgressIndicator) {
46-
indicator.isIndeterminate = true
46+
indicator.isIndeterminate = false
4747
indicator.fraction = 0.1
4848
indicator.text = AutoDevBundle.message("intentions.chat.code.test.step.prepare-context")
4949

@@ -98,7 +98,7 @@ class TestCodeGenTask(val request: TestCodeGenRequest) :
9898
testPromptContext.sourceCode = if(request.element !is PsiNameIdentifierOwner) {
9999
testContext.testElement?.text ?: ""
100100
} else {
101-
request.selectText
101+
request.element.text ?: ""
102102
}
103103

104104
testPromptContext.isNewFile = testContext.isNewFile

src/main/kotlin/cc/unitmesh/devti/intentions/action/test/TestCodeGenRequest.kt

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,5 @@ class TestCodeGenRequest(
99
val file: PsiFile,
1010
val element: PsiElement,
1111
val project: Project,
12-
val editor: Editor,
13-
val selectText: String
12+
val editor: Editor
1413
)

0 commit comments

Comments
 (0)