[lldb-dev] New test summary results formatter
Zachary Turner via lldb-dev
lldb-dev at lists.llvm.org
Thu Dec 3 10:08:18 PST 2015
Ahh I read further and see this was already mentioned by Pavel.
On Thu, Dec 3, 2015 at 10:06 AM Zachary Turner <zturner at google.com> wrote:
> On Wed, Dec 2, 2015 at 10:20 PM Todd Fiala <todd.fiala at gmail.com> wrote:
>
>> On Wed, Dec 2, 2015 at 9:48 PM, Zachary Turner <zturner at google.com>
>> wrote:
>>
>>>
>>>
>>> On Wed, Dec 2, 2015 at 9:44 PM Todd Fiala <todd.fiala at gmail.com> wrote:
>>>
>>>>
>>>>
>>>>> and the classname could be dropped (there's only one class per file
>>>>> anyway, so the classname is just wasted space)
>>>>>
>>>>
>>>> Part of the reason I included that is I've hit several times where copy
>>>> and paste errors lead to the same class name, method name or even file name
>>>> being used for a test. I think, though, that most of those are addressed
>>>> by having the path (relative is fine) to the python test file. I think we
>>>> can probably get by with classname.methodname (relative test path). (From
>>>> your other email, I think you nuke the classname and keep the module name,
>>>> but I'd probably do the reverse, keeping the class name and getting rid of
>>>> the module name since it can be derived from the filename).
>>>>
>>> I don't think the filename can be the same anymore, as things will break
>>> if two filenames are the same.
>>>
>>
>> Maybe, but that wasn't my experience as of fairly recently. When
>> tracking failures sometime within the last month, I tracked something down
>> in a downstream branch with two same-named files that (with the legacy
>> output) made it hard to track down what was actually failing given the
>> limited info of the legacy test summary output. Maybe that has changed
>> since then, but I'm not aware of anything that would have prohibited that.
>>
> Well I only said "things" will break, not everything will break. Most
> likely you just didn't notice the problem or it didn't present itself in
> your scenario. There are definitely bugs surrounding multiple files with
> the same name, because of some places where we use a dictionary keyed on
> filename.
>
>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20151203/0a4743f8/attachment-0001.html>
More information about the lldb-dev
mailing list