chore(ci): update build workflow and add test case for KotlinMethodContextBuilder

- Update build workflow to include a step for building the plugin.
- Add a new test case for KotlinMethodContextBuilder to handle function comments.
- Modify the getSignatureString function in KotlinMethodContextBuilder to handle function comments correctly.

Author: phodal

.github/workflows/build.yml

@@ -94,6 +94,10 @@ jobs:
           name: tests-result
           path: ${{ github.workspace }}/build/reports/tests
 
+      # Run tests
+      - name: Build Plugin
+        run: ./gradlew :plugin:buildPlugin
+
       # Upload Kover report to CodeCov
       - name: Upload Code Coverage Report
         uses: codecov/codecov-action@v3
@@ -103,15 +107,6 @@ jobs:
         with:
           files: ${{ github.workspace }}/build/reports/kover/xml/report.xml
 
-      - name: Upload coverage reports to Codecov
-        uses: codecov/codecov-action@v3
-        env:
-          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
-
-      # Run tests
-      - name: Build Plugin
-        run: ./gradlew :plugin:buildPlugin
-
       # Prepare plugin archive content for creating artifact
       - name: Prepare Plugin Artifact
         id: artifact

kotlin/src/main/kotlin/cc/unitmesh/kotlin/context/KotlinMethodContextBuilder.kt

@@ -42,26 +42,19 @@ class KotlinMethodContextBuilder : MethodContextBuilder {
     }
 
     object Util {
-        fun getSignatureString(ktNamedFunction: KtNamedFunction): String {
-            val bodyBlockExpression = ktNamedFunction.bodyBlockExpression
+
+        fun getSignatureString(signatureString: KtNamedFunction): String {
+            val bodyBlockExpression = signatureString.bodyBlockExpression
             val startOffsetInParent = if (bodyBlockExpression != null) {
                 bodyBlockExpression.startOffsetInParent
             } else {
-                val bodyExpression = ktNamedFunction.bodyExpression
-                bodyExpression?.startOffsetInParent ?: ktNamedFunction.textLength
-            }
-
-            val docEnd = ktNamedFunction.docComment?.endOffset ?: 0
-
-            val text = ktNamedFunction.text
-
-            val result = if (docEnd < startOffsetInParent) {
-                text.substring(docEnd, startOffsetInParent)
-            } else {
-                text.substring(ktNamedFunction.startOffsetInParent, startOffsetInParent)
+                val bodyExpression = signatureString.bodyExpression
+                bodyExpression?.startOffsetInParent ?: signatureString.textLength
             }
 
-            return result.replace("\n", " ").trim()
+            val text = signatureString.text
+            val substring = text.substring(0, startOffsetInParent)
+            return substring.replace('\n', ' ').trim()
         }
     }
 }

kotlin/src/test/kotlin/cc/unitmesh/kotlin/context/KotlinMethodContextBuilderTest.kt

@@ -25,10 +25,10 @@ class KotlinMethodContextBuilderTest : LightPlatformTestCase() {
         val clz = PsiTreeUtil.findChildOfType(createFile, KtNamedFunction::class.java)!!
 
         val signatureString = KotlinMethodContextBuilder.Util.getSignatureString(clz)
-        TestCase.assertEquals(signatureString, "fun main()")
+        TestCase.assertEquals(signatureString, "/**  * It's a hello, world.  */ fun main()")
     }
 
-    fun shouldIgnoreClassFunctionComment() {
+    fun testShouldIgnoreClassFunctionComment() {
         // given
         val code = """
             class UserController {
@@ -45,7 +45,7 @@ class KotlinMethodContextBuilderTest : LightPlatformTestCase() {
         val clz = PsiTreeUtil.findChildOfType(createFile, KtNamedFunction::class.java)!!
 
         val signatureString = KotlinMethodContextBuilder.Util.getSignatureString(clz)
-        TestCase.assertEquals(signatureString, "fun main()")
+        TestCase.assertEquals(signatureString, "/**      * It's a hello, world.      */     fun main()")
     }
 
     fun testShouldHandleNormalClassFunctionComment() {