Upgrade Kotlin version and use more idiomatic Kotlin (#1590)

* Upgrade Kotlin version and use more idiomatic Kotlin

* Add new line at end of file (review comment)

* Update changelog

Author: mlvandijk

CHANGELOG.md

@@ -15,6 +15,7 @@ Please see [CONTRIBUTING.md](https://github.com/cucumber/cucumber/blob/master/CO
     * Allow parameter-types in escaped optional groups ([#cucumber/572](https://github.com/cucumber/cucumber/pull/572), [#cucumber/561](https://github.com/cucumber/cucumber/pull/561) Luke Hill, Jayson Smith, M.P. Korstanje)
     * Prefer expression with the longest non-empty match ([#cucumber/580](https://github.com/cucumber/cucumber/pull/580) M.P. Korstanje)
     * Improve heuristics for creating Cucumber/Regular Expressions from strings ([#cucumber/518](https://github.com/cucumber/cucumber/pull/518) Aslak Hellesøy)
+ *  [Kotlin-Java8] Upgrade Kotlin to v1.3.0 and more idiomatic Kotlin ([#1590](https://github.com/cucumber/cucumber-jvm/pull/1590) Marit van Dijk)
 
 ### Deprecated
 

kotlin-java8/pom.xml

@@ -12,7 +12,7 @@
     <name>Cucumber-JVM: Kotlin Java8</name>
 
     <properties>
-        <kotlin.version>1.2.41</kotlin.version>
+        <kotlin.version>1.3.0</kotlin.version>
     </properties>
 
     <dependencies>

kotlin-java8/src/test/kotlin/cucumber/runtime/kotlin/test/LambdaStepdefs.kt

@@ -50,7 +50,7 @@ class LambdaStepdefs : En {
 
         Given("A statement with a body expression$") { assertTrue(true) }
 
-        Given("A statement with a simple match$", { -> assertTrue(true) })
+        Given("A statement with a simple match$", { assertTrue(true) })
 
         val localInt = 1
         Given("A statement with a scoped argument$", { assertEquals(2, localInt + 1) })
@@ -62,9 +62,6 @@ class LambdaStepdefs : En {
             assertEquals(4, d)
         }
     }
-
-    class Person {
-        var first: String? = null
-        var last: String? = null
-    }
 }
+
+data class Person(val first: String?, val last: String?)

kotlin-java8/src/test/kotlin/cucumber/runtime/kotlin/test/RunCukesTest.kt

@@ -4,5 +4,4 @@ import cucumber.api.junit.Cucumber
 import org.junit.runner.RunWith
 
 @RunWith(Cucumber::class)
-class RunCukesTest {
-}
+class RunCukesTest

kotlin-java8/src/test/kotlin/cucumber/runtime/kotlin/test/TypeRegistryConfiguration.kt

@@ -15,13 +15,10 @@ class TypeRegistryConfiguration : TypeRegistryConfigurer {
 
     override fun configureTypeRegistry(typeRegistry: TypeRegistry) {
         typeRegistry.defineDataTableType(DataTableType(
-                LambdaStepdefs.Person::class.java,
-                TableEntryTransformer<LambdaStepdefs.Person>
+                Person::class.java,
+                TableEntryTransformer<Person>
                 { map: Map<String, String> ->
-                    val person = LambdaStepdefs.Person()
-                    person.first = map.get("first")
-                    person.last = map.get("last")
-                    person
+                    Person(map["first"], map["last"])
                 }))
     }
 }