Fixed old StrictNullChecks to throw exceptions similar to those thrown by new StrictNullChecks

[k163377]

To deprecate the old StrictNullChecks option, changes were made to make the exceptions thrown by the old process the similar to the new StrictNullChecks.
This means that the old StrictNullChecks will no longer throw MissingKotlinParameterException.

In the situation where this error occurs, it was inappropriate to use MissingKotlinParameterException because the input was not missing.
This is also relevant for #617.

---

The following is a comparison of the respective messages displayed when printStackTrace is performed on the result of assertThrows for com.fasterxml.jackson.module.kotlin.test.StrictNullChecksTest(NewStrictNullChecks) and StrictNullChecksTestOld.
As for messages, it has been improved so that even the key of a value that was null can be checked.

ClassWithArrayOfInt

NewStrictNullChecks

com.fasterxml.jackson.databind.exc.InvalidNullException: Invalid `null` value encountered for property "samples"
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 16] (through reference chain: com.fasterxml.jackson.module.kotlin.test.StrictNullChecksTest$ClassWithArrayOfInt["samples"]->java.lang.Object[][1])

Fixed old StrictNullChecks

com.fasterxml.jackson.databind.exc.InvalidNullException: Invalid `null` value encountered for property "samples"
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 21] (through reference chain: com.fasterxml.jackson.module.kotlin.test.StrictNullChecksTestOld$ClassWithArrayOfInt["samples"]->java.lang.Integer[][1])

Conventional old StrictNullChecks

com.fasterxml.jackson.module.kotlin.MissingKotlinParameterException: Instantiation of kotlin.Int kotlin.Array<kotlin.Int> failed for JSON property samples due to null value in a kotlin.Array<kotlin.Int> that does not allow null values
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 21] (through reference chain: com.fasterxml.jackson.module.kotlin.test.StrictNullChecksTestOld$ClassWithArrayOfInt["samples"])

ClassWithListOfInt

NewStrictNullChecks

com.fasterxml.jackson.databind.exc.InvalidNullException: Invalid `null` value encountered for property "samples"
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 16] (through reference chain: com.fasterxml.jackson.module.kotlin.test.StrictNullChecksTest$ClassWithListOfInt["samples"]->java.util.ArrayList[1])

Fixed old StrictNullChecks

com.fasterxml.jackson.databind.exc.InvalidNullException: Invalid `null` value encountered for property "samples"
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 21] (through reference chain: com.fasterxml.jackson.module.kotlin.test.StrictNullChecksTestOld$ClassWithListOfInt["samples"]->java.util.ArrayList[1])

Conventional old StrictNullChecks

com.fasterxml.jackson.module.kotlin.MissingKotlinParameterException: Instantiation of kotlin.Int kotlin.collections.List<kotlin.Int> failed for JSON property samples due to null value in a kotlin.collections.List<kotlin.Int> that does not allow null values
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 21] (through reference chain: com.fasterxml.jackson.module.kotlin.test.StrictNullChecksTestOld$ClassWithListOfInt["samples"])

ClassWithMapOfStringToInt

NewStrictNullChecks

com.fasterxml.jackson.databind.exc.InvalidNullException: Invalid `null` value encountered for property "samples"
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 23] (through reference chain: com.fasterxml.jackson.module.kotlin.test.StrictNullChecksTest$ClassWithMapOfStringToInt["samples"]->java.util.LinkedHashMap["key"])

Fixed old StrictNullChecks

com.fasterxml.jackson.databind.exc.InvalidNullException: Invalid `null` value encountered for property "samples"
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 30] (through reference chain: com.fasterxml.jackson.module.kotlin.test.StrictNullChecksTestOld$ClassWithMapOfStringToInt["samples"]->java.util.LinkedHashMap["key"])

Conventional old StrictNullChecks

com.fasterxml.jackson.module.kotlin.MissingKotlinParameterException: Instantiation of kotlin.Int kotlin.collections.Map<kotlin.String, kotlin.Int> failed for JSON property samples due to null value in a kotlin.collections.Map<kotlin.String, kotlin.Int> that does not allow null values
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 30] (through reference chain: com.fasterxml.jackson.module.kotlin.test.StrictNullChecksTestOld$ClassWithMapOfStringToInt["samples"])

[Copilot]

Pull Request Overview

This PR aligns the old strict-null-check behavior with the new implementation by replacing MissingKotlinParameterException with InvalidNullException and updating the value instantiator to throw InvalidNullException for null elements in collections, maps, and arrays.

• Tests now expect InvalidNullException instead of MissingKotlinParameterException.
• KotlinValueInstantiator imports and uses InvalidNullException.from(...) to construct and wrap exceptions when encountering null items.
• Removed the old branching logic and replaced it with a when block that creates and throws the new exception type.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/test/kotlin/com/fasterxml/jackson/module/kotlin/test/StrictNullChecksTestOld.kt	Updated tests to assert InvalidNullException and removed the old exception import.
src/main/kotlin/com/fasterxml/jackson/module/kotlin/KotlinValueInstantiator.kt	Added import for InvalidNullException and replaced the old null-check branches with logic that constructs and throws InvalidNullException.

Comments suppressed due to low confidence (2)

src/main/kotlin/com/fasterxml/jackson/module/kotlin/KotlinValueInstantiator.kt:109

• [nitpick] Variable name ex is quite generic. Rename it to something more descriptive like nullException to clarify its purpose.

                val ex = when {

src/test/kotlin/com/fasterxml/jackson/module/kotlin/test/StrictNullChecksTestOld.kt:35

• Consider extending this test to verify that the thrown InvalidNullException includes the correct JSON path or property name/index, ensuring the wrapping behavior is fully covered.

        assertThrows<InvalidNullException> {

[Copilot]

[nitpick] The when block that builds InvalidNullException contains repeated patterns for collection and array cases. Consider extracting a helper function to handle index-based null checks and exception wrapping to reduce duplication and improve readability.