Bug 1398450 - Fix prefs test regression. r?jgraham draft
authorAndreas Tolfsen <ato@sny.no>
Sat, 09 Sep 2017 16:58:54 +0100
changeset 661933 8cb3599563f543b328b917097a26505a63a9a5d3
parent 661929 2a9cffb19ab58a7875aee5492c565c549c037511
child 730704 556c348efc5d89714a7aba6f455c84f2e6111a2e
push id78907
push userbmo:ato@sny.no
push dateSat, 09 Sep 2017 15:59:51 +0000
reviewersjgraham
bugs1398450
milestone57.0a1
Bug 1398450 - Fix prefs test regression. r?jgraham The marionette.defaultPrefs.port preference was changed to marionette.port, but because we currently do not run tests in CI we missed updating the test. MozReview-Commit-ID: LKstRYmJcMO
testing/geckodriver/src/capabilities.rs
--- a/testing/geckodriver/src/capabilities.rs
+++ b/testing/geckodriver/src/capabilities.rs
@@ -489,34 +489,39 @@ mod tests {
         assert_eq!(prefs.get("startup.homepage_welcome_url"),
                    Some(&Pref::new("data:text/html,PASS")));
     }
 
     #[test]
     fn test_prefs() {
         let encoded_profile = example_profile();
         let mut prefs: BTreeMap<String, Json> = BTreeMap::new();
-        prefs.insert("browser.display.background_color".into(),
-                     Json::String("#00ff00".into()));
+        prefs.insert(
+            "browser.display.background_color".into(),
+            Json::String("#00ff00".into()),
+        );
 
         let mut firefox_opts = Capabilities::new();
         firefox_opts.insert("profile".into(), encoded_profile);
         firefox_opts.insert("prefs".into(), Json::Object(prefs));
 
         let opts = make_options(firefox_opts);
         let mut profile = opts.profile.unwrap();
 
         let handler = MarionetteHandler::new(Default::default());
         handler
             .set_prefs(2828, &mut profile, true, opts.prefs)
             .unwrap();
 
         let prefs_set = profile.user_prefs().unwrap();
         println!("{:#?}", prefs_set.prefs);
 
-        assert_eq!(prefs_set.get("startup.homepage_welcome_url"),
-                   Some(&Pref::new("data:text/html,PASS")));
-        assert_eq!(prefs_set.get("browser.display.background_color"),
-                   Some(&Pref::new("#00ff00")));
-        assert_eq!(prefs_set.get("marionette.defaultPrefs.port"),
-                   Some(&Pref::new(2828)));
+        assert_eq!(
+            prefs_set.get("startup.homepage_welcome_url"),
+            Some(&Pref::new("data:text/html,PASS"))
+        );
+        assert_eq!(
+            prefs_set.get("browser.display.background_color"),
+            Some(&Pref::new("#00ff00"))
+        );
+        assert_eq!(prefs_set.get("marionette.port"), Some(&Pref::new(2828)));
     }
 }