[Gluster-devel] 1st results: gcov/lcov code coverage of glusterfs

[Vijay Bellur]

On 05/18/2013 08:05 PM, John Smith wrote:
> Hi,
>
>
> Ah, thanks, I was looking in the wrong place. I was worried that he
> results was invalid there for a moment.
>
> Perhaps the directory '/usr/local/glusterfs/*' should be filtered out
> of the report entirely ?

Yes, filtering that out could help.

Thanks,
Vijay

>
>
> Thanks again,
>
>
> Regards,
>
>
> John Smith.
>
> On Sat, May 18, 2013 at 3:45 PM, Vijay Bellur <vbellur at redhat.com> wrote:
>> On 05/17/2013 02:01 AM, John Smith wrote:
>>>
>>> Hi,
>>>
>>>
>>> Im not quite sure what im seeing here, but ...
>>>
>>> the prove test './tests/basic/posixonly.t' should test, well, posix ;).
>>>
>>> and yet the gcov report says its not being hit:
>>>
>>> http://lbalbalba.x90x.net/lcov/glusterfs/usr/local/glusterfs/lib/glusterfs/3git/xlator/storage/posix.c.gcov.html
>>>
>>> Does anyone who has a deeper understanding of both the codebase and
>>> the test know if that test should indeed hit that file ? If so, the
>>> report is wrong. And if not, the test may need to be changed.
>>>
>>
>> The test should indeed hit posix.c. Wouldn't this be indicative of what
>> lines were covered by this test:
>>
>> http://lbalbalba.x90x.net/lcov/glusterfs/xlators/storage/posix/src/posix.c.gcov.html
>>
>> Thanks,
>> Vijay
>>
>
>