Skip to content

Commit 06149ff

Browse files
committed
Merge branch '2.12' into 2.13
2 parents 3933aeb + 00f8432 commit 06149ff

File tree

2 files changed

+53
-43
lines changed

2 files changed

+53
-43
lines changed

release-notes/VERSION-2.x

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ Co-maintainers:
1818

1919
No changes since 2.12
2020

21-
2.12.2 (not yet released)
21+
2.12.2 (03-Mar-2021)
2222

2323
#409: `module-info.java` missing "exports"
2424
(reported by Elisha P)

src/test/kotlin/com/fasterxml/jackson/module/kotlin/test/github/Github145.kt

+52-42
Original file line numberDiff line numberDiff line change
@@ -17,57 +17,68 @@ class TestGithub145 {
1717
@JsonProperty("preName") val preName: String,
1818
@JsonProperty("lastName") val lastName: String
1919
) {
20-
constructor(preNameAndLastName:String): this(preNameAndLastName.substringBefore(","),preNameAndLastName.substringAfter(","))
20+
constructor(preNameAndLastName: String) : this(
21+
preNameAndLastName.substringBefore(","),
22+
preNameAndLastName.substringAfter(",")
23+
)
2124
}
2225

2326
val objectMapper = ObjectMapper()
24-
val personA = objectMapper.readValue("""{"preName":"TestPreName","lastName":"TestLastname"}""", Person1::class.java)
27+
val personA =
28+
objectMapper.readValue("""{"preName":"TestPreName","lastName":"TestLastname"}""", Person1::class.java)
2529
val personB = objectMapper.readValue(""""TestPreName,TestLastname"""", Person1::class.java)
2630
}
2731

2832
@Test
29-
fun testPersonGood2() {
30-
class PersonGood1(@JsonProperty("preName") val preName:String, @JsonProperty("lastName") val lastName:String)
31-
{
32-
@JsonCreator constructor(preNameAndLastName:String): this(preNameAndLastName.substringBefore(","),preNameAndLastName.substringAfter(","))
33+
fun testPerson2() {
34+
class Person2(
35+
@JsonProperty("preName") val preName: String,
36+
@JsonProperty("lastName") val lastName: String
37+
) {
38+
@JsonCreator
39+
constructor(preNameAndLastName: String) : this(
40+
preNameAndLastName.substringBefore(","),
41+
preNameAndLastName.substringAfter(",")
42+
)
3343
}
3444

35-
val personGood1String = objectMapper.readValue<PersonGood1>(""""TestPreName,TestLastname"""")
36-
val personGood1Json =
37-
objectMapper.readValue<PersonGood1>("""{"preName":"TestPreName","lastName":"TestLastname"}""")
45+
val person1String = objectMapper.readValue<Person2>(""""TestPreName,TestLastname"""")
46+
val person1Json = objectMapper.readValue<Person2>("""{"preName":"TestPreName","lastName":"TestLastname"}""")
3847
}
3948

4049
@Test
41-
fun testPersonGood3() {
42-
class PersonGood2(val preName:String, val lastName:String)
43-
{
44-
@JsonCreator constructor(preNameAndLastName:String): this(preNameAndLastName.substringBefore(","),preNameAndLastName.substringAfter(","))
50+
fun testPerson3() {
51+
class Person3(val preName: String, val lastName: String) {
52+
@JsonCreator
53+
constructor(preNameAndLastName: String) : this(
54+
preNameAndLastName.substringBefore(","),
55+
preNameAndLastName.substringAfter(",")
56+
)
4557
}
4658

47-
val personGood2String = objectMapper.readValue<PersonGood2>(""""TestPreName,TestLastname"""")
48-
val personGood2Json =
49-
objectMapper.readValue<PersonGood2>("""{"preName":"TestPreName","lastName":"TestLastname"}""")
59+
val person2String = objectMapper.readValue<Person3>(""""TestPreName,TestLastname"""")
60+
val person2Json = objectMapper.readValue<Person3>("""{"preName":"TestPreName","lastName":"TestLastname"}""")
5061
}
5162

5263
@Test
53-
fun testPersonGood4() {
54-
class PersonGood4(preNameAndLastName: String) {
55-
val preName:String
56-
val lastName:String
64+
fun testPerson4() {
65+
class Person4(preNameAndLastName: String) {
66+
val preName: String
67+
val lastName: String
5768

5869
init {
5970
this.preName = preNameAndLastName.substringBefore(",")
6071
this.lastName = preNameAndLastName.substringAfter(",")
6172
}
6273
}
6374

64-
val personGood4String = objectMapper.readValue<PersonGood4>(""""TestPreName,TestLastname"""")
75+
val person4String = objectMapper.readValue<Person4>(""""TestPreName,TestLastname"""")
6576
// person4 does not have parameter bound constructor, only string
6677
}
6778

6879
@Test
69-
fun testPersonGood5() {
70-
class PersonGood5 @JsonCreator constructor(
80+
fun testPerson5() {
81+
class Person5 @JsonCreator constructor(
7182
@JsonProperty("preName") val preName: String,
7283
@JsonProperty("lastName") val lastName: String
7384
) {
@@ -76,13 +87,12 @@ class TestGithub145 {
7687
this(preNameAndLastName.substringBefore(","), preNameAndLastName.substringAfter(","))
7788
}
7889

79-
val personGood5String = objectMapper.readValue<PersonGood5>(""""TestPreName,TestLastname"""")
80-
val personGood5Json =
81-
objectMapper.readValue<PersonGood5>("""{"preName":"TestPreName","lastName":"TestLastname"}""")
90+
val person5String = objectMapper.readValue<Person5>(""""TestPreName,TestLastname"""")
91+
val person5Json = objectMapper.readValue<Person5>("""{"preName":"TestPreName","lastName":"TestLastname"}""")
8292
}
8393

8494
// Cannot have companion object in class declared within function
85-
class PersonGood6 private constructor(val preName: String, val lastName: String) {
95+
class Person6 private constructor(val preName: String, val lastName: String) {
8696
private constructor(preNameAndLastName: String) : this(
8797
preNameAndLastName.substringBefore(","),
8898
preNameAndLastName.substringAfter(",")
@@ -91,26 +101,26 @@ class TestGithub145 {
91101
companion object {
92102
@JsonCreator
93103
@JvmStatic
94-
fun createFromJson(preNameAndLastName: String): PersonGood6 {
95-
return PersonGood6(preNameAndLastName)
104+
fun createFromJson(preNameAndLastName: String): Person6 {
105+
return Person6(preNameAndLastName)
96106
}
97107

98108
@JsonCreator
99109
@JvmStatic
100-
fun createFromData(preName: String, lastName: String): PersonGood6 {
101-
return PersonGood6(preName, lastName)
110+
fun createFromData(preName: String, lastName: String): Person6 {
111+
return Person6(preName, lastName)
102112
}
103113
}
104114
}
115+
105116
@Test
106-
fun testPersonGood6() {
107-
val personGood6String = objectMapper.readValue<PersonGood6>(""""TestPreName,TestLastname"""")
108-
val personGood6Json =
109-
objectMapper.readValue<PersonGood6>("""{"preName":"TestPreName","lastName":"TestLastname"}""")
117+
fun testPerson6() {
118+
val person6String = objectMapper.readValue<Person6>(""""TestPreName,TestLastname"""")
119+
val person6Json = objectMapper.readValue<Person6>("""{"preName":"TestPreName","lastName":"TestLastname"}""")
110120
}
111121

112122
// Cannot have companion object in class declared within function
113-
class PersonGood7 constructor(val preName: String, val lastName: String) {
123+
class Person7 constructor(val preName: String, val lastName: String) {
114124
private constructor(preNameAndLastName: String) : this(
115125
preNameAndLastName.substringBefore(","),
116126
preNameAndLastName.substringAfter(",")
@@ -119,15 +129,15 @@ class TestGithub145 {
119129
companion object {
120130
@JsonCreator
121131
@JvmStatic
122-
fun createFromJson(preNameAndLastName: String): PersonGood7 {
123-
return PersonGood7(preNameAndLastName)
132+
fun createFromJson(preNameAndLastName: String): Person7 {
133+
return Person7(preNameAndLastName)
124134
}
125135
}
126136
}
137+
127138
@Test
128-
fun testPersonGood7() {
129-
val personGood7String = objectMapper.readValue<PersonGood7>(""""TestPreName,TestLastname"""")
130-
val personGood7Json =
131-
objectMapper.readValue<PersonGood7>("""{"preName":"TestPreName","lastName":"TestLastname"}""")
139+
fun testPerson7() {
140+
val person7String = objectMapper.readValue<Person7>(""""TestPreName,TestLastname"""")
141+
val person7Json = objectMapper.readValue<Person7>("""{"preName":"TestPreName","lastName":"TestLastname"}""")
132142
}
133143
}

0 commit comments

Comments
 (0)