[PATCH v2] convert bin/date over to ATF
Julio Merino
jmmv at freebsd.org
Sun Mar 2 21:23:25 UTC 2014
On Mon, Jan 27, 2014 at 5:03 AM, Garrett Cooper <yaneurabeya at gmail.com> wrote:
> The attached patch incorporates the input from Julio for converting bin/date/tests over to ATF.
Apologies for the delay here as well...
Pretty much all style nits:
* This file is full of boilerplate and it's really hard to actually
see the tests. I think this would benefit from a simple helper
function that calls atf_test_case and evals the tiny body() with the
call to check.
* I'd get rid of the descr definitions; they provide no value.
* I'd also get rid of all the horizontal dividing lines. If you make
the test case definitions compact, they just serve to increase the
file length.
* Remove the TZ and LC_ALL overrides. This is a given within ATF.
* Why "format_string_functional_test"? I'd just call it "format_string_test".
* I don't think the # vim trailing line belongs in the tree.
More information about the freebsd-testing
mailing list