Merge pull request #108 from Financial-Times/remove-safari-again

Remove Safari because tests are broken
Financial-Times · Feb 8, 2019 · 148bfcb · 148bfcb
2 parents 9fea2a0 + 41a9f82
commit 148bfcb
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/lib/smoke/setup-page.js b/lib/smoke/setup-page.js
@@ -20,7 +20,8 @@ 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', 'safari', 'internet explorer', 'MicrosoftEdge'];
+	const enabledBrowsers = globalOpts.browsers || ['chrome', 'firefox', 'internet explorer', 'MicrosoftEdge'];
+	// const enabledBrowsers = globalOpts.browsers || ['chrome', 'firefox', 'safari', 'internet explorer', 'MicrosoftEdge'];
 
 	options.browsersToRun = ['chrome'];
 	if(Array.isArray(browsers)) {