Closed
Bug 937953
Opened 12 years ago
Closed 12 years ago
Bug numbers should be removed from all Scratchpad test file names
Categories
(DevTools Graveyard :: Scratchpad, defect)
DevTools Graveyard
Scratchpad
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 28
People
(Reporter: bbenvie, Assigned: lpy)
Details
(Whiteboard: [good-first-bug] [mentor=bbenvie@mozilla.com] [lang=js])
Attachments
(1 file, 1 obsolete file)
18.56 KB,
patch
|
lpy
:
review+
|
Details | Diff | Splinter Review |
We've stopped adding bug numbers to new test names, and would like to remove the bug numbers from existing test names. Example:
> browser/devtools/scratchpad/test/browser_scratchpad_bug684546_reset_undo.js
renamed to:
> browser/devtools/scratchpad/test/browser_scratchpad_reset_undo.js
A comment should also be added to the file indicating the bug number for reference so it can still be found if desired.
This will also involve updating browser.ini in the test directory to match the updated filenames.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → pylaurent1314
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #831450 -
Flags: review?(bbenvie)
Reporter | ||
Comment 2•12 years ago
|
||
Comment on attachment 831450 [details] [diff] [review]
bug937953.patch
Review of attachment 831450 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good!
Attachment #831450 -
Flags: review?(bbenvie) → review+
Reporter | ||
Comment 3•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 4•12 years ago
|
||
Attachment #831450 -
Attachment is obsolete: true
Attachment #832181 -
Flags: review+
Comment 5•12 years ago
|
||
Flags: in-testsuite+
Keywords: checkin-needed
Whiteboard: [good-first-bug] [mentor=bbenvie@mozilla.com] [lang=js] → [good-first-bug] [mentor=bbenvie@mozilla.com] [lang=js][fixed-in-fx-team]
Comment 6•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [good-first-bug] [mentor=bbenvie@mozilla.com] [lang=js][fixed-in-fx-team] → [good-first-bug] [mentor=bbenvie@mozilla.com] [lang=js]
Target Milestone: --- → Firefox 28
Updated•7 years ago
|
Product: Firefox → DevTools
Updated•6 years ago
|
Product: DevTools → DevTools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•