Merge "Selenium: move logic for running tests from package.json to selenium.sh"

diff --git a/package.json b/package.json
index c33fb74..a282d69 100644 (file)
--- a/package.json
+++ b/package.json
@@ -5,7 +5,7 @@
     "qunit": "grunt qunit",
     "doc": "jsduck",
     "postdoc": "grunt copy:jsduck",
-    "selenium": "killall -0 chromedriver 2>/dev/null || chromedriver --url-base=/wd/hub --port=4444 & grunt webdriver:test; killall chromedriver"
+    "selenium": "./tests/selenium/selenium.sh"
   },
   "devDependencies": {
     "bluebird": "3.5.1",

diff --git a/tests/selenium/selenium.sh b/tests/selenium/selenium.sh
new file mode 100755 (executable)
index 0000000..519b7be
--- /dev/null
+++ b/tests/selenium/selenium.sh
@@ -0,0 +1,9 @@
+#!/usr/bin/env bash
+set -euo pipefail
+chromedriver --url-base=/wd/hub --port=4444 &
+# Make sure it is killed to prevent file descriptors leak
+function kill_chromedriver() {
+    killall chromedriver > /dev/null
+}
+trap kill_chromedriver EXIT
+./node_modules/.bin/grunt webdriver:test