Closed
Bug 887777
Opened 11 years ago
Closed 11 years ago
Add unit tests for the suggestions
Categories
(Firefox OS Graveyard :: Gaia::Keyboard, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: janjongboom, Assigned: janjongboom)
Details
Attachments
(1 file)
46 bytes,
patch
|
Details | Diff | Splinter Review |
We don't have any test coverage for the suggestions part of latin.js. Let's do that.
Assignee | ||
Comment 1•11 years ago
|
||
This adds some structure to the keyboard suggestions where we can add some more tests in the future. It mocks the worker and has a simple interface to talk to the IM. The extra lines in latin.js are there because we had an undefined global hanging around that we crashed upon in tests.
Assignee: nobody → janjongboom
Attachment #768282 -
Flags: review?(rlu)
Assignee | ||
Comment 2•11 years ago
|
||
Also added a test file for the workers, didn't write any useful tests, but please use it as a framework for building upon.
Assignee | ||
Updated•11 years ago
|
Attachment #768282 -
Flags: review?(rlu) → review?(dflanagan)
Comment 3•11 years ago
|
||
Jan, Thanks for these test cases. I won't have time to review before my vacation, however. I'll be back on 7/19. Feel free to reassign the review if you'd like to get this landed before then.
Assignee | ||
Updated•11 years ago
|
Attachment #768282 -
Flags: review?(dflanagan) → review?(rlu)
Assignee | ||
Comment 4•11 years ago
|
||
Superseeded by bug 803189.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
Updated•11 years ago
|
Attachment #768282 -
Flags: review?(rlu)
You need to log in
before you can comment on or make changes to this bug.
Description
•