Open
Bug 1206409
Opened 9 years ago
Updated 2 years ago
Some really long subtest names
Categories
(Testing :: General, defect)
Testing
General
Tracking
(Not tracked)
NEW
People
(Reporter: ekyle, Unassigned)
Details
We can now see all the subtests coming from the structured logs. Some of the subtest names are very long, and from my limited perspective, should belong in the subtest message. Maybe the log calls can be fixed?
This query will take long to run (and more nodes would help), but will give you a short list of some recent and egregious subtest names.
http://activedata.allizom.org/tools/query.html#query_id=86PHkVA1
> {
> "from":"unittest.result.subtests",
> "select":[
> {"name":"subtest_name","value":"name"},
> "run.suite",
> "result.test"
> ],
> "where":{"and":[
> {"gt":[{"length":"name"},2000]},
> {"gt":{"run.timestamp":"{{today-day}}"}}
> ]},
> "limit":10,
> "format":"list"
> }
Reporter | ||
Comment 1•9 years ago
|
||
For example:
https://dxr.mozilla.org/mozilla-central/source/dom/xul/test/test_bug398289.html#34
from
> ok(equal, "persistent attribute in tab box broken, expected: "+str1+" got: "+str2);
to
> ok(equal, "persistent attribute in tab box broken", "expected: "+str1+" got: "+str2);
Comment 2•9 years ago
|
||
This is a consequence of the integration of mochitest and mozlog structured logging. Individual tests weren't rewritten as a part of this integration, so in cases like the assertions calls quoted in comment 1 the existing messages are used for identification.
Comment 3•9 years ago
|
||
If there's a need to go through these and clean them up, I don't think there will be any resistance beyond the manual effort required.
Reporter | ||
Comment 4•9 years ago
|
||
I did not expect this to be a priority unless the auto-starring effort, or some other initiative, requires them to be cleaner.
Thanks
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•