Closed Bug 902989 Opened 11 years ago Closed 11 years ago

Cover the call_screen.js with units tests

Categories

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

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: etienne, Unassigned)

References

Details

The soon to be in it's own file CallScreen component (see bug 902983) needs unit tests.

It's a nice singleton so it should be pretty straightforward even if there is a bit of dom manipulation.
|telephony_helper_test.js| will probably be a great source of inspiration.
Blocks: 902980
Depends on: 902983
Gabriele, you mentioned some interest in adding unit tests to the dialer, ping me if you want to take this bug, glad to help!
Yes I would like to add some tests related to the CDMA call waiting functionality (bug 882980) which is somewhat orthogonal to GSM call waiting and introduces some fairly subtle changes. I'll try to land the changes first because they've been a pretty thorny issue and then I'd like to come back and add the tests.
Quick update: the call_screen_test.js file now exists. New features are covered.

But this bug is still valid since someone need to do a pass and add all the missing ones.

After discussing this with Rik, we should also switch from binding everything at evaluation to having a (tested) init method. Should make things a log cleaner.
Calling this one fixed :)
At this point we would clearly require any CallScreen change to come with a test, so mission accomplished.

https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/dialer/test/unit/call_screen_test.js
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.