Make paging-testing multiplatform with only a JVM implementation #586
Conversation
| plugins { | ||
| id("AndroidXPlugin") | ||
| id("com.android.library") | ||
| id("org.jetbrains.kotlin.android") |
There was a problem hiding this comment.
I think we need to continue targeting android because the first stable version is to be released with android support. @ianhanniballake What are your thoughts on this?
There was a problem hiding this comment.
Given that these are testing paging-common APIs, I think it makes to match the same set of targets.
There was a problem hiding this comment.
@veyndan We are in the process of refactoring Logger in paging-common with added android target so paging-common will become an android library. Working on the CL as we speak, should be up today. I'll cc you on it.
There was a problem hiding this comment.
I've updated paging-common into an android library now.
There was a problem hiding this comment.
Just updated the PR to add back the android target!
veyndan force-pushed the veyndan/paging-test-jvm branch from 01c6cda to ccf7f1d Compare | implementation(project(":paging:paging-common")) | ||
| } | ||
| } | ||
| jvmTest { |
There was a problem hiding this comment.
Does command ./gradlew paging:paging-testing:connectedCheck still run successfully? I know we don't have android tests but in paging-common I had to add android-specific test dependencies or else connectCheck task errors out. Not sure if we need it for paging-testing.
There was a problem hiding this comment.
I just ran it and it completed successfully
veyndan force-pushed the veyndan/paging-test-jvm branch from ccf7f1d to 7250175 Compare 
3 participants
Test: ./gradlew paging:paging-testing:allTests