Closed
Bug 1491390
Opened 6 years ago
Closed 6 years ago
Use only test name without it's path when searching for baseline coverage tests.
Categories
(Testing :: Code Coverage, enhancement)
Tracking
(firefox64 fixed)
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: sparky, Assigned: sparky)
Details
Attachments
(1 file)
No description provided.
Assignee | ||
Comment 1•6 years ago
|
||
This bug is to fix when baseline coverage tests are found, like in this chunk: https://treeherder.mozilla.org/#/jobs?repo=try&selectedJob=199296465&revision=8076f4afb29f82e5321906ce3feadf6ec9577ace It requires us to search only the test file name, excluding the path to the test, for the test suite name if it exists.
Assignee | ||
Comment 2•6 years ago
|
||
This patch fixes an issue where a suite was being found in the full path to the test which led to the baseline coverage tests being considered on a per-suite basis (like browser-chrome), when it should be on a per-file-extension basis for '.html' files. This is fixed by only considering the test name and excluding the full path to it.
Assignee | ||
Comment 3•6 years ago
|
||
Here's a passing test run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=4c263ea90d6502a23faa6afea3539bdf7a38a8da And here's a failing test run before applying the patch: https://treeherder.mozilla.org/#/jobs?repo=try&revision=731d6b1304f4f0baca49e158b09c55258023dbe1
Comment 4•6 years ago
|
||
Comment on attachment 9009207 [details] Bug 1491390 - When searching for baseline coverage tests, only look at the test name. r?jmaher Joel Maher ( :jmaher ) (UTC-4) has approved the revision.
Attachment #9009207 -
Flags: review+
Pushed by jmaher@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/368a1b20a52c When searching for baseline coverage tests, only look at the test name. r=jmaher
Comment 6•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/368a1b20a52c
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Updated•6 years ago
|
Assignee: nobody → gmierz2
You need to log in
before you can comment on or make changes to this bug.
Description
•