KeyEmpty value causes yamlKT parser to nest susequent key/value pairs into Empty field

[philip0402]

Example of Bug:

  val validYaml = "id: lgcbkyz4c6b5gnfsupczfkn\n" + 
                             "title: Truth\n" + 
                             "desc: \n" + 
                             "updated: 1704581169935\n" + 
                             "created: 1704049864411\n"
   val map: YamlMap = Yaml.decodeYamlMapFromString(validYaml)
   println("Parsed YAML map: $map")

Expected output:

ParsedYAMLmap:{id:lgcbkyz4c6b5gnfsupczfkn,title:Truth,desc: ,updated:1704581169935,created:1704049864411}

Actual output:

ParsedYAMLmap:{id:lgcbkyz4c6b5gnfsupczfkn,title:Truth,desc:{updated:1704581169935,created:1704049864411}}

Test which reproduces bug:

@Test
    fun testBlankDescription() {
        val validYaml = "id: lgcbkyz4c6b5gnfsupczfkn\n" + "title: Truth\n" + "desc: \n" + "updated: 1704581169935\n" + "created: 1704049864411\n"
        val map: YamlMap = Yaml.decodeYamlMapFromString(validYaml)
        println("Parsed YAML map: $map")
        assertTrue { map.containsKey(YamlPrimitive("desc")) }
        assertTrue { map.containsKey(YamlPrimitive("updated")) }
        assertTrue { map.containsKey(YamlPrimitive("created")) }
    }

Test Results:

Pass:

• assertTrue { map.containsKey(YamlPrimitive("desc")) }

Assertion Errors:

• assertTrue { map.containsKey(YamlPrimitive("updated")) }
• assertTrue { map.containsKey(YamlPrimitive("created")) }

Debugging Logic:

Function decodeYamlMapFromString is called. Stepping into the function implementation.

Function decodeYamlMapFromString calls decodeFromString. Stepping into the function implementation.

Note: The function was edited for debugging purposes. val yamlMap was created to keep track of the return value of the deserialize function call.

 deserializer.deserialize(YamlDecoder(configuration, TokenStream(string), serializersModule))

Function deserialize is called. Stepping into the function implementation. The program steps into the conditional code block and calls the deserialize function from the kotlinx library.

Results:

The issue seems to come from the parsing done by the deserialize function call made from the kotlinx library.

Moving forward:

The yamlkt parser should explicitly address the issue of empty values in the key/value mappings.