launchpad-reviewers team mailing list archive

[Francesco Banconi <francesco.banconi@xxxxxxxxxxxxx>]

Francesco Banconi has proposed merging lp:~frankban/launchpad/bug-992958-intermittent into lp:launchpad.

Requested reviews:
  Launchpad Yellow Squad (yellow)
Related bugs:
  Bug #992958 in Launchpad itself: "lp.archivepublisher.tests.test_generate_ppa_htaccess.TestPPAHtaccessTokenGeneration.test_getNewTokensSinceLastRun_includes_tokens_during_last_run fails with scriptactivity_pkey error intermittently in parallel tests"
  https://bugs.launchpad.net/launchpad/+bug/992958

For more details, see:
https://code.launchpad.net/~frankban/launchpad/bug-992958-intermittent/+merge/104400

== Changes ==

lp.soyuz.scripts.tests.test_processupload.TestProcessUpload.testSimpleRun now explicitly marks the DB dirty.


-- 
https://code.launchpad.net/~frankban/launchpad/bug-992958-intermittent/+merge/104400
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.

=== modified file 'lib/lp/soyuz/scripts/tests/test_processupload.py'
--- lib/lp/soyuz/scripts/tests/test_processupload.py	2012-01-01 02:58:52 +0000
+++ lib/lp/soyuz/scripts/tests/test_processupload.py	2012-05-02 15:19:22 +0000
@@ -73,6 +73,9 @@
         # just to check if local assertion is working as expect.
         self.assertRaises(AssertionError, self.assertQueuePath, 'foobar')
 
+        # Explicitly mark the database dirty.
+        self.layer.force_dirty_database()
+
     def testTopLevelLockFile(self):
         """Try a simple process-upload run.