Only run with eval disabled on Node 12+

- NODE_OPTIONS=--disallow-code-generation-from-strings is not available until Node 12
- split travis scripts to ensure failure of either stage will fail the build

Author: davidje13

.travis.yml

@@ -86,11 +86,13 @@ before_install:
       npm rebuild
     fi
 script:
-  # Run test script
+  # Run test script with eval disabled (requires Node 12+ for NODE_OPTIONS=--disallow-code-generation-from-strings)
   - |
-    if [[ "$(cut -d. -f1 <<< "$TRAVIS_NODE_VERSION")" -gt 8 ]]; then
+    if [[ "$(cut -d. -f1 <<< "$TRAVIS_NODE_VERSION")" -gt 11 ]]; then
       npm run test-noeval
     fi
+  # Run test script with eval enabled (including code coverage if available)
+  - |
     if npm -ps ls nyc | grep -q nyc; then
       npm run test-ci
     else

appveyor.yml

@@ -76,11 +76,11 @@ test_script:
   - ps: |
       node --version
       npm --version
-  # Run test script with eval disabled, if supported (node >= 9)
+  # Run test script with eval disabled, if supported (node 12+)
   - ps: |
-      if ([int]$env:nodejs_version.split(".")[0] -gt 8) {
+      if ([int]$env:nodejs_version.split(".")[0] -gt 11) {
         npm run test-noeval
       }
-  # Run test script
+  # Run test script with eval enabled
   - npm run test-eval
 version: "{build}"