Update run_cukes overloading

Author: pirj

script/custom_build_functions.sh

@@ -1,22 +1,21 @@
 function run_cukes {
   if [ -d features ]; then
-    echo "${PWD}/bin/cucumber"
+    if is_ruby_25_plus; then
+      echo "${PWD}/bin/cucumber"
 
-    if is_mri_192; then
-      # For some reason we get SystemStackError on 1.9.2 when using
-      # the bin/cucumber approach below. That approach is faster
-      # (as it avoids the bundler tax), so we use it on rubies where we can.
-      bundle exec cucumber --strict
-    elif is_jruby; then
-      echo "WARNING: Cucumber is skipped on JRuby on rspec-core due to" \
-           "excessive build times (>45 minutes) causing timeouts on Travis"
-      return 0
+      if is_jruby; then
+        echo "WARNING: Cucumber is skipped on JRuby on rspec-core due to" \
+             "excessive build times (>45 minutes) causing timeouts on Travis"
+        return 0
+      else
+        # Prepare RUBYOPT for scenarios that are shelling out to ruby,
+        # and PATH for those that are using `rspec` or `rake`.
+        RUBYOPT="${RUBYOPT} -I${PWD}/../bundle -rbundler/setup" \
+           PATH="${PWD}/bin:$PATH" \
+           bin/cucumber --strict
+      fi
     else
-      # Prepare RUBYOPT for scenarios that are shelling out to ruby,
-      # and PATH for those that are using `rspec` or `rake`.
-      RUBYOPT="${RUBYOPT} -I${PWD}/../bundle -rbundler/setup" \
-         PATH="${PWD}/bin:$PATH" \
-         bin/cucumber --strict
+      echo "skipping features"
     fi
   fi
 }