Closed
Bug 1197373
Opened 9 years ago
Closed 9 years ago
Imagecompare: add wait after scroll in calendar until the scroll bar disappears
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: njpark, Assigned: njpark)
Details
Attachments
(1 file)
Scroll bar shows right after the scroll action, and it disappears in 1-2 seconds. need to wait so the screen capture does not capture the scrollbar.
Comment 1•9 years ago
|
||
Assignee | ||
Comment 2•9 years ago
|
||
Comment on attachment 8651341 [details] [review] [gaia] npark-mozilla:1197373 > mozilla-b2g:master So the calendar app shows the scroll bar during scroll action, and it disappears after the motion has stopped. It takes about half a second for it to disappear, so I'm adding a manual wait of 1 second. (3 second for the first one, since that transition seems to take longer)
Attachment #8651341 -
Flags: review?(martijn.martijn)
Comment 3•9 years ago
|
||
Comment on attachment 8651341 [details] [review] [gaia] npark-mozilla:1197373 > mozilla-b2g:master Why wasn't this a problem before with this test? Or it was actually? I thought the idea was to prevent time.sleep calls as much as possible (something that should be/is in our style guide?), but I guess there is no way of detecting scrollbar disappearance.
Attachment #8651341 -
Flags: review?(martijn.martijn) → review+
Assignee | ||
Comment 4•9 years ago
|
||
This was failing intermittently, but recently this became noticeable, so I decided to fix it for good. I was looking for the scrollbar element, but I don't think we can see it in the source.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → npark
Assignee | ||
Comment 5•9 years ago
|
||
merged: https://github.com/mozilla-b2g/gaia/commit/b441bde54293bea5254dc340845effe951fa3906
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•