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

RESOLVED FIXED

Status

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: martijn.martijn, Assigned: martijn.martijn)

Tracking

unspecified
x86
macOS
Bug Flags:
in-qa-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

()

Attachments

(1 attachment, 3 obsolete attachments)

Assignee

Description

5 years ago
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.
Assignee

Updated

5 years ago
Component: Gaia::UI Tests → Gaia::Video
Assignee

Comment 1

5 years ago
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.
Assignee

Comment 2

5 years ago
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.
Attachment #8514694 - Flags: feedback?(zcampbell)

Comment 3

5 years ago
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).

Updated

5 years ago
Attachment #8514694 - Flags: feedback?(zcampbell)
Assignee

Updated

5 years ago
QA Whiteboard: fxosqa-auto-s3, fxosqa-auto-points=8,
Flags: in-qa-testsuite?(martijn.martijn)
Assignee

Comment 4

5 years ago
Posted patch capture.diff (obsolete) — Splinter Review
So something like this?

If you agree with this, I would need to also change the other callers of getFiles.
Attachment #8514694 - Attachment is obsolete: true
Attachment #8517355 - Flags: feedback?(zcampbell)
Assignee

Updated

5 years ago
Summary: Write an automated test for "Play the video you recorded. Check for video and sound" manual smoketest → Add a test that ensures test_camera_capture_video.py saves a video file of a minimum size
Assignee

Updated

5 years ago
Component: Gaia::Video → Gaia::UI Tests
Assignee

Comment 5

5 years ago
(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

5 years ago
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.
Attachment #8517355 - Flags: feedback?(zcampbell)
Assignee

Comment 7

5 years ago
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).
Assignee

Updated

5 years ago
QA Whiteboard: fxosqa-auto-s3, fxosqa-auto-points=8, → fxosqa-auto-s4, fxosqa-auto-points=8, fxosqa-auto-from-s3
Assignee

Comment 8

5 years ago
Attachment #8517355 - Attachment is obsolete: true
Assignee

Updated

5 years ago
Attachment #8522213 - Flags: review?(zcampbell)

Comment 9

5 years ago
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.
Attachment #8522213 - Flags: review?(zcampbell) → review-
Assignee

Comment 10

5 years ago
Posted file capture3
Like this?
Attachment #8522213 - Attachment is obsolete: true
Attachment #8526142 - Flags: review?(zcampbell)

Comment 11

5 years ago
Comment on attachment 8526142 [details] [review]
capture3

r+, Yep that's great, very clean!
Attachment #8526142 - Flags: review?(zcampbell) → review+
Assignee

Updated

5 years ago
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
QA Whiteboard: fxosqa-auto-s4, fxosqa-auto-points=8, fxosqa-auto-from-s3 → fxosqa-auto-s4, fxosqa-auto-points=8, fxosqa-auto-from-s3, fxosqa-auto-from-s2
Assignee

Updated

5 years ago
Flags: in-qa-testsuite?(martijn.martijn) → in-qa-testsuite+
You need to log in before you can comment on or make changes to this bug.