tarantool / go-config / 36549127496

83%

Build Type

push

github

Committed by bigbes

Commit Message

tarantool: read an empty YAML value as a string

YAML resolves a scalar with no content, such as the value of `console:`,
to null, and so does the YAML collector. Tarantool's YAML decoder reads
it as "" instead. The two disagreed in both directions: the validator
coerced the null `console` to {} and accepted a config that Tarantool
rejects with "Unexpected data type for a record", and it rejected an
empty `console.socket:` as null while Tarantool accepts it as "".

collectors.NewYamlFormat now takes options, and EmptyAsString reads an
empty scalar as "" while `~` and `null` stay null. The Tarantool builder
uses it for the config file, the config directory and the storage, so
it reads a config the way Tarantool does. The default YAML format keeps
following the YAML spec.

Coverage Stats

33 of 33 new or added lines in 2 files covered. (100.0%)

85 existing lines in 7 files now uncovered.

4684 of 5300 relevant lines covered (88.38%)

1357.04 hits per line