export for mocking

Author: jennifer-shehane

packages/server/lib/browsers/chrome.ts

@@ -106,7 +106,7 @@ const _getChromePreferencesWithDefaults = (userDir: string): Bluebird<ChromePref
   return _getChromePreferences(userDir)
   .then((existingPrefs) => {
     // Merge default preferences with existing preferences
-    const defaultPrefs = _getDefaultChromePreferences()
+    const defaultPrefs = module.exports._getDefaultChromePreferences()
 
     return _mergeChromePreferences(defaultPrefs, existingPrefs)
   })
@@ -600,7 +600,7 @@ export = {
 
     // Merge preferences BEFORE writing them to disk
     // Start with defaults merged with raw preferences
-    let finalPreferences = _mergeChromePreferences(_getDefaultChromePreferences(), rawPreferences)
+    let finalPreferences = _mergeChromePreferences(module.exports._getDefaultChromePreferences(), rawPreferences)
 
     if (launchOptions.preferences) {
       finalPreferences = _mergeChromePreferences(finalPreferences, launchOptions.preferences as ChromePreferences)

packages/server/test/unit/browsers/chrome_spec.js

@@ -1050,7 +1050,24 @@ describe('lib/browsers/chrome', () => {
 
       return chrome._getChromePreferencesWithDefaults('/foo')
       .then((result) => {
-        expect(result).to.deep.eq(mockDefaults)
+        // Should merge defaults with existing preferences, where existing values take precedence
+        expect(result).to.deep.eq({
+          default: {
+            fake_preference: {
+              value: 'value',
+            },
+            existing: 'value', // existing preference should be merged in
+          },
+          defaultSecure: {
+            secure: 'value', // existing preference should be merged in
+          },
+          localState: {
+            fake_local_state: {
+              value: 'value',
+            },
+            local: 'value', // existing preference should be merged in
+          },
+        })
       })
       .finally(() => {
         mockDefaultPrefs.restore()