Launchpad itself

Merge lp:~frankban/launchpad/bug-987442-testSimpleScriptRun into lp:launchpad

  1. bug-987442-testSimpleScriptRun
  2. Merge into devel
Proposed by Francesco Banconi on 2012-04-26
Status: Merged
Approved by: Gary Poster on 2012-04-26
Approved revision: no longer in the source branch.
Merged at revision: 15158
Proposed branch: lp:~frankban/launchpad/bug-987442-testSimpleScriptRun
Merge into: lp:launchpad
Diff against target: 19 lines (+2/-0)
1 file modified
lib/lp/soyuz/tests/test_initializedistroseriesjob.py (+2/-0)
To merge this branch: bzr merge lp:~frankban/launchpad/bug-987442-testSimpleScriptRun
Reviewer Review Type Date Requested Status
Gary Poster (community) 2012-04-26 Approve on 2012-04-26
Review via email: mp+103706@code.launchpad.net

Description of the Change

== Changes ==

This is similar to bug 849056: I've found another test running a cron script and not marking the db dirty.
Applied the fix used by Graham.

To post a comment you must log in.
Gary Poster (gary) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/soyuz/tests/test_initializedistroseriesjob.py'
2--- lib/lp/soyuz/tests/test_initializedistroseriesjob.py 2012-04-20 18:28:17 +0000
3+++ lib/lp/soyuz/tests/test_initializedistroseriesjob.py 2012-04-26 14:23:49 +0000
4@@ -36,6 +36,7 @@
5 from lp.testing.layers import (
6 CeleryJobLayer,
7 DatabaseFunctionalLayer,
8+ DatabaseLayer,
9 LaunchpadZopelessLayer,
10 )
11
12@@ -352,6 +353,7 @@
13 def test_cronscript(self):
14 run_script(
15 'cronscripts/run_jobs.py', ['-v', 'initializedistroseries'])
16+ DatabaseLayer.force_dirty_database()
17
18
19 class TestViaCelery(TestCaseWithFactory):