758626 - account provisioner: cannot subscribe a new email address if there an account provisioner tab already open

Status: RESOLVED FIXED

(Thunderbird :: General, defect)

Description

[Jb Piacentino]

STR:

- Select a suggestion in account provisionner suggestion list -> an account provisionner tab opens
- File/New/Get a new email address... get a suggestion list
- select a suggestion
- suggestion list dlg window does not disappear (beach ball), already opened account provisioner tab is not updated (or no new tab is opened for new suggestion)

Comment 1

[Mike Conley (:mconley) (:⚙️)]

Blake and I talked about this, and we think the ideal behaviour would be that if there is an Account Provisioner tab opened, then going to File > New > Get a New Mail Account should focus that tab.

Comment 2

[Mike Conley (:mconley) (:⚙️)]

Attachment: Patch v1

Assumes patches from the following bugs applied in this order:

bug 757823
bug 758237
bug 758707
bug 758618
bug 758626

Comment 3

[Mike Conley (:mconley) (:⚙️)]

Ok, you should be able to just apply this on top of comm-central properly now, without the other patches (which have now landed).

Comment 4

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 627738 [details] [diff] [review]
Patch v1

Okay, this does seem to do what we agreed on, and it even handles the case where I create an existing mail account, and then hit the button to try to create a new one…  ;)  ui-r=me!

>+++ b/mail/test/mozmill/newmailaccount/test-newmailaccount.js
>@@ -1145,8 +1145,25 @@ function test_search_button_disabled_if_
>+function test_get_new_account_focuses_existing_ap_tab() {
>+  get_to_order_form("[e-mail]");
>+  let apTab = mc.tabmail.getTabInfoForCurrentOrFirstModeInstance(
>+    mc.tabmail.tabModes["accountProvisionerTab"]);
>+
>+  // Switch back to the inbox tab.
>+  mc.tabmail.switchToTab(0);
>+
>+  // Try to re-open the provisioner dialog
>+  open_provisioner_window();
>+
>+  // If we got here, that means that we weren't blocked by a dialog
>+  // being opened, which is good.
>+  assert_selected_tab(apTab);
>+  mc.tabmail.closeTab(apTab);
>+}

I think we're okay without testing all the various ways to get to that state, but if you wanted to add a check for launching the accountWizard, and then clicking the "get a new account" button, I wouldn't complain too loudly either.

Other than that, it seems fine to me.

r=me!

Later,
Blake.

Comment 5

[Mike Conley (:mconley) (:⚙️)]

Attachment: Patch v2 (carrying over r+/ui-r+ from bwinton)

Okie doke, I've augmented the test to check the button from the wizard as well.

Comment 6

[Mike Conley (:mconley) (:⚙️)]

comm-central: https://hg.mozilla.org/comm-central/rev/a69ccbd5ab6e
comm-aurora: https://hg.mozilla.org/releases/comm-aurora/rev/edec0900161f
comm-beta: https://hg.mozilla.org/releases/comm-beta/rev/0aa1e03d1893