Closed
Bug 849183
Opened 12 years ago
Closed 11 years ago
[B2G] Marionette tap in Mail app is not working
Categories
(Remote Protocol :: Marionette, defect)
Tracking
(Not tracked)
RESOLVED
WORKSFORME
People
(Reporter: zcampbell, Assigned: automatedtester)
References
Details
(Keywords: pi-marionette-goal, pi-marionette-userinput, Whiteboard: [touch])
Attachments
(1 file)
(deleted),
text/plain
|
Details |
This is high priority due to the importance of automating the Email app tests.
When Marionette clicks 'Delete' to confirm the deletion of the Email account the confirmation box clears but the account is not deleted.
Tapping manually is successful.
Reporter | ||
Updated•12 years ago
|
Summary: [B2G] Marionette tap in Mail app is not causing → [B2G] Marionette tap in Mail app is not working
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → dburns
Assignee | ||
Comment 1•12 years ago
|
||
11:29 zac: I forgot to mention you need to setup an account manually
Assignee | ||
Comment 2•12 years ago
|
||
currently checking in bug 848727 if could be the cause
Assignee | ||
Comment 3•12 years ago
|
||
no that wasnt the issue but after a closer look I think its related to bug 779284
Assignee | ||
Comment 4•12 years ago
|
||
https://github.com/mozilla-b2g/gaia/blob/master/apps/email/index.html#L1070 shows it as a confirm.
Assignee | ||
Comment 5•12 years ago
|
||
so this looks more like a dupe.
I changed the dupe ID, as in 779284, locally to account-delete-ok-real and when query elements in the confirm, marionette things they are of size 0 and live in 0x, 0y so marionette doesnt understand them. Below is the html I was I trying to work with.
<form data-type="confirm" class="tng-account-delete-confirm" role="dialog">\n <section>\n <h1 data-l10n-id="confirm-dialog-title">Confirmation</h1>\n <p>Are you sure you want to delete your tynecastle@gmail.com mail account? You will have to re-download your mail if you re-create the account.</p>\n </section>\n <menu>\n <button data-l10n-id="message-multiedit-cancel" id="account-delete-cancel-real">Cancel</button>\n <button data-l10n-id="settings-account-delete-confirm" class="danger" id="account-delete-ok-real">Delete</button>\n </menu>\n </form></body></html>
Assignee | ||
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Comment 7•12 years ago
|
||
I no long think bug 779284 is related
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---
Assignee | ||
Comment 8•12 years ago
|
||
Updated•11 years ago
|
Comment 9•11 years ago
|
||
From https://bugzilla.mozilla.org/show_bug.cgi?id=779284#c14
Are you setting explicitOriginalTarget? We depend on that in ConfirmDialog in mail-common.js
Comment 10•11 years ago
|
||
This is an older bug, using self.marionette.tap, and I can't reproduce this test flow now. Is this still relevant, or can I mark this as resolved?
Assignee | ||
Updated•11 years ago
|
Whiteboard: [touch]
Assignee | ||
Updated•11 years ago
|
Keywords: ateam-marionette-goal,
ateam-marionette-userinput
Assignee | ||
Comment 11•11 years ago
|
||
Going on comment 10, is this still relevant?
Flags: needinfo?(zcampbell)
Reporter | ||
Comment 12•11 years ago
|
||
Thankyou zac of the 8th of March 2013 for creating the test case, little did you realise it would save you so much time 13 months later.
This works fine now.
Status: REOPENED → RESOLVED
Closed: 12 years ago → 11 years ago
Flags: needinfo?(zcampbell)
Resolution: --- → WONTFIX
Updated•11 years ago
|
Resolution: WONTFIX → WORKSFORME
Updated•2 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•