Bug 617320: fix for tests around engine disabling. r=mconnor
authorRichard Newman <rnewman@mozilla.com>
Tue, 07 Dec 2010 09:17:52 -0800
changeset 58797 f11c4d60b43cacb5650008d8991e69b69d54f023
parent 58765 680d3557cafee52e1db57829238e61c8ee3ee42f
child 58799 55adcf8cfb94a55c60c72cd10a12f860734a522d
child 58877 83b27b845f94535e97dafdbcfbf4b5c486ac2a58
push id1
push usershaver@mozilla.com
push dateTue, 04 Jan 2011 17:58:04 +0000
reviewersmconnor
bugs617320
Bug 617320: fix for tests around engine disabling. r=mconnor
services/sync/tests/unit/test_service_sync_updateEnabledEngines.js
--- a/services/sync/tests/unit/test_service_sync_updateEnabledEngines.js
+++ b/services/sync/tests/unit/test_service_sync_updateEnabledEngines.js
@@ -211,21 +211,21 @@ function test_disabledRemotely() {
     Service._ignorePrefObserver = true;
     engine.enabled = true;
     Service._ignorePrefObserver = false;
 
     _("Sync.");
     Weave.Service.login();
     Weave.Service.sync();
 
-    _("Engine is disabled.");
-    do_check_false(engine.enabled);
+    _("Engine is not disabled: only one client.");
+    do_check_true(engine.enabled);
+    
+    // TODO: add a second client and verify that the local pref is changed.
 
-    _("Meta record isn't uploaded.");
-    do_check_false(!!metaWBO.data.engines.steam);
   } finally {
     server.stop(do_test_finished);
     Service.startOver();
   }
 }
 
 function test_dependentEnginesEnabledLocally() {
   _("Test: Engine is disabled on remote clients and enabled locally");