[Gluster-devel] A suggestion on naming test files.

Lalatendu Mohanty lmohanty at redhat.com
Wed Aug 27 10:02:01 UTC 2014


On 08/27/2014 03:29 PM, Lalatendu Mohanty wrote:
> On 08/27/2014 02:13 PM, Niels de Vos wrote:
>> On Wed, Aug 27, 2014 at 01:50:40PM +0530, Kaushal M wrote:
>>> I'll do that, if someone could point me to it. Is it on the community
>>> wiki or in doc/ directory in the source?
>> Seems to be only (?) on the wiki:
>> - 
>> http://www.gluster.org/community/documentation/index.php/Using_the_Gluster_Test_Framework#Creating_your_own_tests
>>
>> Niels
>
> We should also update README.md file in source code i.e. glusterfs/tests.
>

+ Forgot to add that README.md does not a section about writing tests, 
either we add a new section or we can update the wiki page and update 
the wiki link in the README file.
> -Lala
>>> ~kaushal
>>>
>>> On Wed, Aug 27, 2014 at 1:12 PM, Niels de Vos <ndevos at redhat.com> 
>>> wrote:
>>>> On Wed, Aug 27, 2014 at 12:21:56PM +0530, Kaushal M wrote:
>>>>> Hi all,
>>>>>
>>>>> Currently we name tests associated with a bug with just the bug-id as
>>>>> 'bug-XXXXXXX.t'. This naming doesn't provide any context directly, 
>>>>> and
>>>>> someone looking at just the name wouldn't know what it does.
>>>>>
>>>>> I suggest that we use slightly more descriptive names to the tests.
>>>>> Lets name the tests as 'bug-XXXXXXX-checking-if-feature-Y-works.t'.
>>>>> This provides just enough context to understand what the test is
>>>>> trying to do from just the name.
>>>>>
>>>>> How do you all feel about this?
>>>> Sounds good to me. Please make sure to update the documentation about
>>>> writing test-cases in a few days, if nobody objects.
>>>>
>>>> Thanks,
>>>> Niels
>> _______________________________________________
>> Gluster-devel mailing list
>> Gluster-devel at gluster.org
>> http://supercolony.gluster.org/mailman/listinfo/gluster-devel
>
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel at gluster.org
> http://supercolony.gluster.org/mailman/listinfo/gluster-devel



More information about the Gluster-devel mailing list