Closed
Bug 788928
Opened 12 years ago
Closed 7 years ago
B2G SMS: add test cases for nsIDOMSmsManager.ondelivered and nsIDOMSmsRequest.onerror events
Categories
(Firefox OS Graveyard :: RIL, defect)
Tracking
(blocking-b2g:-)
RESOLVED
WONTFIX
blocking-b2g | - |
People
(Reporter: vicamo, Assigned: vicamo)
References
Details
Attachments
(3 files, 1 obsolete file)
(deleted),
text/x-github-pull-request
|
Details | |
(deleted),
text/x-github-pull-request
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review |
See review comments in bug 787370. The Android emulator doesn't have SMS-STATUS-REPORT unsolicited event implemented, but we might still need it to verify SMS ondelivered event.
Assignee | ||
Comment 1•12 years ago
|
||
Marshall mentioned a technique to insert arbitrary SMS PDU into emulator in bug 787370 comment #18. Code examples are here:
http://dxr.mozilla.org/mozilla-central/dom/telephony/test/marionette/pdu_builder.js.html
http://dxr.mozilla.org/mozilla-central/dom/telephony/test/marionette/test_voicemail_statuschanged.js.html#l10
Assignee | ||
Updated•12 years ago
|
Summary: B2G SMS: add test cases for nsIDOMSmsManager.ondelivered event → B2G SMS: add test cases for nsIDOMSmsManager.ondelivered and nsIDOMSmsRequest.onerror events
Assignee | ||
Comment 2•12 years ago
|
||
(In reply to Vicamo Yang [:vicamo][:vyang] from comment #1)
> Marshall mentioned a technique to insert arbitrary SMS PDU into emulator in
> bug 787370 comment #18. Code examples are here:
> http://dxr.mozilla.org/mozilla-central/dom/telephony/test/marionette/
> pdu_builder.js.html
> http://dxr.mozilla.org/mozilla-central/dom/telephony/test/marionette/
> test_voicemail_statuschanged.js.html#l10
Actually, `sms pdu` command alone can't accomplish the purpose here. SMS-STATUS-REPORT contains a TP-MR field that is used to pair sent message with the report itself. However, the value is currently an internal state of the emulator and there is no emulator command to fetch it.
To solve this issue, you'll either 1) make the emulator send a pre-configured[1] status-report status(TP-ST) to every outgoing sms, or 2) ask emulator the latest used TP-MR value and construct SMS-STATUS-REPORT PDUs with it. In either way, you'll still have to modify emulator code. The second method provides much more flexibility than the first one.
[1]: or, we may always return "destination unreachable", and "success" for inter emulator instances communication.
Depends on: 742790
Assignee | ||
Comment 3•12 years ago
|
||
I think I'll really need this feature to help verifying delivery events :(
Severity: minor → normal
Assignee | ||
Updated•12 years ago
|
Assignee | ||
Comment 4•12 years ago
|
||
Assignee | ||
Comment 5•12 years ago
|
||
Assignee | ||
Comment 6•12 years ago
|
||
Attachment #695819 -
Attachment is obsolete: true
Assignee | ||
Comment 7•12 years ago
|
||
test cases for request.onerror
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → vyang
Updated•11 years ago
|
Attachment mime type: text/plain text/plain → text/x-github-pull-request text/x-github-pull-request
Assignee | ||
Updated•11 years ago
|
Component: DOM: Device Interfaces → RIL
Product: Core → Firefox OS
Updated•10 years ago
|
blocking-b2g: backlog → -
Comment 9•7 years ago
|
||
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•