1086662 - Add a test that ensures test_camera_capture_video.py saves a video file of a minimum size

Status: RESOLVED FIXED

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

Description

[Martijn Wargers (dead)]

This is moztrap test https://moztrap.mozilla.org/manage/case/2478/

No idea how to check if the sound of the video is working, though.

Comment 1

[Martijn Wargers (dead)]

From the Moztrap test:
Prerequisite: A video has been recorded with sound, using the onboard camera.

    Open the video app.

    Select recently recorded video.

    Video should play back. There should be visuals with no gap. There should be continuous sound in sync with the video, also with no gaps.

Comment 2

[Martijn Wargers (dead)]

Attachment: test_camera_capture_video.py.diff

This is rather lame, this is just adding test_play_webm_video.py on top of test_camera_capture_video.py.

I'm not sure how useful that is. This tests at least that the recorded video is playable. 
But I don't know if testing if there are problems with the sound or video smoothness is possible.

Comment 3

[Zac C (:zac)]

Martijn, I tend to agree.. The two test cases taken separately will be more stable and reliable.

Something you may want to do is validate more strictly that the video file is recorded onto the sdcard, that its file size is >0 for example.

I don't think that testing the sound or video is easily done, however perhaps as an addition to test_play_*_video you could check that the correct audio channel is being opened (but that'd be a separate bug).

Comment 4

[Martijn Wargers (dead)]

Attachment: capture.diff

So something like this?

If you agree with this, I would need to also change the other callers of getFiles.

Comment 5

[Martijn Wargers (dead)]

(In reply to Zac C (:zac) from comment #3)
> I don't think that testing the sound or video is easily done, however
> perhaps as an addition to test_play_*_video you could check that the correct
> audio channel is being opened (but that'd be a separate bug).

I filed bug 1094144 for this.

Comment 6

[Zac C (:zac)]

Comment on attachment 8517355 [details] [diff] [review]
capture.diff

Yes I think something like that but we'd want the video_files property to be a Python list so we can read it using Python syntax.

As this will break a lot of other tests it will need to be of high importance to be worth the risk.

Comment 7

[Martijn Wargers (dead)]

Looking at what might break, I see only sdcard_files that is prone to breakage.
tests/functional/contacts/test_export_contacts_to_sdcard.py
tests/functional/contacts/test_import_contacts_from_sdcard.py 
The rest, music_files, picture_files, video_files, media_files, seem to work (or isn't used at all).

Comment 8

[Martijn Wargers (dead)]

Attachment: https://github.com/mozilla-b2g/gaia/pull/26114

Comment 9

[Zac C (:zac)]

Comment on attachment 8522213 [details] [review]
https://github.com/mozilla-b2g/gaia/pull/26114

r-, if we make a small change to the atom then we can do everything in Python within the test instead of trying to convert data types.

Comment 10

[Martijn Wargers (dead)]

Attachment: capture3

Like this?

Comment 11

[Zac C (:zac)]

Comment on attachment 8526142 [details] [review]
capture3

r+, Yep that's great, very clean!

Comment 12

[Zac C (:zac)]

Merged:
https://github.com/mozilla-b2g/gaia/commit/d06a4419beb4c52f988a413498af9d7538d560a6