[FIR] Implement checkers for FIR parcelize plugin

There is one of checks left unimplemented (FirParcelizePropertyChecker.checkParcelableClassProperty) because it requires huge commonization of detecting which type can be serialized and which not, which is not prioritized job for now
2021-10-26 16:04:14 +03:00
parent eac9a9fc79
commit 8cdddbfd9d
37 changed files with 1089 additions and 62 deletions
@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 // WITH_STDLIB
 package test

@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 // WITH_STDLIB
 package test

@@ -0,0 +1,45 @@
+// WITH_STDLIB
+package test
+
+import kotlinx.parcelize.*
+import android.os.*
+
+object StringParceler : Parceler<String> {
+    override fun create(parcel: Parcel) = TODO()
+    override fun String.write(parcel: Parcel, flags: Int) = TODO()
+}
+
+object CharSequenceParceler : Parceler<CharSequence> {
+    override fun create(parcel: Parcel) = TODO()
+    override fun CharSequence.write(parcel: Parcel, flags: Int) = TODO()
+}
+
+class StringClassParceler : Parceler<String> {
+    override fun create(parcel: Parcel) = TODO()
+    override fun String.write(parcel: Parcel, flags: Int) = TODO()
+}
+
+@<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER, NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>TypeParceler<!><String, StringParceler>
+class MissingParcelizeAnnotation(val a: @<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>WriteWith<!><StringParceler> String)
+
+@Parcelize
+@<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER, NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>TypeParceler<!><String, StringClassParceler>
+class ShouldBeClass(val a: @<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>WriteWith<!><StringClassParceler> String) : Parcelable
+
+@Parcelize
+class Test(
+    val a: @<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>WriteWith<!><StringParceler> Int,
+    val b: @<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>WriteWith<!><StringParceler> String,
+    val c: @<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>WriteWith<!><StringParceler> CharSequence,
+    val d: @<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>WriteWith<!><CharSequenceParceler> String,
+    val e: @<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>WriteWith<!><CharSequenceParceler> CharSequence
+) : Parcelable
+
+@Parcelize
+@<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER, NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>TypeParceler<!><String, StringParceler>
+class Test2(@<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER, NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER, NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER, NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>TypeParceler<!><String, StringParceler> val a: String) : Parcelable
+
+@Parcelize
+@<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER, NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>TypeParceler<!><String, StringParceler>
+@<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER, NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>TypeParceler<!><String, CharSequenceParceler>
+class Test3(val a: String) : Parcelable
@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 package test

 import kotlinx.parcelize.Parcelize
@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 package test

 import kotlinx.parcelize.Parcelize
@@ -0,0 +1,46 @@
+// WITH_STDLIB
+package test
+
+import kotlinx.android.parcel.*
+import android.os.Parcel
+import android.os.Parcelable
+
+object <!DEPRECATED_PARCELER!>Parceler1<!> : Parceler<String> {
+    override fun create(parcel: Parcel) = parcel.readInt().toString()
+
+    override fun String.write(parcel: Parcel, flags: Int) {
+        parcel.writeInt(length)
+    }
+}
+
+object <!DEPRECATED_PARCELER!>Parceler2<!> : Parceler<List<String>> {
+    override fun create(parcel: Parcel) = listOf(parcel.readString()!!)
+
+    override fun List<String>.write(parcel: Parcel, flags: Int) {
+        parcel.writeString(this.joinToString(","))
+    }
+}
+
+<!DEPRECATED_ANNOTATION!>@Parcelize<!>
+<!FORBIDDEN_DEPRECATED_ANNOTATION!>@<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER, NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>TypeParceler<!><String, Parceler2><!>
+data class Test(
+    val a: String,
+    val b: <!FORBIDDEN_DEPRECATED_ANNOTATION!>@<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>WriteWith<!><Parceler1><!> String,
+    val c: <!FORBIDDEN_DEPRECATED_ANNOTATION!>@<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>WriteWith<!><Parceler2><!> List<<!FORBIDDEN_DEPRECATED_ANNOTATION!>@<!NEW_INFERENCE_NO_INFORMATION_FOR_PARAMETER!>WriteWith<!><Parceler1><!> String>
+) : Parcelable {
+    <!DEPRECATED_ANNOTATION!>@IgnoredOnParcel<!>
+    val x by lazy { "foo" }
+}
+
+interface <!DEPRECATED_PARCELER!>ParcelerForUser<!>: Parceler<User>
+
+<!DEPRECATED_ANNOTATION!>@Parcelize<!>
+class User(val name: String) : Parcelable {
+    private companion object : ParcelerForUser {
+        override fun User.write(parcel: Parcel, flags: Int) {
+            parcel.writeString(name)
+        }
+
+        override fun create(parcel: Parcel) = User(parcel.readString()!!)
+    }
+}
@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 package test

 import kotlinx.parcelize.Parcelize
@@ -0,0 +1,20 @@
+package test
+
+import kotlinx.parcelize.*
+import android.os.*
+
+class Box(val value: String)
+
+@Parcelize
+class Foo(val box: Box): Parcelable {
+    companion object : Parceler<Foo> {
+        override fun create(parcel: Parcel) = Foo(Box(parcel.readString()))
+
+        override fun Foo.write(parcel: Parcel, flags: Int) {
+            parcel.writeString(box.value)
+        }
+    }
+}
+
+@Parcelize
+class Foo2(val box: Box): Parcelable
@@ -0,0 +1,31 @@
+package test
+
+import kotlinx.parcelize.Parcelize
+import android.os.Parcelable
+
+@Parcelize
+open class Open(val foo: String) : Parcelable
+
+@Parcelize
+class Final(val foo: String) : Parcelable
+
+@Parcelize
+<!PARCELABLE_SHOULD_BE_INSTANTIABLE!>abstract<!> class Abstract(val foo: String) : Parcelable
+
+@Parcelize
+sealed class Sealed(val foo: String) : Parcelable {
+    class X : Sealed("")
+}
+
+class Outer {
+    @Parcelize
+    <!PARCELABLE_CANT_BE_INNER_CLASS!>inner<!> class Inner(val foo: String) : Parcelable
+}
+
+fun foo() {
+    @Parcelize
+    <!ABSTRACT_MEMBER_NOT_IMPLEMENTED, PARCELABLE_SHOULD_BE_CLASS!>object<!> : Parcelable {}
+
+    @Parcelize
+    class <!NO_PARCELABLE_SUPERTYPE, PARCELABLE_CANT_BE_LOCAL_CLASS!>Local<!> {}
+}
@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 package test

 import kotlinx.parcelize.Parcelize
@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 // WITH_STDLIB
 package test

@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 package test

 import kotlinx.parcelize.Parcelize
@@ -0,0 +1,17 @@
+package test
+
+import kotlinx.parcelize.Parcelize
+import kotlinx.parcelize.RawValue
+import android.os.Parcelable
+
+@Parcelize
+class User(
+        val a: String,
+        val b: Any,
+        val c: Any?,
+        val d: Map<Any, String>,
+        val e: @RawValue Any?,
+        val f: @RawValue Map<String, Any>,
+        val g: Map<String, @RawValue Any>,
+        val h: Map<@RawValue Any, List<@RawValue Any>>
+) : Parcelable
@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 package test

 import kotlinx.parcelize.Parcelize
@@ -1,3 +1,4 @@
+// FIR_IDENTICAL
 package test

 import kotlinx.parcelize.Parcelize