remove wait

package.json

@@ -9,7 +9,7 @@
     "posttest:ci:functional": "bash tests/functional/test.sh stop",
     "test:ci:lint": "eslint ./src/**/*.js",
     "test:dev:unit": "./node_modules/karma/bin/karma start",
-    "test:dev:functional": "BROWSER=phantomjs bash tests/functional/test.sh start",
+    "test:dev:functional": "BROWSER=chrome bash tests/functional/test.sh start",
     "posttest:dev:functional": "bash tests/functional/test.sh stop",
     "test:dev:lint": "eslint ./src/scripts/**/*.js",
     "test:stylelint": "stylelint './src/**/*.scss' --config webpack/.stylelintrc",

tests/functional/pageobjects/BasePage.js

@@ -1,19 +1,15 @@
 module.exports = class BasePage {
 
-  constructor() {
+  constructor(selectors) {
 
-    this.selectors = {
-      SURAH_NAME: '.surah-body .navbar-text.surah-name'
-    };
-  }
-
-  wait(time = 200) {
+    this.selectors = Object.assign(selectors, {
+      SURAH_PAGE_SURAH_NAME: '.surah-body .navbar-text.surah-name'
+    });
 
-    browser.pause(time);
   }
 
   getSurahName() {
-    return browser.getText(this.selectors.SURAH_NAME);
+    return browser.getText(this.selectors.SURAH_PAGE_SURAH_NAME);
   }
 
   goHome() {

tests/functional/pageobjects/HomePage.js

@@ -1,19 +1,12 @@
 import BasePage from './BasePage';
+import selectors from './selectors';
+
 export default class HomePage extends BasePage {
 
   constructor(browser) {
 
-    const selectors = {
-      LANDING_TEXT: 'h4.title',
-      NEXT: 'a[data-direction="next"]',
-      PREVIOUS: 'a[data-direction="previous"]',
-      SURAH_LIST: '.row .col-md-4 li',
-      SEARCH_FORM: '.searchinput input',
-      SEARCH_RESULT_LIST: '.searchinput a',
-      SURAH_NAME: '.navbar-text.surah-name'
-    };
-
-    super(browser, selectors);
+    super(selectors);
+
     this.browser = browser;
     this.selectors = selectors;
   }
@@ -33,7 +26,7 @@ export default class HomePage extends BasePage {
   searchForSurahAndGoToSurahPage(searchQuery) {
 
     browser.setValue(this.selectors.SEARCH_FORM, searchQuery);
-    this.wait(1000);
+    browser.waitForVisible(this.selectors.SEARCH_RESULT_LIST);
     browser.click(this.selectors.SEARCH_RESULT_LIST);
     return this.getSurahName();
 
@@ -43,7 +36,6 @@ export default class HomePage extends BasePage {
     const surahs = browser.elements('.row .col-xs-7 span');
     const surahID = surahs.value[number].ELEMENT;
     browser.elementIdClick(surahID);
-    this.wait(1000);
     return browser.getUrl();
 
   }

tests/functional/pageobjects/selectors.js

@@ -0,0 +1,10 @@
+module.exports = {
+  LANDING_TEXT: 'h4.title',
+  NEXT: 'a[data-direction="next"]',
+  PREVIOUS: 'a[data-direction="previous"]',
+  SURAH_LIST: '.row .col-md-4 li',
+  SEARCH_FORM: '.searchinput input',
+  SEARCH_RESULT_LIST: '.searchinput a',
+  SURAH_PAGE: '.surah-body .surah-title',
+  SURAH_NAME: '.navbar-text.surah-name'
+}