[Stingray][Home] Add Unit tests for home.js

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: rexboy, Assigned: rexboy)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Add unit tests for home.js.
Cover possible cases as possible. To cover the whole file, many cases will be related with the original file. So we can see this bug as "round 1" and open follow-up bugs if required.
Blocks: 1141478
No longer depends on: 1141478
Summary: [Meta][Stingray][Home] Add Unit tests for home.js → [Stingray][Home] Add Unit tests for home.js
Created attachment 8634036 [details] [review]
[gaia] rexboy7:home-unit-test-3 > mozilla-b2g:master
Created attachment 8634037 [details] [review]
Patch

Hi dwi2,
I tried to put on a unit-test patch.
Sorry for a big one :-/ you can put it off if you are not available.
Attachment #8634037 - Flags: review?(tzhuang)
Comment on attachment 8634037 [details] [review]
Patch

Generally it looks good to me. Please address comments on PR before landing.

Thanks
Attachment #8634037 - Flags: review?(tzhuang) → review+
Thanks for reviewing the patch. I just addressed your comments and leave some TODO there to confirm we won't forget to add remaining cases.

landed on master:
https://github.com/mozilla-b2g/gaia/commit/4cddebb56792a15619fdc34db790c6d014cb8e3c
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Blocks: 1185965
You need to log in before you can comment on or make changes to this bug.