]> git.eshelyaron.com Git - emacs.git/commit
Expand time-stamp unit tests to cover all formatting options
authorStephen Gildea <stepheng+savannah@gildea.com>
Mon, 23 Sep 2019 03:27:53 +0000 (20:27 -0700)
committerStephen Gildea <stepheng+savannah@gildea.com>
Mon, 23 Sep 2019 03:31:59 +0000 (20:31 -0700)
commita33cda168e35cb93cdf115f7d12765f590226601
tree0f668f2163a47da41ad6cde6ce39659478176b5e
parent4eac64fcf21fc54d562d9ac5903ed241c747372b
Expand time-stamp unit tests to cover all formatting options

* time-stamp-tests.el: Expand unit tests to cover all formatting options.
These tests validate time-stamp-pattern formatting that has existed
since at least Emacs 22 (released in 2007).  The tests cover both
documented behavior and behavior implemented to support future migrations.

* time-stamp.el (time-stamp-string): Add a second argument (TIME) to
open a testing seam.  Have the unit tests call this public function.

* time-stamp.el (time-stamp-string, time-stamp-string-preprocess):
Remove the second pass through time-string--format.  (Previously both
functions called it.)  It was used only to handle "%", but this is now
handled by having time-stamp-string-preprocess not double it.
Not doubling the "%" in time-stamp-string-preprocess fixes the padding
of "%2%", which was discovered by the new unit tests to be wrong.
lisp/time-stamp.el
test/lisp/time-stamp-tests.el