Merge pull request #106 from Financial-Times/skip-safari-temporarily

Skip Safari because it fails
Financial-Times · Feb 6, 2019 · b1e3231 · b1e3231
2 parents ac8a85d + b1f1eb2
commit b1e3231
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)) {