Closed
Bug 594687
Opened 14 years ago
Closed 14 years ago
[SeaMonkey 2.1, mochitest-browser-chrome] TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/suite/common/tests/browser/browser_346337.js and browser_466937.js
Categories
(SeaMonkey :: Testing Infrastructure, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
seamonkey2.1b1
People
(Reporter: bugzilla, Assigned: bugzilla)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
3.59 KB,
patch
|
bugzilla
:
review+
|
Details | Diff | Splinter Review |
TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/suite/common/tests/browser/browser_346337.js | The value for "//input[@type='file'][1]" was correctly restored TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/suite/common/tests/browser/browser_346337.js | The value for "//input[@type='file'][2]" was correctly restored Seems only to occur on Windows: http://brasstacks.mozilla.com/topfails/test/SeaMonkey?name=chrome://mochikit/content/browser/suite/common/tests/browser/browser_346337.js
Assignee | ||
Comment 1•14 years ago
|
||
We're missing the test changes from http://hg.mozilla.org/mozilla-central/rev/8d846fde08cb (bug 583863)
Depends on: 583863
Assignee | ||
Comment 2•14 years ago
|
||
This are the test changes from bug 583863. The test passes for Linux with and without the patch, so I couldn't test it myself.
Assignee | ||
Comment 3•14 years ago
|
||
TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/suite/common/tests/browser/browser_466937.js | normal case: file path was correctly preserved - Got , expected /home/user/regular.file The patch should also fix that test fail.
Summary: [SeaMonkey 2.1, mochitest-browser-chrome] TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/suite/common/tests/browser/browser_346337.js → [SeaMonkey 2.1, mochitest-browser-chrome] TEST-UNEXPECTED-FAIL | chrome://mochikit/content/browser/suite/common/tests/browser/browser_346337.js and browser_466937.js
Comment 4•14 years ago
|
||
Comment on attachment 473468 [details] [diff] [review] update tests for refactored input implementation Probably the paths previously used weren't valid on Windows, but it didn't matter because nobody validated them before.
Attachment #473468 -
Flags: review?(neil) → review+
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Comment 5•14 years ago
|
||
Comment on attachment 473468 [details] [diff] [review] update tests for refactored input implementation On second thoughts, r=me if you remove all the calls to createUnique.
Assignee | ||
Comment 6•14 years ago
|
||
Ok, updating the patch (removing checkin-needed until I have an updated patch)
Keywords: checkin-needed
Assignee | ||
Comment 7•14 years ago
|
||
(In reply to comment #5) > Comment on attachment 473468 [details] [diff] [review] > update tests for refactored input implementation > > On second thoughts, r=me if you remove all the calls to createUnique. Taking over r+
Attachment #473468 -
Attachment is obsolete: true
Attachment #473485 -
Flags: review+
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Comment 8•14 years ago
|
||
Comment on attachment 473485 [details] [diff] [review] updated tests without the calls to createUnique Could we use ProfD instead of TmpD, fwiw?
Assignee | ||
Comment 9•14 years ago
|
||
Does it make any difference?
Comment 10•14 years ago
|
||
For future: http://blog.bonardo.net/2010/06/22/so-youre-about-to-use-checkin-needed Which you do a decent job of, only comment is (especially if you are already doing a final upload) is specify bug # and specify who reviewed. That said, pushed as: http://hg.mozilla.org/comm-central/rev/3747f1d60f0a
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•14 years ago
|
Keywords: checkin-needed
Comment 11•14 years ago
|
||
(In reply to comment #9) > Does it make any difference? It helps to keep TmpD clean, and to put all test files in the same place. ***** V.Fixed, per http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1284108915.1284114436.22079.gz WINNT 5.2 comm-central-trunk debug test mochitest-other on 2010/09/10 01:55:15
Status: RESOLVED → VERIFIED
Target Milestone: --- → seamonkey2.1b1
Updated•14 years ago
|
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•