Noissue - Make subscribe link tests more reliable by waiting for options page to be ready

test/wrappers/pages.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 36 matching lines @@
     () => driver.switchTo().window(returnTo)
   );
 }
 
 function testSubscribeLink(driver)
 {
   return driver.findElement(By.id("subscribe-button")).click().then(() =>
     driver.wait(() =>
       driver.getAllWindowHandles().then(handles =>
         handles.length > 2 ? handles : null
-      ), 1000
+      ), 3000
     )
   ).then(handles =>
     closeWindow(driver, handles[2], handles[1], () =>
       driver.wait(until.ableToSwitchToFrame(0), 1000).then(() =>

[tlucas, 2018/09/26 08:28:17]

In order to not reject subscriptions, which might take longer (in case that
could happen), "wait()" should be called without a timeout IMHO.
Same below.

[Sebastian Noack, 2018/09/26 11:07:36]

But if there is a bug (caught by this test) with the subscription links, then
the tests would never complete rather than failing.

[tlucas, 2018/09/26 18:19:40]

Acknowledged.

         driver.wait(until.elementLocated(By.id("dialog-content-predefined")),
                     1000)
       ).then(dialog =>
         Promise.all([
           dialog.isDisplayed(),
           dialog.findElement(By.css("h3")).getText()
         ]).then(([displayed, title]) =>
         {
           assert.ok(displayed, "subscribe link: dialog shown");
           assert.equal(title, "ABP Testcase Subscription",
@@ skipping 173 matching lines @@
                 title
               )
             );
           });
         }
         return p2;
       });
     return p1;
   });
 });