Skip to content

Commit 654d4a3

Browse files
committed
test(javascript): add assertion and logging statements
- Added an assertion statement to check the size of the `relevantClass` list. - Added logging statements to print the formatted elements of the `relevantClass` list. These changes were made to improve the test coverage and provide better visibility into the `relevantClass` list during testing.
1 parent 10d9d87 commit 654d4a3

File tree

2 files changed

+13
-4
lines changed

2 files changed

+13
-4
lines changed

javascript/src/main/kotlin/cc/unitmesh/ide/javascript/provider/testing/JSWriteTestService.kt

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,13 @@ class JSWriteTestService : WriteTestService() {
5757
}
5858

5959
val underTestObj = ReadAction.compute<String, Throwable> {
60-
val underTestObj = JavaScriptClassContextBuilder().getClassContext(elementToTest, false)?.format()
60+
val underTestObj = JavaScriptClassContextBuilder()
61+
.getClassContext(elementToTest, false)?.format()
62+
6163
if (underTestObj == null) {
62-
val funcObj = JavaScriptMethodContextBuilder().getMethodContext(elementToTest, false, false)?.format()
64+
val funcObj = JavaScriptMethodContextBuilder()
65+
.getMethodContext(elementToTest, false, false)?.format()
66+
6367
return@compute funcObj ?: ""
6468
} else {
6569
return@compute underTestObj
@@ -94,7 +98,7 @@ class JSWriteTestService : WriteTestService() {
9498
val result = mutableMapOf<String, ClassContext>()
9599
jsFunction.parameterList?.parameters?.map {
96100
it.typeElement?.let { typeElement ->
97-
resolveByType(typeElement, it.typeElement!!.text)
101+
result += resolveByType(typeElement, it.typeElement!!.text)
98102
}
99103
}
100104

@@ -123,11 +127,13 @@ class JSWriteTestService : WriteTestService() {
123127
result += mapOf(typeName to it)
124128
}
125129
}
130+
126131
else -> {
127132
println("resolveReferenceLocally is not TypeScriptInterface")
128133
}
129134
}
130135
}
136+
131137
else -> {
132138
println("returnType is not TypeScriptSingleType")
133139
}

javascript/src/test/kotlin/cc/unitmesh/ide/javascript/provider/testing/JSWriteTestServiceTest.kt

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import com.intellij.lang.javascript.psi.ecmal4.JSClass
88
import com.intellij.psi.PsiFileFactory
99
import com.intellij.psi.util.PsiTreeUtil
1010
import com.intellij.testFramework.LightPlatformTestCase
11+
import junit.framework.TestCase
1112
import java.io.File
1213

1314
class JSWriteTestServiceTest : LightPlatformTestCase() {
@@ -85,6 +86,8 @@ class JSWriteTestServiceTest : LightPlatformTestCase() {
8586
val function = PsiTreeUtil.findChildOfType(clazz, JSFunction::class.java)!!
8687

8788
val relevantClass = JSWriteTestService().lookupRelevantClass(project, function)
88-
println(relevantClass)
89+
90+
TestCase.assertEquals(relevantClass.size, 2)
91+
println(relevantClass.map { it.format() })
8992
}
9093
}

0 commit comments

Comments
 (0)