Merge pull request #109 from Financial-Times/reinstate-safari-again

Reinstate Safari tests again
Financial-Times · Feb 8, 2019 · e354a1d · e354a1d
2 parents 148bfcb + 7d7945e
commit e354a1d
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/lib/smoke/setup-page.js b/lib/smoke/setup-page.js
@@ -20,8 +20,7 @@ module.exports = (path, urlOpts, suiteOpts, globalOpts) => {
 	options.breakpoint = urlOpts.breakpoint || suiteOpts.breakpoint || globalOpts.breakpoint;
 
 	const browsers = urlOpts.browsers || suiteOpts.browsers;
-	const enabledBrowsers = globalOpts.browsers || ['chrome', 'firefox', 'internet explorer', 'MicrosoftEdge'];
-	// const enabledBrowsers = globalOpts.browsers || ['chrome', 'firefox', 'safari', 'internet explorer', 'MicrosoftEdge'];
+	const enabledBrowsers = globalOpts.browsers || ['chrome', 'firefox', 'safari', 'internet explorer', 'MicrosoftEdge'];
 
 	options.browsersToRun = ['chrome'];
 	if(Array.isArray(browsers)) {