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)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:-)

RESOLVED WONTFIX
blocking-b2g -

People

(Reporter: vicamo, Assigned: vicamo)

References

Details

Attachments

(3 files, 1 obsolete file)

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.
Summary: B2G SMS: add test cases for nsIDOMSmsManager.ondelivered event → B2G SMS: add test cases for nsIDOMSmsManager.ondelivered and nsIDOMSmsRequest.onerror events
(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
I think I'll really need this feature to help verifying delivery events :(
Severity: minor → normal
Blocks: 824465
No longer blocks: 824465
Depends on: 824465
Attached file Github Pull Request for hardware/ril (deleted) —
Attached file Github WIP branch for external/qemu (obsolete) (deleted) —
Attached file Github Pull Request for external/qemu (deleted) —
Attachment #695819 - Attachment is obsolete: true
Depends on: 819937
Attached patch marionette test case : WIP (deleted) — Splinter Review
test cases for request.onerror
Assignee: nobody → vyang
Depends on: 863130
Depends on: 911026
Depends on: 911690
Depends on: 913313
Attachment mime type: text/plain text/plain → text/x-github-pull-request text/x-github-pull-request
Component: DOM: Device Interfaces → RIL
Product: Core → Firefox OS
Please this into backlog.
blocking-b2g: --- → backlog
blocking-b2g: backlog → -
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.

Attachment

General

Created:
Updated:
Size: