Merge lp:~abentley/launchpad/disable-isolation-test into lp:launchpad

[Aaron Bentley]

Disable spuriously-failing tests

This branch disables tests that are failing because of ordering issues per bug #551751

[Brad Crittenden (bac)]

review: Approve (code)

[Deryck Hodge (deryck)]

Aaron, are you going to land this until we can fix this properly?

[Aaron Bentley (abentley)]

On 04/06/2010 08:37 AM, Deryck Hodge wrote:
> Aaron, are you going to land this until we can fix this properly?

I am having trouble getting the test suite to pass on any of my
machines. On my karmic vm, the appserver dies, and PyGPGME is busted on
Lucid.

Aaron

=== added directory 'lib/canonical/launchpad/apidoc'
=== modified file 'lib/lp/bugs/externalbugtracker/tests/test_isolation.py'
--- lib/lp/bugs/externalbugtracker/tests/test_isolation.py	2010-03-18 18:07:26 +0000
+++ lib/lp/bugs/externalbugtracker/tests/test_isolation.py	2010-04-07 14:06:25 +0000
@@ -29,6 +29,9 @@
         stores[0].execute('SELECT 1')
 
     def test_gen_store_statuses(self):
+        # XXX: AaronBentley 2010-03-30 bug=551751: Test fails inconsistently.
+        return
+
         # All stores are either disconnected or idle when all
         # transactions have been aborted.
         transaction.abort()
@@ -43,6 +46,9 @@
                 for _, status in isolation.gen_store_statuses()))
 
     def test_is_transaction_in_progress(self):
+        # XXX: AaronBentley 2010-03-30 bug=551751: Test fails inconsistently.
+        return
+
         # is_transaction_in_progress() returns False when all
         # transactions have been aborted.
         transaction.abort()
@@ -53,6 +59,9 @@
         self.failUnless(isolation.is_transaction_in_progress())
 
     def test_check_no_transaction(self):
+        # XXX: AaronBentley 2010-03-30 bug=551751: Test fails inconsistently.
+        return
+
         # check_no_transaction() should be a no-op when there are no
         # transactions in operation.
         transaction.abort()
@@ -65,6 +74,9 @@
             isolation.check_no_transaction)
 
     def test_ensure_no_transaction(self):
+        # XXX: AaronBentley 2010-03-30 bug=551751: Test fails inconsistently.
+        return
+
         # ensure_no_transaction() is a decorator that raises
         # TransactionInProgress if a transaction has begun, else it
         # simply calls the wrapped function.