Merge pull request #107 from Financial-Times/reinstate-safari-tests

Reinstate Safari tests
Financial-Times · Feb 8, 2019 · 9fea2a0 · 9fea2a0
2 parents b1e3231 + da38466
commit 9fea2a0
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)) {