androidx.navigation.safeargs.gradle.IncrementalPluginTest fails when upgrading to Kotlin 1.4 [165307851]

Obsolete

Bug

Status Update

No update yet.

Description

au...@google.com

created issue #1

Aug 18, 2020 05:39PM

> Task :compileDebugKotlin FAILED
e: /tmp/junit6598847376903397767/build/generated/source/navigation-args/debug/androidx/navigation/testapp/NextFragmentArgs.kt: (108, 73): 'map((T) -> R): List<R>' is only available since Kotlin 1.3.50 and cannot be used in Kotlin 1.3

Comments

au...@google.com <au...@google.com> #2Aug 18, 2020 05:48PM

Hi Ed, Thank you so much for these suggestions. I've been reviewing them and merging them in. Hopefully it should be live. I've included a thank you note too in the article.

da...@google.com <da...@google.com> #3Aug 18, 2020 05:52PM

Great! Thanks a lot, I'll look for the live updates soon!

il...@google.com <il...@google.com> Aug 18, 2020 05:52PM

Assigned to da...@google.com.

au...@google.com <au...@google.com> #4Aug 18, 2020 05:56PM

This topic https://android-review.googlesource.com/q/topic:%22kotlin1_4%22+(status:open%20OR%20status:merged)

da...@google.com <da...@google.com> #5Aug 20, 2020 10:01PM

I can't figure this one out... We use Gradke's Test Kit (https://docs.gradle.org/current/userguide/test_kit.html) which allows us invoke Gradle from the host-side test. The plugins are inherited, including the one under test so it should be using the updated Kotlin plugin 1.4 (and compiler I suppose) but for some reason that log seems to indicate its in 1.3 and I'm not sure why. Even forcing the plugin version in the test project (the Gradle project that is build in the test-itself) does not solve the issue.

Some interesting things... adding the following debug lines to the test when Kotlin is suppose to be 1.4:

println(KotlinVersion.CURRENT)
println(System.getProperty(System.getProperty("java.class.path")!!.replace(':', '\n')))

print the following: https://paste.googleplex.com/5601006115094528?raw. Notice the Kotlin version is 1.3.72 and in the classpath the Kotlin 1.4 stdlib jar is trumped from one coming from Gradle /usr/local/google/home/danysantiago/.gradle/wrapper/dists/gradle-6.5.1-bin/1xb5adrwzprfzochsd294a1j8/gradle-6.5.1/lib/kotlin-stdlib-1.3.72.jar.

Meanwhile if I revert the Kotlin version in AndroidX to 1.3.71 I get the following output: https://paste.googleplex.com/6095261354950656?raw, the version printed is still 1.3.72 but coming from the same jar which overrides the 1.3.71 version.

The diff between these two outputs (and the classpath) is here: https://diff.googleplex.com/#key=1iXMqf9StmP9.

One part that is a unknown to me is the effect of the gradleTestKit() function used to add the dependency, I don't know if it has some some or dependency logic to that makes ultimately overrides the Kotlin version in the test. Maybe Gradle needs to update their Kotlin version? (https://github.com/gradle/gradle/issues/12660).

da...@google.com <da...@google.com> #6Aug 20, 2020 10:05PM

Another thing I tried: If I take the test project (in https://cs.android.com/androidx/platform/frameworks/support/+/androidx-master-dev:navigation/navigation-safe-args-gradle-plugin/src/test/test-data/app-project-kotlin/) and duplicate the same build.gradle we create in the test along with actually adding a buildscript closure (not needed in the actual test since its suppose to come from the test kit), the test project then builds, I attached it for reference.

KotlinPluginTestProject.zip

311 KB

Download

au...@google.com <au...@google.com> #7Aug 21, 2020 04:14PM

+gavra@ in case he as insights on how to force a kotlin plugin version to 1.4 in these tests.

ga...@google.com <ga...@google.com> #8Aug 25, 2020 06:12PM

Looking at https://source.corp.google.com/search?q=KOTLIN_GRADLE_PLUGIN%20package:aosp-androidx&sq=, it seems that the kotlin gradle plugin on the classpath will be the one coming from Dependencies.kt.

@aurimas Your CLs do not seem to update that. Can you try running the tests with that change?

ap...@google.com <ap...@google.com> #9Aug 27, 2020 08:30PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 33ebdb9c4da8e3bbdc297bab586eeb84d1e83855
Author: Aurimas Liutikas <aurimas@google.com>
Date: Mon Aug 17 08:53:12 2020

Upgrade androidx to Kotlin 1.4

- clean up new Kotlin warnings after upgrading.
- enable multidex for appcompat and preference tests as
we are hitting dex limit
- remove new redundant extension method in fragment-ktx

Test: ./busytown/androidx.sh
Bug: 165307851, 165300826
Relnote: Upgrade androidx to use Kotlin 1.4

Change-Id: Id647100407925c16d734c8c43392b4e2d108d0e3

M appcompat/appcompat/build.gradle
M appcompat/appcompat/src/androidTest/AndroidManifest.xml
M buildSrc/build_dependencies.gradle
M camera/integration-tests/camerapipetestapp/src/main/java/androidx/camera/integration/camera2/pipe/CameraPipeApplication.kt
M camera/integration-tests/camerapipetestapp/src/main/java/androidx/camera/integration/camera2/pipe/SimpleCamera.kt
M camera/integration-tests/timingtestapp/src/main/java/androidx/camera/integration/antelope/CameraUtils.kt
M core/core-ktx/src/androidTest/java/androidx/core/view/ViewTest.kt
M dynamic-animation/dynamic-animation-ktx/api/current.txt
M dynamic-animation/dynamic-animation-ktx/api/public_plus_experimental_current.txt
M dynamic-animation/dynamic-animation-ktx/api/restricted_current.txt
M fragment/fragment-ktx/api/current.txt
M fragment/fragment-ktx/api/public_plus_experimental_current.txt
M fragment/fragment-ktx/api/restricted_current.txt
M fragment/fragment-ktx/src/main/java/androidx/fragment/app/Fragment.kt
D fragment/fragment-ktx/src/main/java/androidx/fragment/app/FragmentResultOwner.kt
M gradle.properties
M jetifier/jetifier/core/src/test/kotlin/com/android/tools/build/jetifier/core/config/ConfigParserTest.kt
M jetifier/jetifier/processor/src/main/kotlin/com/android/tools/build/jetifier/processor/Processor.kt
M lifecycle/integration-tests/kotlintestapp/src/test-common/java/androidx.lifecycle/LifecycleCoroutineScopeTestBase.kt
M lifecycle/lifecycle-livedata-core-ktx/src/test/java/androidx/lifecycle/LiveDataTest.kt
M lifecycle/lifecycle-livedata-ktx/src/test/java/androidx/lifecycle/TransformationsTest.kt
M lifecycle/lifecycle-viewmodel-ktx/src/androidTest/java/androidx/lifecycle/ViewModelTest.kt
M navigation/navigation-safe-args-gradle-plugin/src/test/kotlin/androidx/navigation/safeargs/gradle/IncrementalPluginTest.kt
M navigation/navigation-safe-args-gradle-plugin/src/test/kotlin/androidx/navigation/safeargs/gradle/KotlinPluginTest.kt
M paging/common/src/main/kotlin/androidx/paging/PagePresenter.kt
M paging/common/src/test/kotlin/androidx/paging/CachingTest.kt
M paging/samples/src/main/java/androidx/paging/samples/PagingDataAdapterSample.kt
M preference/preference/build.gradle
M recyclerview/recyclerview/src/androidTest/java/androidx/recyclerview/widget/LinearLayoutManagerRemoveShownItemsTest.kt
M room/compiler/src/test/kotlin/androidx/room/ext/ElementExtTest.kt
M room/compiler/src/test/kotlin/androidx/room/processor/PojoProcessorTargetMethodTest.kt
M room/compiler/src/test/kotlin/androidx/room/processor/PojoProcessorTest.kt
M room/compiler/src/test/kotlin/androidx/room/processor/ProjectionExpanderTest.kt
M room/compiler/src/test/kotlin/androidx/room/processor/autovalue/AutoValuePojoProcessorDelegateTest.kt
M room/compiler/src/test/kotlin/androidx/room/solver/TypeAdapterStoreTest.kt
M room/integration-tests/kotlintestapp/src/androidTest/java/androidx/room/integration/kotlintestapp/test/BooksDaoTest.kt
M samples/Support4Demos/src/main/java/com/example/android/supportv4/app/FragmentViewModelSupport.kt
M viewpager2/viewpager2/src/androidTest/java/androidx/viewpager2/widget/MutableCollectionsTest.kt

https://android-review.googlesource.com/1399323

au...@google.com <au...@google.com> #10Aug 27, 2020 09:03PM

Status: Won't Fix (Obsolete)

It seems to have fixed itself #yolo

cj...@gmail.com <cj...@gmail.com> #11Oct 8, 2020 11:01AM

#YOLO :pray:

ap...@google.com <ap...@google.com> #12Nov 29, 2021 01:37AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 33ebdb9c4da8e3bbdc297bab586eeb84d1e83855
Author: Aurimas Liutikas <aurimas@google.com>
Date: Mon Aug 17 08:53:12 2020

Upgrade androidx to Kotlin 1.4

- clean up new Kotlin warnings after upgrading.
- enable multidex for appcompat and preference tests as
we are hitting dex limit
- remove new redundant extension method in fragment-ktx

Test: ./busytown/androidx.sh
Bug: 165307851, 165300826
Relnote: Upgrade androidx to use Kotlin 1.4

Change-Id: Id647100407925c16d734c8c43392b4e2d108d0e3

M lifecycle/lifecycle-viewmodel-ktx/src/androidTest/java/androidx/lifecycle/ViewModelTest.kt
M lifecycle/integration-tests/kotlintestapp/src/test-common/java/androidx.lifecycle/LifecycleCoroutineScopeTestBase.kt
M appcompat/appcompat/build.gradle
M navigation/navigation-safe-args-gradle-plugin/src/test/kotlin/androidx/navigation/safeargs/gradle/IncrementalPluginTest.kt
M room/compiler/src/test/kotlin/androidx/room/processor/autovalue/AutoValuePojoProcessorDelegateTest.kt
M lifecycle/lifecycle-livedata-core-ktx/src/test/java/androidx/lifecycle/LiveDataTest.kt
M camera/integration-tests/camerapipetestapp/src/main/java/androidx/camera/integration/camera2/pipe/CameraPipeApplication.kt
M buildSrc/build_dependencies.gradle
M jetifier/jetifier/processor/src/main/kotlin/com/android/tools/build/jetifier/processor/Processor.kt
M samples/Support4Demos/src/main/java/com/example/android/supportv4/app/FragmentViewModelSupport.kt
M appcompat/appcompat/src/androidTest/AndroidManifest.xml
M dynamic-animation/dynamic-animation-ktx/api/restricted_current.txt
M preference/preference/build.gradle
M paging/common/src/main/kotlin/androidx/paging/PagePresenter.kt
M lifecycle/lifecycle-livedata-ktx/src/test/java/androidx/lifecycle/TransformationsTest.kt
M room/compiler/src/test/kotlin/androidx/room/processor/PojoProcessorTest.kt
M room/compiler/src/test/kotlin/androidx/room/processor/PojoProcessorTargetMethodTest.kt
M room/compiler/src/test/kotlin/androidx/room/solver/TypeAdapterStoreTest.kt
M jetifier/jetifier/core/src/test/kotlin/com/android/tools/build/jetifier/core/config/ConfigParserTest.kt
M dynamic-animation/dynamic-animation-ktx/api/current.txt
M fragment/fragment-ktx/api/public_plus_experimental_current.txt
M paging/samples/src/main/java/androidx/paging/samples/PagingDataAdapterSample.kt
M fragment/fragment-ktx/api/current.txt
M core/core-ktx/src/androidTest/java/androidx/core/view/ViewTest.kt
M fragment/fragment-ktx/api/restricted_current.txt
M navigation/navigation-safe-args-gradle-plugin/src/test/kotlin/androidx/navigation/safeargs/gradle/KotlinPluginTest.kt
M room/compiler/src/test/kotlin/androidx/room/processor/ProjectionExpanderTest.kt
M gradle.properties
M viewpager2/viewpager2/src/androidTest/java/androidx/viewpager2/widget/MutableCollectionsTest.kt
M recyclerview/recyclerview/src/androidTest/java/androidx/recyclerview/widget/LinearLayoutManagerRemoveShownItemsTest.kt
M room/integration-tests/kotlintestapp/src/androidTest/java/androidx/room/integration/kotlintestapp/test/BooksDaoTest.kt
M camera/integration-tests/camerapipetestapp/src/main/java/androidx/camera/integration/camera2/pipe/SimpleCamera.kt
M paging/common/src/test/kotlin/androidx/paging/CachingTest.kt
M fragment/fragment-ktx/src/main/java/androidx/fragment/app/Fragment.kt
M camera/integration-tests/timingtestapp/src/main/java/androidx/camera/integration/antelope/CameraUtils.kt
D fragment/fragment-ktx/src/main/java/androidx/fragment/app/FragmentResultOwner.kt
M dynamic-animation/dynamic-animation-ktx/api/public_plus_experimental_current.txt
M room/compiler/src/test/kotlin/androidx/room/ext/ElementExtTest.kt