Make sure custom purposes aren't there usually

as-ideas · Jun 26, 2018 · a855167 · a855167
1 parent ed59f62
commit a855167
Showing 1 changed file with 16 additions and 4 deletions.
diff --git a/test/e2e/advanced_settings_custom_purposes_test.js b/test/e2e/advanced_settings_custom_purposes_test.js
@@ -11,17 +11,29 @@ module.exports = {
     browser
       .url(browser.globals.launch_url_host1 + 'demos/advanced-settings-custom-purposes.html')
       .deleteCookies();
+  },
 
+  'Shows five purposes by default': function (browser) {
     browser
-      .url(browser.globals.launch_url_host1 + 'demos/advanced-settings-custom-purposes.html')
+      .url(browser.globals.launch_url_host1 + 'demos/advanced-settings.html')
       .useCss()
       .waitForElementVisible('body', 1000, false)
-      .useXpath()
-      .waitForElementVisible(OIL_YES_BUTTON, 3000, false);
+      .useXpath().waitForElementVisible(OIL_YES_BUTTON, 2000, false)
+      .click(OIL_ADVANCED_SETTINGS)
+      .pause(200)
+      .waitForElementVisible(OIL_ADVANCED_SETTINGS_WRAPPER, 1000, false)
+      .pause(100)
+      .waitForElementNotPresent(OIL_ADVANCED_SETTINGS_CUSTOM_PURPOSE_HEADER, 1000, false)
+      .end();
   },
 
   'Displays custom purposes in list': function (browser) {
     browser
+      .url(browser.globals.launch_url_host1 + 'demos/advanced-settings-custom-purposes.html')
+      .useCss()
+      .waitForElementVisible('body', 1000, false)
+      .useXpath()
+      .waitForElementVisible(OIL_YES_BUTTON, 2000, false)
       .click(OIL_ADVANCED_SETTINGS)
       .pause(200)
       .waitForElementVisible(OIL_ADVANCED_SETTINGS_WRAPPER, 1000, false)
@@ -31,7 +43,7 @@ module.exports = {
       // FIXME selector fails in IE10
       // .assert.containsText(OIL_ADVANCED_SETTINGS_CUSTOM_PURPOSE_HEADER,'Custom Purpose 1')
       .pause(100)
-      .end()
+      .end();
   }
 
 };