Closed
Bug 1162422
Opened 9 years ago
Closed 9 years ago
Perma-failing marionette test, TEST-UNEXPECTED-FAIL | test_fakevolume.js | state - got -1, expected 4
Categories
(Firefox OS Graveyard :: General, defect)
Tracking
(firefox41 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: edgar, Assigned: kershaw)
References
Details
Attachments
(1 file, 1 obsolete file)
3.43 KB,
patch
|
kershaw
:
review+
|
Details | Diff | Splinter Review |
https://treeherder.mozilla.org/logviewer.html#?job_id=1866668&repo=b2g-inbound 17:56:35 ERROR - TEST-UNEXPECTED-FAIL | test_fakevolume.js | state - got -1, expected 4 17:56:35 ERROR - TEST-UNEXPECTED-FAIL | test_fakevolume.js | mount generation should be incremented - false was false, expected true 17:56:37 ERROR - TEST-UNEXPECTED-FAIL | test_fakevolume.js | AssertionError: 2 tests failed It seems a regression of bug 1126694.
Reporter | ||
Comment 1•9 years ago
|
||
Hi Kershaw, could you help to take a look? Thank you. https://treeherder.mozilla.org/#/jobs?repo=b2g-inbound&filter-searchStr=mnw&exclusion_profile=false
Flags: needinfo?(kechang)
Assignee | ||
Comment 2•9 years ago
|
||
(In reply to Edgar Chen [:edgar][:echen] from comment #1) > Hi Kershaw, could you help to take a look? Thank you. > > https://treeherder.mozilla.org/#/jobs?repo=b2g-inbound&filter- > searchStr=mnw&exclusion_profile=false Hi Dave, There is a test case test_fakevolume.js expects that the initial state of a fake volume should be STATE_INIT, not STATE_MOUNTED. It seems that we should revert CreateFakeVolume back in order to let this test pass. What do you think? Thanks.
Assignee: nobody → kechang
Flags: needinfo?(kechang) → needinfo?(dhylands)
Comment 3•9 years ago
|
||
(In reply to Kershaw Chang [:kershaw] from comment #2) > (In reply to Edgar Chen [:edgar][:echen] from comment #1) > > Hi Kershaw, could you help to take a look? Thank you. > > > > https://treeherder.mozilla.org/#/jobs?repo=b2g-inbound&filter- > > searchStr=mnw&exclusion_profile=false > > Hi Dave, > There is a test case test_fakevolume.js expects that the initial state of a > fake volume should be STATE_INIT, not STATE_MOUNTED. It seems that we should > revert CreateFakeVolume back in order to let this test pass. > What do you think? > Thanks. I think we just fix the test. Having the fake volumes start in STATE_INIT just feels wrong to me. We currently don't use fake volumes in any shipping devices, so this shouldn't cause any problems.
Flags: needinfo?(dhylands)
Assignee | ||
Comment 4•9 years ago
|
||
Hi Dave, Could you please take a look at this patch? I've also modified CreateFakeVolume to call SetState in order to increase the value of sMountGeneration. Thanks.
Attachment #8604507 -
Flags: review?(dhylands)
Comment 5•9 years ago
|
||
Comment on attachment 8604507 [details] [diff] [review] Fix test cases Review of attachment 8604507 [details] [diff] [review]: ----------------------------------------------------------------- Looks good. Thanks
Attachment #8604507 -
Flags: review?(dhylands) → review+
Assignee | ||
Comment 6•9 years ago
|
||
Carry reviewer's name. test_fakevolume.js pass now. https://treeherder.mozilla.org/#/jobs?repo=try&revision=675e374f3229&exclusion_profile=false
Attachment #8604507 -
Attachment is obsolete: true
Attachment #8605101 -
Flags: review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Reporter | ||
Comment 7•9 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/bd5d5dd4be0c
Keywords: checkin-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•