Closed Bug 1184820 Opened 9 years ago Closed 9 years ago

update imagecompare tests to accomodate changes in Bug 1180819

Categories

(Firefox OS Graveyard :: Gaia::UI Tests, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: njpark, Assigned: njpark)

Details

Attachments

(1 file)

Existing imagecompare tests need to use new scroll/pinch method, as well as deal with the reduced time.sleep() duration before taking screenshot.
Attachment #8635091 - Flags: review?(martijn.martijn)
Assignee: nobody → npark
Comment on attachment 8635091 [details] [review]
[gaia] npark-mozilla:update > mozilla-b2g:master

I ran one test, test_orientation_zoom.py, and that works. I got a failure, but that appears to be, because of the log, which shows slight differences between the 2 runs (perhaps the fuzz factor needs to be increased?).
Attachment #8635091 - Flags: review?(martijn.martijn) → review+
(In reply to Martijn Wargers [:mwargers] (QA) from comment #2)
> Comment on attachment 8635091 [details] [review]
> [gaia] npark-mozilla:update > mozilla-b2g:master
> 
> I ran one test, test_orientation_zoom.py, and that works. I got a failure,
> but that appears to be, because of the log, which shows slight differences
> between the 2 runs (perhaps the fuzz factor needs to be increased?).

That's actually a good point, I've been meaning to increase the default fuzz factor to 20~25% because that seems to be more ideal tradeoff.
Merged:

https://github.com/mozilla-b2g/gaia/commit/a47e0c58dd1340a17be7cc96ccd90fcad15a922b
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.

Attachment

General

Created:
Updated:
Size: