gluesql / gluesql / 25985361787

99%

push

github

Hide chrono::ParseError from EvaluateError::FormatParseError (#1913)

Store the parse error as an owned String instead of carrying `chrono::format::ParseError` through the public error surface.
This removes the bespoke serializer helper and the `#[allow(clippy::trivially_copy_pass_by_ref)]` introduced in #1838.

Construction sites in evaluate/function.rs now call `EvaluateError::FormatParseError(err.to_string())`.
Tests in test-suite that matched on `chrono::format::ParseErrorKind` were relaxed to assert the FormatParseError variant; the prior `.kind()` checks were effectively testing chrono's parser rather than gluesql.

10 of 12 new or added lines in 3 files covered. (83.33%)

43259 of 44082 relevant lines covered (98.13%)

72574.52 hits per line