Updated userAgent test suite

Author: Zaydek Michels-Gualtieri

src/lib/Client/userAgent/index.js

@@ -5,23 +5,16 @@ function testUserAgent(substr) {
 	return navigator.userAgent.indexOf(substr) >= 0
 }
 
-// TODO: Refactor to isAppleDevice?
-// TODO: Refactor to isAndroidDevice?
+// Tests the user agent.
+//
+// TODO: Rename to userAgent.isApple?
 const userAgent = process.env.NODE_ENV === "test"
 	? Object.freeze({
-		// https://apple.com
 		get isAAPL() {
 			return testUserAgent("Mac OS X")
 		},
-		// https://android.com
-		get isGOOG() {
-			return testUserAgent("Android")
-		},
 	}) : Object.freeze({
-		// https://apple.com
 		isAAPL: testUserAgent("Mac OS X"),
-		// https://android.com
-		isGOOG: testUserAgent("Android"),
 	})
 
 export default userAgent

src/lib/Client/userAgent/index.test.js

@@ -1,42 +1,16 @@
-// NOTE: import userAgent from "..." breaks the test runner.
+import userAgent from "./index"
+
+import { // Unsorted
+	mockNonMacOS,
+	mockMacOS,
+} from "lib/Client/mockUserAgent"
 
-// https://stackoverflow.com/a/25518045
 test("isAAPL=false", () => {
-	Object.defineProperty(window.navigator, "userAgent", {
-		value: "...",
-		configurable: true,
-	})
-	const isAAPL = navigator.userAgent.indexOf("Mac OS X") >= 0
-	const userAgent = { isAAPL }
+	mockNonMacOS()
 	expect(userAgent.isAAPL).not.toBeTruthy()
 })
 
 test("isAAPL=true", () => {
-	Object.defineProperty(window.navigator, "userAgent", {
-		value: "... Mac OS X ...",
-		configurable: true,
-	})
-	const isAAPL = navigator.userAgent.indexOf("Mac OS X") >= 0
-	const userAgent = { isAAPL }
+	mockMacOS()
 	expect(userAgent.isAAPL).toBeTruthy()
 })
-
-test("isGOOG=false", () => {
-	Object.defineProperty(window.navigator, "userAgent", {
-		value: "...",
-		configurable: true,
-	})
-	const isGOOG = navigator.userAgent.indexOf("Android") >= 0
-	const userAgent = { isGOOG }
-	expect(userAgent.isGOOG).not.toBeTruthy()
-})
-
-test("isGOOG=true", () => {
-	Object.defineProperty(window.navigator, "userAgent", {
-		value: "... Android ...",
-		configurable: true,
-	})
-	const isGOOG = navigator.userAgent.indexOf("Android") >= 0
-	const userAgent = { isGOOG }
-	expect(userAgent.isGOOG).toBeTruthy()
-})