https://kotlinlang.org logo
#ktor
Title
# ktor
k

KamilH

10/09/2020, 6:57 AM
I’m trying to run some tests on JVM/iOS/JS targets with
MockEngine
and it works on the JVM and JS targets, but fails on iOS with following message:
Copy code
kotlin.Error: Ktor native HttpClient requires kotlinx.coroutines version with `native-mt` suffix (like `1.3.9-native-mt`). Consider checking the dependencies.
Where should I add those dependencies? I though I should add it in
iosTest
like that:
Copy code
sourceSets {
        val iosTest by getting {
            dependencies {
                implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.3.9-native-mt")
            }
        }
}
but it doesn’t help. Adding to
iosMain
also doesn’t change anything
Here is a test code:
Copy code
class HttpClientTest {

    private val httpClient: HttpClient = HttpClient(MockEngine) {
        engine {
            addHandler {
                respondOk()
            }
        }
    }

    @Test
    @JsName("testResponse")
    fun `test response`() = runTest {

        val response = httpClient.get<HttpResponse>("<http://www.google.com|www.google.com>")

        assertEquals(response.status, HttpStatusCode.OK)
    }
}
e

e5l

10/09/2020, 7:04 AM
Hey, @KamilH, could you file an issue with reproducer? I will take a look
k

KamilH

10/09/2020, 7:05 AM
Sure, on YouTrack?
e

e5l

10/09/2020, 7:05 AM
Yep
👍 1
t

Tijl

10/09/2020, 7:39 AM
don’t add it to iosMain, add it to commonMain. and use !! to enforce the version:
implementation("org.jetbrains.kotlinx:kotlinx-coroutines-core:1.3.9-native-mt!!")
note there is a
1.3.9-native-mt-2
by now as well
e

e5l

10/09/2020, 8:04 AM
Yep, we also allow
native-mt-2
and any compatible with native version. There is runtime check for that
k

KamilH

10/09/2020, 8:46 AM
@Tijl Thank you it’s working now @e5l Would you still like me to file the issue?
e

e5l

10/09/2020, 10:21 AM
Yep, in Kotlin. It's tooling problem
n

Nikolay Kasyanov

10/28/2020, 1:42 PM
hey folks, do I get it right that the workaround is to put the coroutines dependency into
commonMain
?
t

Tijl

10/28/2020, 1:53 PM
I mean, it’s not really a workaround, it’s the normal way of doing things.
n

Nikolay Kasyanov

10/28/2020, 1:56 PM
I agree, although then I’m confused what bug the OP was asked to file
k

KamilH

10/28/2020, 2:33 PM
At the and of the day I didn’t file a bug, because I didn’t know what to file 😅
n

Nikolay Kasyanov

10/28/2020, 2:33 PM
I see, thanks 🙂
a

Anvith

10/29/2020, 6:35 AM
@Tijl This enforcement (
!!
)causes Intellij to break and all syntax highlighting to disappear since it cannot resolve the artifact, strangely though commandline works and generates stuff. I ended using
1.3.9-native-mt-2
along side the gradle force flag for it to work perfectly with both
Copy code
implementation(CommonDeps.coroutinesCore){
           isForce =true
    //newer gradle directive  of 'strictly' doesn't work for some reason either
  }
k

KamilH

10/29/2020, 6:46 AM
@Anvith thanks for sharing this info. Just yesterday I was testing my library that is using
1.3.9-native-mt!!
and it was working properly when integrated into Android app, but suddenly it stopped working. I spent some time investigating and I’ve found nothing about that. Changing to
1.3.9-native-mt-2
in Kotlin Multiplatform library solved an issue. So strange
👍 1
t

Tijl

10/29/2020, 11:46 AM
I guess it’s an intellij regression then? I’m using Android Studio, which usually uses a lower IJ version as it’s base, seems fine here. I think (not sure) !! is a shortcut for strictly.
3 Views