← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] lp:~cjwatson/launchpad/testfix-answers-mail-permissions into lp:launchpad

 

Colin Watson has proposed merging lp:~cjwatson/launchpad/testfix-answers-mail-permissions into lp:launchpad.

Commit message:
Revert unintended test change in answers-mail-permissions branch.

Requested reviews:
  Colin Watson (cjwatson)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/testfix-answers-mail-permissions/+merge/269265

Revert unintended test change in https://code.launchpad.net/~cjwatson/launchpad/answers-mail-permissions/+merge/269241.  Fixes buildbot failure.
-- 
Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
=== modified file 'lib/lp/answers/tests/test_questionjob.py'
--- lib/lp/answers/tests/test_questionjob.py	2015-08-26 16:07:10 +0000
+++ lib/lp/answers/tests/test_questionjob.py	2015-08-26 19:45:28 +0000
@@ -358,7 +358,7 @@
         make_question_job(
             self.factory, QuestionRecipientSet.ASKER_SUBSCRIBER,
             question=question, body=body)
-        with dbuser(config.answertracker.dbuser), block_on_job(self):
+        with block_on_job(self):
             transaction.commit()
         transaction.commit()
         mail = pop_remote_notifications()


References