Closed Bug 965802 Opened 10 years ago Closed 10 years ago

See if lockscreen camera tests can be enabled on Travis

Categories

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

Other
Gonk (Firefox OS)
defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: zcampbell, Assigned: viorela)

Details

Attachments

(1 file)

These tests don't actually take pictures, they just check the frames open.

So let's see if we can enable the tests.
Assignee: nobody → viorela.ioia
I ran the lockscreen camera tests several times on Travis, they passed: https://pastebin.mozilla.org/4173388,
https://pastebin.mozilla.org/4173419
I will enable them on Travis and TBPL also. Let's see how they behave.
Attachment #8369455 - Flags: review?(zcampbell)
Attachment #8369455 - Flags: review?(andrei.hutusoru)
Attachment #8369455 - Flags: review?(zcampbell)
Attachment #8369455 - Flags: review?(andrei.hutusoru)
Attachment #8369455 - Flags: review-
Attachment #8369455 - Flags: review- → review?(zcampbell)
Comment on attachment 8369455 [details] [review]
Github PR: https://github.com/mozilla-b2g/gaia/pull/15915

r+ but I'll wait for a green Travis before I merge.
Attachment #8369455 - Flags: review?(zcampbell) → review+
Both of the tests failed on Travis.
As this tests failed on travis, I think we can close this bug.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
(In reply to Viorela Ioia [:viorela] from comment #5)
> As this tests failed on travis, I think we can close this bug.

This doesn't make any sense to me. If the tests failed, why is this resolved/fixed?
Flags: needinfo?(viorela.ioia)
Sorry, this should be marked as wontfix, since the test failed on Travis.
Flags: needinfo?(viorela.ioia)
Resolution: FIXED → WONTFIX
But in comment 1 you mentioned that they passed for you, so shouldn't we expect them to pass?
The tests passed when I ran them locally, using the Travis script.
Exactly, so why are they failing on Travis? Shouldn't we look into that? :)
I just thought that locally Gecko could pick up on a camera in our laptops whereas on Travis it's a VM.
Camera can work in a VM too, it's probably more likely to be related to a lack of camera on the host. This bug just seemed to lack any proper resolution. I wouldn't really expect any tests using the camera app to pass on a target with no camera attached.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: