launchpad-reviewers team mailing list archive

[Gavin Panella <gavin.panella@xxxxxxxxxxxxx>]

Gavin Panella has proposed merging lp:~allenap/maas/dig-faster into lp:maas.

Requested reviews:
  MAAS Maintainers (maas-maintainers)

For more details, see:
https://code.launchpad.net/~allenap/maas/dig-faster/+merge/125170
-- 
https://code.launchpad.net/~allenap/maas/dig-faster/+merge/125170
Your team MAAS Maintainers is requested to review the proposed merge of lp:~allenap/maas/dig-faster into lp:maas.

=== modified file 'src/maastesting/tests/test_bindfixture.py'
--- src/maastesting/tests/test_bindfixture.py	2012-09-18 15:30:59 +0000
+++ src/maastesting/tests/test_bindfixture.py	2012-09-19 11:16:23 +0000
@@ -71,10 +71,7 @@
                 # responsible for propagating fixture details.
                 gather_details(fixture.getDetails(), self.getDetails())
                 raise
-        error = self.assertRaises(
-            CalledProcessError, dig_call, fixture.config.port)
-        self.assertEqual(9, error.returncode)
-        # return code 9 means timeout.
+        self.assertFalse(fixture.runner.is_running())
 
     def test_config(self):
         # The configuration can be passed in.