[PATCH v2] convert /bin/sh tests over to ATF

Julio Merino jmmv at freebsd.org
Sun Mar 2 21:14:51 UTC 2014


On Mon, Jan 27, 2014 at 5:02 AM, Garrett Cooper <yaneurabeya at gmail.com> wrote:
> Hi all,
>         Here's the next patch for converting bin/sh/tests over to ATF. I've tried to incorporate all input from folks about simplifying the integration scripts and removing valid_osreldate functionality.

Apologies for the really long delay... I suppose nothing has changed
given that there were no replies to this. Soooo, some more comments:

* Could functional_test be created with ATF_TESTS_SH ?

* Rename the script in subversion to functional_test.sh (including the
extension).

* Do NOT cd out of the current work directory in functional_test,
anywhere.  This breaks all the assumptions in tests regarding the
ability to write to the current work directory without caring about
cleaning up.

* Use atf_get_srcdir to locate where the data files are, not dirname $0.

* Continue to use SUBDIR, not ATF_TESTS_SUBDIRS. The latter does not exist.

* Reintroduce $FreeBSD$ to functional_test.sh.

* The empty Makefiles in all the subdirectories are quite confusing.
At the very least, explicitly define TESTSDIR as is customary
everywhere else.

* I'm still afraid of all this "auto-discovery" of test cases based on
files, because it is quite confusing...  but at least it's now a bit
more self-contained and easier to strip out later.  So we could keep
it.

* The FILES must continue to be installed. Does this actually work
when running your test programs after a "make install"?

Cheers


More information about the freebsd-testing mailing list