Merge pull request #50 from browserstack/mobile_browsers

Mobile browsers

Author: dhimil

lib/configParser.js

@@ -24,8 +24,8 @@ var ConfigParser = {
 
   setBrowserVersion: function(browserObject, verStr) {
     var filteredBrowsers = ConfigParser.bsBrowsers.map(function(currentValue, index, array) {
-      if (currentValue.browser == browserObject.browser) {
-        return currentValue.browser_version; 
+      if (currentValue.browser.toLowerCase() == browserObject.browser) {
+        return (browserObject.mobile ? currentValue.os_version : currentValue.browser_version);
       }
     }).filter(function(currentValue, index, array) {
       return currentValue && array.indexOf(currentValue) === index;
@@ -42,33 +42,62 @@ var ConfigParser = {
 
   populateOsAndOsVersion: function(browserObject) {
     if (!(browserObject.os && browserObject.os_version)) {
-      var windowsFiltered = ConfigParser.bsBrowsers.filter(function(currentValue, index, array) {
-        return currentValue.os == 'Windows' && currentValue.browser == browserObject.browser && parseFloat(currentValue.browser_version).toPrecision(4) == parseFloat(browserObject.browser_version).toPrecision(4);
-      });
-      var osxFiltered = ConfigParser.bsBrowsers.filter(function(currentValue, index, array) {
-        return currentValue.os == 'OS X' && currentValue.browser == browserObject.browser && parseFloat(currentValue.browser_version).toPrecision(4) == parseFloat(browserObject.browser_version).toPrecision(4);
-      });
-      browserObject = windowsFiltered.length > 0 ? windowsFiltered[Math.floor(Math.random() * windowsFiltered.length)] : osxFiltered[Math.floor(Math.random() * osxFiltered.length)];
+      if (browserObject.mobile) {
+
+        var mobileFiltered = ConfigParser.bsBrowsers.filter(function(currentValue, index, array) {
+          return currentValue.browser.toLowerCase() == browserObject.browser && parseFloat(currentValue.os_version).toPrecision(4) == parseFloat(browserObject.os_version).toPrecision(4);
+        });
+
+        browserObject = mobileFiltered[Math.floor(Math.random() * mobileFiltered.length)];
+      }
+      else {
+
+        var windowsFiltered = ConfigParser.bsBrowsers.filter(function(currentValue, index, array) {
+          return currentValue.os == 'Windows' && currentValue.browser == browserObject.browser && parseFloat(currentValue.browser_version).toPrecision(4) == parseFloat(browserObject.browser_version).toPrecision(4);
+        });
+
+        var osxFiltered = ConfigParser.bsBrowsers.filter(function(currentValue, index, array) {
+          return currentValue.os == 'OS X' && currentValue.browser == browserObject.browser && parseFloat(currentValue.browser_version).toPrecision(4) == parseFloat(browserObject.browser_version).toPrecision(4);
+        });
+        browserObject = windowsFiltered.length > 0 ? windowsFiltered[Math.floor(Math.random() * windowsFiltered.length)] : osxFiltered[Math.floor(Math.random() * osxFiltered.length)];
+      }
+
     }
     return browserObject;
   },
 
   getBrowserObject: function(entry) {
     var browserObject = {};
+    var version = null;
+    var sliceStart = 1;
     if (typeof(entry) == 'string') {
       var browserData = entry.split("_");
       var lindex = browserData.length - 1;
-      browserObject.browser = browserData[0];
+      if (browserData[0] == 'mobile' || browserData[0] == 'android' || (browserData[0] == 'opera' && browserData[1] == 'browser')) {
+        browserObject.browser = browserData[0] + ' ' + browserData[1];
+        browserObject.mobile = true;
+        sliceStart = 2;
+      }
+      else {
+        browserObject.browser = browserData[0];
+      }
       if (browserData[lindex] && browserData[lindex].indexOf("+") == -1) {
-        if (["current", "previous"].indexOf(browserData[1]) != -1) {
-          browserObject.browser_version = ConfigParser.setBrowserVersion(browserObject, browserData[1]);
+        if (["current", "previous"].indexOf(browserData[lindex]) != -1) {
+          version = ConfigParser.setBrowserVersion(browserObject, browserData[lindex]);
         }
         else {
-          browserObject.browser_version = browserData.slice(1, lindex + 1).join(".");
+          version = browserData.slice(sliceStart, lindex + 1).join(".");
         }
       }
       else {
-        browserObject.browser_version = browserData.slice(1, lindex + 1).join(".");
+        version = browserData.slice(sliceStart, lindex + 1).join(".");
+      }
+      if (browserObject.mobile) {
+        browserObject.os_version = version;
+        browserObject.browser_version = null;
+      }
+      else {
+        browserObject.browser_version = version;
       }
     } else {
       browserObject = entry;

lib/utils.js

@@ -24,7 +24,7 @@ var uuid = function uuidGenerator() {
 };
 
 var browserString = function browserString(config) {
-  return config.os + ' ' + config.os_version + ', ' + titleCase(config.browser || config.device) + ' ' + config.browser_version;
+  return config.os + ' ' + config.os_version + ', ' + titleCase(config.browser || config.device) + ' ' + (config.browser_version || config.device);
 };
 
 var objectSize = function objectSize(obj) {