test and bugfix

Author: Jolanrensen

core/src/main/kotlin/org/jetbrains/kotlinx/dataframe/impl/codeGen/CodeGeneratorImpl.kt

@@ -53,17 +53,19 @@ private val letterCategories = setOf(
     CharCategory.DECIMAL_DIGIT_NUMBER
 )
 
-internal fun String.needsQuoting(): Boolean {
-    return isBlank() ||
+internal fun String.needsQuoting(): Boolean =
+    if (isQuoted()) false
+    else isBlank() ||
         first().isDigit() ||
         contains(charsToQuote) ||
         HardKeywords.VALUES.contains(this) ||
         ModifierKeywords.VALUES.contains(this) ||
         all { it == '_' } ||
         any { it != '_' && it.category !in letterCategories }
-}
 
-internal fun String.quoteIfNeeded() = if (needsQuoting()) "`$this`" else this
+public fun String.isQuoted(): Boolean = startsWith("`") && endsWith("`")
+
+public fun String.quoteIfNeeded(): String = if (needsQuoting()) "`$this`" else this
 
 internal fun List<Code>.join() = joinToString("\n")
 
@@ -250,7 +252,7 @@ internal open class ExtensionsCodeGeneratorImpl(
         val markerName = marker.name
         val markerType = "$markerName${marker.typeArguments}"
         val visibility = renderTopLevelDeclarationVisibility(marker)
-        val shortMarkerName = markerName.substring(markerName.lastIndexOf('.') + 1)
+        val shortMarkerName = markerName.substring(markerName.lastIndexOf('.') + 1).removeQuotes()
         val nullableShortMarkerName = "Nullable$shortMarkerName"
 
         fun String.toNullable() = if (this.last() == '?' || this == "*") this else "$this?"

plugins/symbol-processor/src/main/kotlin/org/jetbrains/dataframe/ksp/GetQualifiedNameOrThrow.kt

@@ -1,7 +1,20 @@
 package org.jetbrains.dataframe.ksp
 
 import com.google.devtools.ksp.symbol.KSDeclaration
+import org.jetbrains.kotlinx.dataframe.impl.codeGen.quoteIfNeeded
 
-fun KSDeclaration.getQualifiedNameOrThrow(): String {
-    return (qualifiedName ?: error("@DataSchema declaration ${simpleName.asString()} at $location must have qualified name")).asString()
-}
+fun KSDeclaration.getQualifiedNameOrThrow(): String =
+    qualifiedName
+        ?.let {
+            buildString {
+                val qualifier = it.getQualifier()
+                if (qualifier.isNotEmpty()) {
+                    for (it in qualifier.split('.')) {
+                        append(it.quoteIfNeeded() + '.')
+                    }
+                }
+
+                append(it.getShortName().quoteIfNeeded())
+            }
+        }
+        ?: error("@DataSchema declaration ${simpleName.asString()} at $location must have qualified name")

plugins/symbol-processor/src/test/kotlin/org/jetbrains/dataframe/ksp/DataFrameSymbolProcessorTest.kt

@@ -33,6 +33,48 @@ class DataFrameSymbolProcessorTest {
         KspCompilationTestRunner.compilationDir.deleteRecursively()
     }
 
+    @Test
+    fun `interface with backticked name`() {
+        val result = KspCompilationTestRunner.compile(
+            TestCompilationParameters(
+                sources = listOf(
+                    SourceFile.kotlin(
+                        "MySources.kt",
+                        """
+                $imports
+
+                class OuterClass
+
+                @DataSchema(isOpen = false)
+                interface `Hello Something` {
+                    val name: String
+                    val `test name`: NestedClass
+                    val nullableProperty: Int?
+                    val a: () -> Unit
+                    val d: List<List<*>>
+                    
+                    class NestedClass
+                }
+
+                val ColumnsContainer<`Hello Something`>.col1: DataColumn<String> get() = name
+                val ColumnsContainer<`Hello Something`>.col2: DataColumn<`Hello Something`.NestedClass> get() = `test name`
+                val ColumnsContainer<`Hello Something`>.col3: DataColumn<Int?> get() = nullableProperty
+                val ColumnsContainer<`Hello Something`>.col4: DataColumn<() -> Unit> get() = a
+                val ColumnsContainer<`Hello Something`>.col5: DataColumn<List<List<*>>> get() = d
+                
+                val DataRow<`Hello Something`>.row1: String get() = name
+                val DataRow<`Hello Something`>.row2: `Hello Something`.NestedClass get() = `test name`
+                val DataRow<`Hello Something`>.row3: Int? get() = nullableProperty
+                val DataRow<`Hello Something`>.row4: () -> Unit get() = a
+                val DataRow<`Hello Something`>.row5: List<List<*>> get() = d
+                        """.trimIndent()
+                    )
+                )
+            )
+        )
+        result.successfulCompilation shouldBe true
+    }
+
     @Test
     fun `all interface`() {
         val result = KspCompilationTestRunner.compile(