Closed
Bug 732414
Opened 13 years ago
Closed 13 years ago
WebSMS: Expose nsIDOMMozSmsFilter to xpcshell
Categories
(Core :: DOM: Device Interfaces, defect)
Core
DOM: Device Interfaces
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: ferjm, Assigned: ferjm)
References
Details
Attachments
(1 file)
(deleted),
patch
|
mounir
:
review+
|
Details | Diff | Splinter Review |
Currently, there is no way to create an SmsFilter in xpcshell, since XPCOM isn´t told about the class, only about the interface.
Assignee | ||
Updated•13 years ago
|
Updated•13 years ago
|
Component: General → DOM: Device Interfaces
QA Contact: general → device-interfaces
Comment 1•13 years ago
|
||
Yeah, that's a good one. Because it's just a DOM object, the C++ implementation doesn't have a CID/ContractID with which we can instantiate it. And implementing our own nsIDOMMozSmsFilter in JS for testing purposes doesn't work either because the interface is marked [builtinclass] -- for a good reason, because the DOM IPC implementation depends on C++ implementation internals.
What do we need to test in xpcshell here that we can't test in the emulator?
Comment 3•13 years ago
|
||
(In reply to Chris Jones [:cjones] [:warhammer] from comment #2) > What do we need to test in xpcshell here that we can't test in the emulator? Functionally, nothing. xpcshell tests are just much quicker to run and don't depend on any UI. So if there's a low hanging fruit to *unit* test the SmsDatabaseService we're writing, I'll take it.
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → ferjmoreno
Comment 5•13 years ago
|
||
Comment on attachment 602923 [details] [diff] [review] WIP v1 Review of attachment 602923 [details] [diff] [review]: ----------------------------------------------------------------- Seems sad to do this *only* for tests... Can't this test be made in a mochitest? r=me because the patch is correct but I would like to discuss the other solutions before landing this.
Attachment #602923 -
Flags: review?(mounir) → review+
Comment 6•13 years ago
|
||
(In reply to Mounir Lamouri (:volkmar) (:mounir) from comment #5) > Seems sad to do this *only* for tests... > Can't this test be made in a mochitest? Hmm, right, it can. That might indeed be better.
Comment 7•13 years ago
|
||
Yup, chrome mochitest works. See bug 733265.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → INVALID
Comment 8•13 years ago
|
||
Actually, not an invalid bug, just one we didn't choose to fix this way.
Resolution: INVALID → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•