From: Theodore Ts'o <tytso@mit.edu>
To: linux-ext4@vger.kernel.org, adilger.kernel@dilger.ca
Subject: Re: A question about the naming of e2fsprogs/tests
Date: Wed, 17 Oct 2012 20:19:24 -0400 [thread overview]
Message-ID: <20121018001924.GA18677@thunk.org> (raw)
In-Reply-To: <20121017131351.GA22794@gmail.com>
On Wed, Oct 17, 2012 at 09:13:51PM +0800, Zheng Liu wrote:
>
> Now I am trying to create some regression tests for e2fsprogs to test
> inline data feature. I have a question about the naming of
> e2fsprogs/tests. I notice that in tests dir there are a lot of
> directories, which have different prefixes, such as d_loaddump,
> e_brel_bma, f_baddir, m_dasd_bs, r_move_itable, t_mmp_1on, and u_mke2fs.
> As far as I can understand, d_ is for debugfs, f_ means test for filesystem,
> m_ is for mke2fs, r_ is for resize2fs, t_ is for tune2fs, and u_ is for e2undo.
> Could anyone tell me what e_ stands for? If I misunderstand something, please
> point it out. Thanks.
The e_ calls were for unit tests for the "extent" functions --- where
extents are not the ext4 extents, but some utility functions which
were are used by resize2fs. These days we generally don't use the
tests directory for unit tests, because it's a lot more work to do
things that way. Instead there are tst_* files in lib/ext2fs that are
built and run with "make check".
I *do* ask that people who create new functions in libext2fs strongly
consider adding unit tests. We are pretty good with integration tests
in the tests directory, but we are a bit light on unit tests coverage
in lib/ext2fs.
And of course, if someone is inspired to write some new unit tests for
existing functions, that would just great.
Thanks!!
- Ted
prev parent reply other threads:[~2012-10-18 0:19 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-17 13:13 A question about the naming of e2fsprogs/tests Zheng Liu
2012-10-18 0:19 ` Theodore Ts'o [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20121018001924.GA18677@thunk.org \
--to=tytso@mit.edu \
--cc=adilger.kernel@dilger.ca \
--cc=linux-ext4@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).