Fix test that broke in earlier CL.

native_client_sdk/src/build_tools/tests/sdk_update_test.py

Index: native_client_sdk/src/build_tools/tests/sdk_update_test.py
diff --git a/native_client_sdk/src/build_tools/tests/sdk_update_test.py b/native_client_sdk/src/build_tools/tests/sdk_update_test.py
index 692243caa3d3e5205ac11a31d2f44511e7f38dab..3b46f1bb262e87796b44c4968bcae3d22cfb0c4b 100755
--- a/native_client_sdk/src/build_tools/tests/sdk_update_test.py
+++ b/native_client_sdk/src/build_tools/tests/sdk_update_test.py
@@ -84,7 +84,7 @@ class TestSDKUpdate(unittest.TestCase):
     self.assertEqual(bundle_list.count('sdk_tools'), 2)
     self.assertEqual(bundle_list.count('test_1'), 2)
     self.assertEqual(bundle_list.count('test_2'), 1)
-    self.assertEqual(bundle_list.count('description:'), 7)
+    self.assertEqual(bundle_list.count('description:'), 8)

[Matt Ball, 2011/12/09 19:40:55]

This test has been a little fragile.  We should probably just remove the exact
numbers....

 
   def testUpdateHelp(self):
     '''Test the help for the update command'''
@@ -115,7 +115,7 @@ class TestSDKUpdate(unittest.TestCase):
     tools = sdk_update.ManifestTools(self._options)
     tools.LoadManifest()
     bundles = tools.GetBundles()
-    self.assertEqual(4, len(bundles))
+    self.assertEqual(5, len(bundles))
     local_manifest = sdk_update.SDKManifestFile(
         os.path.join(self._options.user_data_dir,
                      sdk_update.MANIFEST_FILENAME))
@@ -123,7 +123,7 @@ class TestSDKUpdate(unittest.TestCase):
     for bundle in bundles:
       bundle_name = bundle['name']
       self.assertTrue('revision' in bundle)
-      if bundle_name in ['test_1', 'test_2', 'pepper_1']:
+      if bundle_name in ['test_1', 'test_2', 'pepper_1', 'pepper_phony']:
         self.assertTrue(local_manifest.BundleNeedsUpdate(bundle))
       else:
         self.assertFalse(local_manifest.BundleNeedsUpdate(bundle))
@@ -133,7 +133,7 @@ class TestSDKUpdate(unittest.TestCase):
     tools = sdk_update.ManifestTools(self._options)
     tools.LoadManifest()
     bundles = tools.GetBundles()
-    self.assertEqual(4, len(bundles))
+    self.assertEqual(5, len(bundles))
     local_manifest = sdk_update.SDKManifestFile(
         os.path.join(self._options.user_data_dir,
                      sdk_update.MANIFEST_FILENAME))
@@ -149,7 +149,7 @@ class TestSDKUpdate(unittest.TestCase):
     tools = sdk_update.ManifestTools(self._options)
     tools.LoadManifest()
     bundles = tools.GetBundles()
-    self.assertEqual(4, len(bundles))
+    self.assertEqual(5, len(bundles))
     local_manifest = sdk_update.SDKManifestFile(
         os.path.join(self._options.user_data_dir,
                      sdk_update.MANIFEST_FILENAME))
@@ -157,7 +157,7 @@ class TestSDKUpdate(unittest.TestCase):
     # Test the | operator.
     for bundle in bundles:
       bundle_name = bundle['name']
-      if bundle_name in ['pepper_1', 'test_2']:
+      if bundle_name in ['pepper_1', 'test_2', 'pepper_phony']:
         continue
       local_test_bundle = local_manifest.GetBundleNamed(bundle_name)
       merged_bundle = local_test_bundle.MergeWithBundle(bundle)