From 0f72123c43bad7db4e7ea1c163043c34707e6aa4 Mon Sep 17 00:00:00 2001
From: Chris Rebert <code@rebertia.com>
Date: Mon, 16 Dec 2013 00:30:04 -0800
Subject: [PATCH] move Sauce test browsers list out of Gruntfile into separate
 file

---
 Gruntfile.js       | 85 +------------------------------------------
 sauce_browsers.yml | 91 ++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 92 insertions(+), 84 deletions(-)
 create mode 100644 sauce_browsers.yml

diff --git a/Gruntfile.js b/Gruntfile.js
index d350ce3ad6..1a0531b9f1 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -238,90 +238,7 @@ module.exports = function (grunt) {
           build: process.env.TRAVIS_JOB_ID,
           concurrency: 3,
           urls: ['http://127.0.0.1:3000/js/tests/index.html'],
-          browsers: [
-            // See https://saucelabs.com/docs/platforms/webdriver
-            {
-              browserName: 'safari',
-              version: '7',
-              platform: 'OS X 10.9'
-            },
-            {
-              browserName: 'chrome',
-              version: '31',
-              platform: 'OS X 10.9'
-            },
-            /* FIXME: currently fails 1 tooltip test
-            {
-              browserName: 'firefox',
-              version: '25',
-              platform: 'OS X 10.6'
-            },*/
-            // Mac Opera not currently supported by Sauce Labs
-            /* FIXME: currently fails 1 tooltip test
-            {
-              browserName: 'internet explorer',
-              version: '11',
-              platform: 'Windows 8.1'
-            },*/
-            /*
-            {
-              browserName: 'internet explorer',
-              version: '10',
-              platform: 'Windows 8'
-            },
-            {
-              browserName: 'internet explorer',
-              version: '9',
-              platform: 'Windows 7'
-            },
-            {
-              browserName: 'internet explorer',
-              version: '8',
-              platform: 'Windows 7'
-            },
-            {// unofficial
-              browserName: 'internet explorer',
-              version: '7',
-              platform: 'Windows XP'
-            },
-            */
-            {
-              browserName: 'chrome',
-              version: '31',
-              platform: 'Windows 8.1'
-            },
-            {
-              browserName: 'firefox',
-              version: '25',
-              platform: 'Windows 8.1'
-            },
-            // Win Opera 15+ not currently supported by Sauce Labs
-            {
-              browserName: 'iphone',
-              version: '6.1',
-              platform: 'OS X 10.8'
-            },
-            // iOS Chrome not currently supported by Sauce Labs
-            // Linux (unofficial)
-            {
-              browserName: 'chrome',
-              version: '30',
-              platform: 'Linux'
-            },
-            {
-              browserName: 'firefox',
-              version: '25',
-              platform: 'Linux'
-            }
-            // Android Chrome not currently supported by Sauce Labs
-            /* Android Browser (super-unofficial)
-            {
-              browserName: 'android',
-              version: '4.0',
-              platform: 'Linux'
-            }
-            */
-          ],
+          browsers: grunt.file.readYAML('sauce_browsers.yml')
         }
       }
     }
diff --git a/sauce_browsers.yml b/sauce_browsers.yml
new file mode 100644
index 0000000000..d015d83c82
--- /dev/null
+++ b/sauce_browsers.yml
@@ -0,0 +1,91 @@
+[
+  # Docs: https://saucelabs.com/docs/platforms/webdriver
+
+  {
+    browserName: "safari",
+    version: "7",
+    platform: "OS X 10.9"
+  },
+  {
+    browserName: "chrome",
+    version: "31",
+    platform: "OS X 10.9"
+  },
+
+  # { # FIXME: currently fails 1 tooltip test
+  #   browserName: "firefox",
+  #   version: "25",
+  #   platform: "OS X 10.6"
+  # },
+
+  # Mac Opera not currently supported by Sauce Labs
+
+  # { # FIXME: currently fails 1 tooltip test
+  #   browserName: "internet explorer",
+  #   version: "11",
+  #   platform: "Windows 8.1"
+  # },
+
+  # {
+  #   browserName: "internet explorer",
+  #   version: "10",
+  #   platform: "Windows 8"
+  # },
+  # {
+  #   browserName: "internet explorer",
+  #   version: "9",
+  #   platform: "Windows 7"
+  # },
+  # {
+  #   browserName: "internet explorer",
+  #   version: "8",
+  #   platform: "Windows 7"
+  # },
+
+  # { # Unofficial
+  #   browserName: "internet explorer",
+  #   version: "7",
+  #   platform: "Windows XP"
+  # },
+
+  {
+    browserName: "chrome",
+    version: "31",
+    platform: "Windows 8.1"
+  },
+  {
+    browserName: "firefox",
+    version: "25",
+    platform: "Windows 8.1"
+  },
+
+  # Win Opera 15+ not currently supported by Sauce Labs
+
+  {
+    browserName: "iphone",
+    version: "6.1",
+    platform: "OS X 10.8"
+  },
+
+  # iOS Chrome not currently supported by Sauce Labs
+
+  # Linux (unofficial):
+  {
+    browserName: "chrome",
+    version: "30",
+    platform: "Linux"
+  },
+  {
+    browserName: "firefox",
+    version: "25",
+    platform: "Linux"
+  }
+
+  # Android Chrome not currently supported by Sauce Labs
+
+  # { # Android Browser (super-unofficial)
+  #   browserName: "android",
+  #   version: "4.0",
+  #   platform: "Linux"
+  # }
+]
-- 
GitLab