Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate Kotlin tests to Mockk [DATACASS-629] #796

Closed
spring-projects-issues opened this issue Feb 20, 2019 · 1 comment
Closed

Migrate Kotlin tests to Mockk [DATACASS-629] #796

spring-projects-issues opened this issue Feb 20, 2019 · 1 comment
Assignees

Comments

@spring-projects-issues
Copy link

@spring-projects-issues spring-projects-issues commented Feb 20, 2019

Mark Paluch opened DATACASS-629 and commented

In order to be consistent with Spring Framework tests and recommendation to use Mockk for Kotlin tests, and in order to be consistent with the upcoming Coroutines pull request that is ready to be integrated asap this one is merged


Issue Links:

@spring-projects-issues
Copy link
Author

@spring-projects-issues spring-projects-issues commented Feb 20, 2019

Mark Paluch commented

Tests on branch 2.1.x fail with:

queryForList(String, array) extension should call its Java counterpart(org.springframework.data.cassandra.core.cql.AsyncCqlOperationsExtensionsUnitTests)  Time elapsed: 0.112 sec  <<< ERROR!
java.lang.IllegalAccessError: tried to access method kotlin.collections.ArraysKt___ArraysJvmKt.contentDeepEquals([Ljava/lang/Object;[Ljava/lang/Object;)Z from class io.mockk.InternalPlatformDsl
	at org.springframework.data.cassandra.core.cql.AsyncCqlOperationsExtensionsUnitTests.queryForList(String, array) extension should call its Java counterpart(AsyncCqlOperationsExtensionsUnitTests.kt:99)

Migrating tests only for master branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants