671059 - Add helper method to handle modal install dialog to Addons.js

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[u279076]

All of the discovery pane tests for the Add-ons Manager are reusing the same code over and over again:

>+/**
>+ * Handle the modal dialog to install an addon
>+ */
>+function handleInstallAddonDialog(controller) {
>+  // Wait for the install button is enabled before clicking on it
>+  var installButton = new elementslib.Lookup(controller.window.document, 
>+                                             '/id("xpinstallConfirm")' +
>+                                             '/anon({"anonid":"buttons"})' + 
>+                                             '/{"dlgtype":"accept"}');
>+  controller.waitFor(function(){
>+    return !installButton.getNode().disabled; 
>+  }, "Install button is enabled: got '" + !installButton.getNode().disabled 
>+  + "', expected 'true'");
>+
>+  controller.click(installButton);
>+}

It would be useful if we could get this in the Addons shared module.

Comment 1

[u279076]

Attachment: Patch v1.0

Initial patch -- basically moves the handleAddonInstallDialog() function, which is in use in many discovery pane tests already, into the Addons API. I've tested this patch with some of the Discovery Pane tests and it works great.

Vlad, if Henrik r- this patch, feel free to take over and make the revisions as necessary. Thanks.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 545523 [details] [diff] [review]
Patch v1.0

>+// XPath for modal dialog install button
>+const MODAL_INSTALL_BUTTON = '/id("xpinstallConfirm")/anon({"anonid":"buttons"})/{"dlgtype":"accept"}';

We don't use lookups anymore in this module. This should be moved to a selector or nodeCollector calls.

>+  /**
>+   * Handles the modal dialog to install an add-on
>+   *
>+   * @param {MozMillController} MozMill Controller
>+   */
>+  handleInstallAddonDialog : function AddonsManager_handleInstallAddonDialog(aController) {

This method cannot be part of the add-ons manager. It has to be global in this module.

>+    // Wait for the install button to be enabled
>+    mozmill.utils.waitFor(function() {

nit: please add a space after function.

Comment 3

[Maniac Vlad Florin (:vladmaniac)]

Attachment: patch v1.1

Added patch with requested changes

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 545695 [details] [diff] [review]
patch v1.1

>+  var nodeCollector = new domUtils.nodeCollector(aController.window.document.documentElement);
>+
>+  nodeCollector.queryNodes("#xpinstallConfirm");
>+  nodeCollector.root = nodeCollector.nodes[0];
>+
>+  var installButton = nodeCollector.queryAnonymousNodes("dlgtype", "accept").elements[0];

Can you please separate the last line into two? Means call nodeCollector.elements[0] in the next line. Also remove the empty line after the nodeCollector declaration. Everything is one single block here.

> // Export of functions
> exports.addToWhiteList = addToWhiteList;
> exports.getInstalledAddons = getInstalledAddons;
> exports.removeFromWhiteList = removeFromWhiteList;
> exports.resetAmoPreviewUrls = resetAmoPreviewUrls;
> exports.useAmoPreviewUrls = useAmoPreviewUrls;
>+exports.handleInstallAddonDialog = handleInstallAddonDialog;

Please keep the alphabetical order of the export statements.

Comment 5

[Maniac Vlad Florin (:vladmaniac)]

Attachment: patch v1.2

Patch with requested changes

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/f2584d253444 (default)
http://hg.mozilla.org/qa/mozmill-tests/rev/a3be1bfee355 (aurora)
http://hg.mozilla.org/qa/mozmill-tests/rev/92f43f7247f6 (beta)
http://hg.mozilla.org/qa/mozmill-tests/rev/0a247523c281 (release)

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Vlad, can you please file a new bug so we can get the workaround removed?