Closed
Bug 949062
Opened 12 years ago
Closed 12 years ago
Update test_music.py file to be more explicit about failing to play the mp3 file
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: zcampbell, Assigned: AndreiH)
References
Details
Attachments
(1 file)
If this test fails because the mp3 file (codec) was not played properly we want to be clear that it is because of the mp3 file.
Some tasks for this:
* Update the filename to both describe the test case and include 'mp3'
* Update the assertion messages to be clear when it is the mp3 file fails and when the UI fails
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → andrei.hutusoru
| Assignee | ||
Comment 1•12 years ago
|
||
I will start updating this, as I already made some changes working on bug 949084
| Assignee | ||
Comment 2•12 years ago
|
||
Attachment #8350066 -
Flags: review?(zcampbell)
Attachment #8350066 -
Flags: review?(viorela.ioia)
Attachment #8350066 -
Flags: review?(florin.strugariu)
Updated•12 years ago
|
Attachment #8350066 -
Flags: review?(viorela.ioia) → review-
| Assignee | ||
Updated•12 years ago
|
Attachment #8350066 -
Flags: review- → review?(viorela.ioia)
| Reporter | ||
Comment 3•12 years ago
|
||
Comment on attachment 8350066 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/14853
r- ,some tweaks to assertions messages to be done.
Attachment #8350066 -
Flags: review?(zcampbell) → review-
| Assignee | ||
Updated•12 years ago
|
Attachment #8350066 -
Flags: review- → review?(zcampbell)
Updated•12 years ago
|
Attachment #8350066 -
Flags: review?(florin.strugariu) → review+
Updated•12 years ago
|
Attachment #8350066 -
Flags: review?(viorela.ioia) → review+
| Reporter | ||
Comment 4•12 years ago
|
||
Comment on attachment 8350066 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/14853
r+
Attachment #8350066 -
Flags: review?(zcampbell) → review+
| Reporter | ||
Comment 5•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•