NetBSD-Bugs archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: misc/45038: lack of tests options
On 6/16/11 3:52 PM, Nicolas Joly wrote:
> On Thu, Jun 16, 2011 at 02:44:21PM +0100, Julio M. Merino Vidal wrote:
>> On 6/16/11 2:25 PM, Nicolas Joly wrote:
>>> Hi,
>>>
>>> Does the following patch fit your needs ?
>>
>> Looks nice! Some minor comments:
>>
>>> Index: share/man/man7/tests.7
>>> ===================================================================
>>> RCS file: /cvsroot/src/share/man/man7/tests.7,v
>>> retrieving revision 1.4
>>> diff -u -p -r1.4 tests.7
>>> --- share/man/man7/tests.7 7 Nov 2010 17:49:33 -0000 1.4
>>> +++ share/man/man7/tests.7 16 Jun 2011 13:19:28 -0000
>>
>> Don't forget to bump the date.
>
> Will do. Thanks for the reminder.
>
>>> @@ -163,6 +163,16 @@ The suite-specific configuration file im
>>> which contains properties shared among all test suites.
>>> These files conform to the configuration file format described in
>>> .Xr atf-formats 5 .
>>> +.Pp
>>> +The following configuration variables are available in the
>>> +.Nx
>>> +test suite :
>>> +.Bl -tag -width "fstype"
>>> +.It fstype
>>> +When set to a filesystem type, restrict tests programs from the
>>> +.Pa /usr/tests/fs/vfs
>>
>> I'd add a trailing slash here to denote a directory.
>
> Done.
>
>>> +tree to only run test cases for the given type.
>>> +.El
>>
>> Also document the unprivileged_user option. It's used explicitly by
>> some tests. (It defaults to _atf.)
>
> I don't find it anywhere. Is it "require.user" = "unprivileged" ?
Ah sorry, it's unprivileged-user.
> IMHO, we should only document in tests(7) the configuration variables
> which are not part of ATF.
Well, I guess this one is a bit of a compromise. unprivileged-user was
_not_ an atf-specific variable; it used to be defined by NetBSD only.
These days, it has a special meaning in atf itself, but some NetBSD
tests query it directly to do their own stuff. In other words, it is
not only used by require.user=unprivileged, but also to perform some
specific actions in the tmpfs tests.
--
Julio Merino / @jmmv
Home |
Main Index |
Thread Index |
Old Index