Fix IE8 tests by downgrading uglify

Also adding Safari to Sauce browser matrix

Author: nfrasser

.eslintrc

@@ -28,5 +28,4 @@
 		"describe": false,
 		"it": false
 	}
-
 }

gulpfile.js

@@ -18,6 +18,7 @@ const eslint = require('gulp-eslint');
 const mocha = require('gulp-mocha');
 const rename = require('gulp-rename');
 const replace = require('gulp-replace');
+// NOTE: DO NOT Upgrade gulp-uglify, it breaks IE8 and the options don't seem to be working
 const uglify = require('gulp-uglify');
 const wrap = require('gulp-wrap');
 
@@ -347,6 +348,7 @@ gulp.task('build-benchmark', ['build-legacy'], () =>
 	.pipe(gulp.dest('build/benchmark'))
 );
 
+// NOTE: DO NOT Upgrade gulp-uglify, it breaks IE8 and the options don't seem to be working
 gulp.task('uglify', ['build-legacy'], () => {
 	let options = {
 		mangleProperties: {

package.json

@@ -45,7 +45,7 @@
     "gulp-mocha": "^3.0.1",
     "gulp-rename": "^1.2.0",
     "gulp-replace": "^0.5.4",
-    "gulp-uglify": "^2.0.0",
+    "gulp-uglify": "^1.5.4",
     "gulp-util": "^3.0.7",
     "gulp-wrap": "^0.13.0",
     "jsdom": "^9.3.0",

src/linkify/core/state.js

@@ -182,13 +182,11 @@ const TokenState = inherits(BaseState, createStateClass(), {
 	@return {Array} list of newly-created states
 */
 function stateify(str, start, endToken, defaultToken) {
-
-	let newStates = [];
-
-	var i = 0;
-	var len = str.length;
-	var state = start;
-	var nextState;
+	let i = 0,
+		len = str.length,
+		state = start,
+		newStates = [],
+		nextState;
 
 	// Find the next state without a jump to the next character
 	while (i < len && (nextState = state.next(str[i]))) {

test/ci.amd.conf.js

@@ -1,8 +1,7 @@
 // Karma CI configuration
 
-var
-base = require('./amd.conf'),
-extend = require('lodash/extend');
+const base = require('./amd.conf');
+const extend = require('lodash/extend');
 
 module.exports = function (config) {
 
@@ -18,6 +17,11 @@ module.exports = function (config) {
 			browserName: 'firefox',
 			version: '45'
 		},
+		sl_safari: {
+			base: 'SauceLabs',
+			browserName: 'safari',
+			version: '7'
+		},
 		sl_edge: {
 			base: 'SauceLabs',
 			browserName: 'MicrosoftEdge',
@@ -49,11 +53,11 @@ module.exports = function (config) {
 		// level of logging
 		// possible values: config.LOG_DISABLE || config.LOG_ERROR || config.LOG_WARN || config.LOG_INFO || config.LOG_DEBUG
 		sauceLabs: {
-			testName: 'Linkify Browser AMD Tests',
-			browserDisconnectTimeout: 30000,
-			browserNoActivityTimeout: 30000,
-			browserDisconnectTolerance: 3
+			testName: 'Linkify Browser AMD Tests'
 		},
+		browserDisconnectTimeout: 30000,
+		browserNoActivityTimeout: 30000,
+		browserDisconnectTolerance: 3,
 		customLaunchers: customLaunchers,
 		browsers: Object.keys(customLaunchers),
 		autoWatch: false,

test/ci.conf.js

@@ -52,7 +52,7 @@ module.exports = function (config) {
 		// level of logging
 		// possible values: config.LOG_DISABLE || config.LOG_ERROR || config.LOG_WARN || config.LOG_INFO || config.LOG_DEBUG
 		sauceLabs: {
-			testName: 'Linkify Browser Tests',
+			testName: 'Linkify Browser Tests'
 		},
 		browserDisconnectTimeout: 30000,
 		browserNoActivityTimeout: 30000,