add comments

Author: Patrick Jackson

common/build.gradle

@@ -15,7 +15,7 @@ kotlin {
         js()
         jvm()
         //  iOS commented out b/c appling test to JS & iOS does not work.  Comment JS tests for iOS to work
-
+        /*
         def buildForDevice = project.findProperty("device")?.toBoolean() ?: false
         def iosPreset = (buildForDevice) ? presets.iosArm64 : presets.iosX64
         fromPreset(iosPreset, 'ios') {
@@ -28,6 +28,7 @@ kotlin {
                 }
             }
         }
+         */
 
     }
     sourceSets {
@@ -62,14 +63,15 @@ kotlin {
                 implementation kotlin("stdlib-js")
             }
         }
-        iosTest {
-            dependsOn commonTest
-        }
+//        iosTest {
+//            dependsOn commonTest
+//        }
 
     }
 
 }
 
+/*
 
 task iosTest {
     def device = project.findProperty("iosDevice")?.toString() ?: "iPhone 8"
@@ -101,13 +103,14 @@ task copyFramework {
     }
 }
 
+ */
+
 
 
 // workaround for https://youtrack.jetbrains.com/issue/KT-27170
 configurations {
     compileClasspath
 }
-/*
 //Workaround to copy kotlin libraries so they are visible during testing
 def jsLibDir = "$compileKotlinJs.destinationDir/lib"
 def jsTestLibDir = "$compileTestKotlinJs.destinationDir/lib"
@@ -152,4 +155,3 @@ task runMocha(type: NodeTask, dependsOn: [installMocha, jsMainClasses, jsTestCla
 }
 jsTest.dependsOn runMocha
 
- */

common/src/test/kotlin/FunctionCheckTest.kt

@@ -0,0 +1,45 @@
+import kotlin.test.Test
+import kotlin.test.fail
+
+typealias Dispatch = (Any) -> Any
+typealias GetState = () -> Any
+typealias Thunk = (Dispatch, GetState, Any?) -> Any
+
+data class AppState(val string: String)
+
+val dispatch: Dispatch = { }
+val getState: GetState = { AppState("myState") }
+
+fun myThunk(dispatch: Dispatch, getState: GetState, arg: Any?) {
+    println("Inside thunk")
+}
+
+
+fun multiParamFunctionTest(foo: Any) {
+    if (foo is Function<*>) {
+        (foo as Thunk)(dispatch, getState, null)
+    } else {
+        fail()
+    }
+}
+
+
+/**
+ * Demonstrates bug with `is Function<*>.  Passes on JVM & Native, fails on JS
+ */
+class FunctionCheckTest {
+
+
+    @Test
+    fun passIsCheck() {
+        if ({ str: String -> } is Function<*>) {
+        } else {
+            fail("Function not recognized in `is Function<*>`")
+        }
+    }
+
+    @Test
+    fun typeAliasCheck() {
+        multiParamFunctionTest(::myThunk)
+    }
+}