[LTP] Rename tst_test_* to tst_require_*

Joerg Vehlow lkml@jv-coder.de
Fri Oct 11 15:39:33 CEST 2019


> On Fri, 2019-10-11 at 12:06 +0200, Petr Vorel wrote:
>> Hi,
>>
>> Cc: Li and Clemens
>>
>>> these patches rename tst_test_* to tst_require_*, to better
>>> describe
>>> their use. There is also tst_require_root, that has the same
>>> behavior: It also calls tst_brk in case of a failing requirement.
>>> You can also get this patch from the following repo:
>>> https://github.com/MofX/ltp/commits/rename_tst_test-tst_require
>> sorry for not thinking first, I wonder if we want to sync
>> tst_test_* (function name) vs $TST_NEEDS_* (test API variable name),
>> e.g.: tst_require_drivers $TST_NEEDS_DRIVERS
> I fully agree with Petr that we must be consistent in naming between
> variable- and function-name.
Actually I am more with cyril's reasoning, that one is metadata and one 
is function call here.
It wasn't consistent in the past. The require is more like an imperative 
here and the needs is
descriptive.
I'm also no native speaker, but in my opinon needing a command sounds a 
bit strange.
One thing I found is that a need is required for living, while a 
requirement must be
fulfilled, for something to happen. With this definition requirement 
fits. There is also the term
requierement in software engineering and no term "needment"

But strangely I still have no problem with TST_NEEDS_*, but I would also 
be ok with
TST_REQUIRES_. Actually I'd propably chose it, if I was designing the 
library.

I do not really like renaming tst_test_* to test_needs_*. Needs does not 
sound
strong enough to me in that context.  But I just made a quick search for 
"needs_"
and there is so much needs in the c library as well, that touching all 
of this is maybe
to much.


More information about the ltp mailing list