Merge pull request #357 from Kantis/fix-path-with-spaces

Avoiding to use the URL-encoded path

Author: Jolanrensen

core/generated-sources/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/common.kt

@@ -55,6 +55,6 @@ public fun isFile(url: URL): Boolean = url.protocol == "file"
 
 public fun asFileOrNull(url: URL): File? = if (isFile(url)) File(url.path) else null
 
-public fun urlAsFile(url: URL): File = File(url.path)
+public fun urlAsFile(url: URL): File = File(url.toURI())
 
 public fun isProtocolSupported(url: URL): Boolean = url.protocol in setOf("http", "https", "ftp")

core/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/common.kt

@@ -55,6 +55,6 @@ public fun isFile(url: URL): Boolean = url.protocol == "file"
 
 public fun asFileOrNull(url: URL): File? = if (isFile(url)) File(url.path) else null
 
-public fun urlAsFile(url: URL): File = File(url.path)
+public fun urlAsFile(url: URL): File = File(url.toURI())
 
 public fun isProtocolSupported(url: URL): Boolean = url.protocol in setOf("http", "https", "ftp")

data/jetbrains_repositories.csv renamed to data/jetbrains repositories.csv

@@ -2,6 +2,7 @@ package org.jetbrains.dataframe.gradle
 
 import com.beust.klaxon.KlaxonException
 import io.kotest.assertions.asClue
+import io.kotest.assertions.throwables.shouldNotThrowAny
 import io.kotest.assertions.throwables.shouldThrow
 import io.kotest.assertions.throwables.shouldThrowAny
 import io.kotest.matchers.shouldBe
@@ -13,6 +14,8 @@ import java.io.FileNotFoundException
 import java.io.IOException
 import java.net.URL
 import java.nio.file.Files
+import java.nio.file.Paths
+import kotlin.io.path.absolutePathString
 
 class DataFrameReadTest {
 
@@ -75,10 +78,12 @@ class DataFrameReadTest {
             }
         }
     }
-    
+
     @Test
     fun `data accessible and readable`() {
-        val df = DataFrame.read(File("../../data/jetbrains_repositories.csv"))
+        shouldNotThrowAny {
+            DataFrame.read(Paths.get("../../data/jetbrains repositories.csv").absolutePathString())
+        }
     }
 
     @Test

plugins/dataframe-gradle-plugin/src/test/kotlin/org/jetbrains/dataframe/gradle/DataFrameReadTest.kt

@@ -1042,7 +1042,7 @@ class DataFrameSymbolProcessorTest {
         result.successfulCompilation shouldBe false
     }
 
-    private val jetbrainsCsv = File("../../data/jetbrains_repositories.csv")
+    private val jetbrainsCsv = File("../../data/jetbrains repositories.csv")
 
     @Test
     fun `imported schema resolved`() {