Closed Bug 1170643 Opened 10 years ago Closed 10 years ago

[Music] [Front-end] Add unit tests for ui/views/player_view.js

Categories

(Firefox OS Graveyard :: Gaia::Music, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::Music
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
FxOS-S3 (24Jul)

People

(Reporter: dkuo, Assigned: hub)

References

Details

Attachments

(1 file)

[gaia] hfiguiere:bug1170643-player-view-test > mozilla-b2g:master
46 bytes, text/x-github-pull-request
squib
: review+
Details | Review
Currently the unit tests for ui/views/player_view.js are not complete, so let's add more for it.
Assignee: nobody → hub
Status: NEW → ASSIGNED
Comment on attachment 8630756 [details] [review] [gaia] hfiguiere:bug1170643-player-view-test > mozilla-b2g:master I think all is covered.
Attachment #8630756 - Flags: review?(dkuo)
Flags: in-testsuite- → in-testsuite+
Status: REOPENED → ASSIGNED
Flags: in-testsuite-
Comment on attachment 8630756 [details] [review] [gaia] hfiguiere:bug1170643-player-view-test > mozilla-b2g:master Bouncing review back to Jim. Jim, since you plan on refactoring the player view, this is relevant. I believe some of that testing will need to be change to reflect the refactoring.
Attachment #8630756 - Flags: review?(dkuo) → review?(squibblyflabbetydoo)
Comment on attachment 8630756 [details] [review] [gaia] hfiguiere:bug1170643-player-view-test > mozilla-b2g:master Looks good!
Attachment #8630756 - Flags: review?(squibblyflabbetydoo) → review+
Merged https://github.com/mozilla-b2g/gaia/commit/1c5c3516f2d7c0b6f587c7642a23ae5b9f136da8
Status: ASSIGNED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → FxOS-S3 (24Jul)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: