refactor

main.js

@@ -1,34 +1,8 @@
-function getsysInfo(sysInfo) { /* istanbul ignore next */
+function getsysInfo(sysInfo) {
   if (sysInfo) {
     return sysInfo;
   } else {
-    var os = require("os");
-    switch (os.platform()) {
-      case "darwin":
-        return "Macintosh; Intel Mac OS X " + require("get-osx-version").get();
-      case "windows":
-        switch (os.arch()) {
-          case "x64":
-            return "Windows; Win64; x64";
-          case "x86":
-            return "Windows; Win32; x86";
-          case "arm":
-            return "Windows; ARM";
-          default:
-            return "Windows";
-        }
-      case "linux":
-        switch (os.arch()) {
-          case "x64":
-            return "Linux; x86_64";
-          case "i686":
-            return "Linux; i686";
-          case "arm":
-            return "Linux; ARM";
-          default:
-            return "Linux";
-        }
-    }
+    return require("./sysInfo").get();
   }
 }
 module.exports = {

main.test.js

@@ -63,3 +63,11 @@ test('Safari Advanced', () => {
   expect(ua.browser.name).toBe('Safari');
   expect(ua.browser.version).toBe(version);
 });
+
+test('sysInfo', () => {
+  var version = '41.0';
+  var sysInfo = 'Macintosh; Intel Mac OS X 10.11';
+  ua = parser(useragent.firefox(version, sysInfo));
+  expect(ua.browser.name).toBe('Firefox');
+  expect(ua.browser.version).toBe(version);
+});

package.json

@@ -1,6 +1,6 @@
 {
   "name": "get-user-agent",
-  "version": "3.0.0",
+  "version": "3.1.0",
   "description": "easily get a useragent",
   "main": "main.js",
   "scripts": {
@@ -36,6 +36,7 @@
   "files": [
     "README.md",
     "LICENSE",
-    "main.js"
+    "main.js",
+    "sysInfo.js"
   ]
 }

sysInfo.js

@@ -0,0 +1,31 @@
+module.exports = {
+    get: function get() {
+    var os = require("os");
+    switch (os.platform()) {
+      case "darwin":
+        return "Macintosh; Intel Mac OS X " + require("get-osx-version").get();
+      case "windows":
+        switch (os.arch()) {
+          case "x64":
+            return "Windows; Win64; x64";
+          case "x86":
+            return "Windows; Win32; x86";
+          case "arm":
+            return "Windows; ARM";
+          default:
+            return "Windows";
+        }
+      case "linux":
+        switch (os.arch()) {
+          case "x64":
+            return "Linux; x86_64";
+          case "i686":
+            return "Linux; i686";
+          case "arm":
+            return "Linux; ARM";
+          default:
+            return "Linux";
+        }
+    }
+  }
+};