B2G SMS: add marionette test script for strict 7bit encoding

RESOLVED FIXED in mozilla20

Status

()

Core
DOM: Device Interfaces
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: vicamo, Assigned: vicamo)

Tracking

unspecified
mozilla20
ARM
Gonk (Firefox OS)
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

5 years ago
The marionette test script was first uploaded in bug 816082 but removed temporarily later[1] due to bug 822207. Here we have some work-around, specifically limit the timeout in this script and increase MARIONETTE_TIMEOUT.

[1]: bug 816082 comment #33.
(Assignee)

Comment 2

5 years ago
still fails, maybe we have to rewrite the test case, save db transactions instead.
(Assignee)

Comment 3

5 years ago
Created attachment 697322 [details] [diff] [review]
marionette test case : v2

Concatenate test strings into one. This saves hundreds SMS db transactions and avoid bug 822207. Try: https://tbpl.mozilla.org/?tree=Try&rev=64003f2f846f.
Attachment #695619 - Attachment is obsolete: true
Attachment #697322 - Flags: review?(htsai)
(Assignee)

Updated

5 years ago
Depends on: 816082
(Assignee)

Comment 4

5 years ago
Created attachment 697749 [details] [diff] [review]
marionette test case : v2

Update the patch used in the try run.
Attachment #697322 - Attachment is obsolete: true
Attachment #697322 - Flags: review?(htsai)
(Assignee)

Updated

5 years ago
Attachment #697749 - Flags: review?(htsai)
Comment on attachment 697749 [details] [diff] [review]
marionette test case : v2

Review of attachment 697749 [details] [diff] [review]:
-----------------------------------------------------------------

Nice!
Attachment #697749 - Flags: review?(htsai) → review+

Comment 7

5 years ago
https://hg.mozilla.org/mozilla-central/rev/e905c6778d61
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
You need to log in before you can comment on or make changes to this bug.