add basic test suite to isMobile browser utility

jlmakes · jlmakes · commit 1ec2629a741a · 2016-12-25T12:35:51.000+01:00
diff --git a/src/utils/browser.js b/src/utils/browser.js
@@ -1,3 +1,8 @@
+export function isMobile (agent = navigator.userAgent) {
+	return /Android|iPhone|iPad|iPod/i.test(agent)
+}
+
+
 export function isNode (target) {
 	return typeof window.Node === 'object'
 		? target instanceof window.Node
@@ -31,9 +36,3 @@ export function transitionSupported () {
 	const style = document.documentElement.style
 	return 'transition' in style || 'WebkitTransition' in style
 }
-
-
-/* istanbul ignore next */
-export function isMobile () {
-	return /Android|iPhone|iPad|iPod/i.test(navigator.userAgent)
-}
diff --git a/test/utils/browser.spec.js b/test/utils/browser.spec.js
@@ -1,8 +1,40 @@
-import { isNode, isNodeList, transformSupported, transitionSupported } from '../../src/utils/browser'
+import { isMobile, isNode, isNodeList, transformSupported, transitionSupported } from '../../src/utils/browser'
 
 
 describe('Browser Utilities', () => {
 
+	describe('isMobile()', () => {
+
+		it('should return true when passed a mobile user agent', () => {
+
+			const android = `Mozilla/5.0 (Linux; U; Android 4.2; en-us;
+				Android SDK built for x86 Build/JOP40C) AppleWebKit/534.30
+				(KHTML, like Gecko) Version/4.0 Mobile Safari/534.30`
+
+			const iPhone = `Mozilla/5.0 (iPhone; CPU iPhone OS 10_10_5 like Mac OS X)
+				AppleWebKit/600.1.4 (KHTML, like Gecko) Version/8.0 Mobile/12B411 Safari/600.1.4`
+
+			expect(isMobile(android)).to.be.true
+			expect(isMobile(iPhone)).to.be.true
+		})
+
+		it('should return false when passed a desktop user agent', () => {
+
+			const chrome = `Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36
+				(KHTML, like Gecko) Chrome/50.0.2661.75 Safari/537.36`
+
+			const firefox = `Mozilla/5.0 (compatible; MSIE 10.0; Windows NT 6.1;
+				WOW64; Trident/6.0; SLCC2; .NET CLR 2.0.50727; .NET4.0C; .NET4.0E)`
+
+			expect(isMobile(chrome)).to.be.false
+			expect(isMobile(firefox)).to.be.false
+		})
+
+		it('should work when not passed an explicit user agent', () => {
+			expect(isMobile()).to.be.a('boolean')
+		})
+	})
+
 	describe('isNode()', () => {
 
 		it('should return true when passed a DOM node', () => {
