DeRuina / timberjack / 26469706198

89%

push

github

fix(format): detect fractional seconds correctly when validating BackupTimeFormat (#108)

ValidateBackupTimeFormat truncated the reference time to the precision returned by
countDigitsAfterDot, which counted the digits after the first '.' anywhere in the layout.
When a custom BackupTimeFormat used '.' as a separator (e.g. "2006-01-02_15.04.05") rather
than as a fractional-second indicator, the precision was mis-detected, the round-trip check
failed, and the format was silently rejected — falling back to the default format.

Replace countDigitsAfterDot with fractionalSecondDigits, which counts a '.' or ',' followed
by a run of '0's or '9's (how Go denotes fractional seconds in a layout), so '.' used as a
separator is no longer mistaken for fractional seconds.

Co-authored-by: Dean Ruina <81315494+DeRuina@users.noreply.github.com>

18 of 18 new or added lines in 1 file covered. (100.0%)

762 of 850 relevant lines covered (89.65%)

43.11 hits per line