Thanks Gary, this is the change I did to the test.
=== modified file 'lib/lp/soyuz/scripts/tests/test_copypackage.py' --- lib/lp/soyuz/scripts/tests/test_copypackage.py 2009-10-16 15:11:05 +0000 +++ lib/lp/soyuz/scripts/tests/test_copypackage.py 2009-10-16 16:41:07 +0000 @@ -161,6 +161,9 @@ previously_open_files = number_of_open_files()
public_file = re_upload_file(private_file) + # The above call would've raised an error if the upload failed, but + # better safe than sorry. + self.assertIsNot(None, public_file)
open_files = number_of_open_files() - previously_open_files self.assertEqual(0, open_files)
« Back to merge proposal
Thanks Gary, this is the change I did to the test.
=== modified file 'lib/lp/ soyuz/scripts/ tests/test_ copypackage. py' soyuz/scripts/ tests/test_ copypackage. py 2009-10-16 15:11:05 +0000 soyuz/scripts/ tests/test_ copypackage. py 2009-10-16 16:41:07 +0000
previously_ open_files = number_ of_open_ files()
--- lib/lp/
+++ lib/lp/
@@ -161,6 +161,9 @@
+ # The above call would've raised an error if the upload failed, but
+ # better safe than sorry.
+ self.assertIsNo
open_files = number_ of_open_ files() - previously_ open_files
self. assertEqual( 0, open_files)