loman: don't report error when adding an exact duplicate.

bin/loman_unittest.py

 #!/usr/bin/python
 
 # Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Unittests for loman."""
 
 import os
 import StringIO
@@ skipping 34 matching lines @@
     self.assertEqual(
       ptree.ToString(),
       self.utf8 + '<manifest>\n'
       '<project name="foo" path="path/to/foo" workon="True" />\n'
       '</manifest>')
 
   def testAddDup(self):
     ptree = loman.LocalManifest('<manifest>\n</manifest>')
     ptree.Parse()
     ptree.AddWorkonProject('foo', 'path/to/foo')
-    self.assertTrue(not ptree.AddWorkonProject('foo', 'path/to/foo'))
+    self.assertTrue(ptree.AddWorkonProject('foo', 'path/to/foo'))
     self.assertTrue(not ptree.AddWorkonProject('foo', 'path/foo'))
     self.assertTrue(not ptree.AddWorkonProject('foobar', 'path/to/foo'))
 
 class MainTest(unittest.TestCase):
 
   def setUp(self):
     self.utf8 = "<?xml version='1.0' encoding='UTF-8'?>\n"
     self.tiny_manifest = '<manifest>\n</manifest>'
     self.stderr = sys.stderr
     sys.stderr = StringIO.StringIO()
@@ skipping 29 matching lines @@
       '<project name="foo" path="path/to/foo" workon="True" />\n'
       '</manifest>\n')
 
   def testAddDup(self):
     temp = tempfile.NamedTemporaryFile('w')
     print >> temp, '<manifest>\n</manifest>'
     temp.flush()
     os.fsync(temp.fileno())
     loman.main(['loman', 'add', '--workon', '-f',
                 temp.name, 'foo', 'path/to/foo'])
+    loman.main(['loman', 'add', '--workon', '-f',
+                temp.name, 'foo', 'path/to/foo'])
     self.assertRaises(SystemExit, loman.main,
                       ['loman', 'add', '--workon', '-f',
-                       temp.name, 'foo', 'path/to/foo'])
+                       temp.name, 'foo', 'path/foo'])
 
 
 if __name__ == '__main__':
   unittest.main()