launchpad-reviewers team mailing list archive

[İlker Emre Koç <mp+493348@xxxxxxxxxxxxxxxxxx>]

İlker Emre Koç has proposed merging ~ilkeremrekoc/launchpad:exportvul-test-fix into launchpad:master.

Commit message:
Use slow celery test for export vulnerability job

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~ilkeremrekoc/launchpad/+git/launchpad/+merge/493348

Tests for slow celery task queues needs a corresponding test layer.
Else, they timeout within the buildbot.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~ilkeremrekoc/launchpad:exportvul-test-fix into launchpad:master.

diff --git a/lib/lp/bugs/tests/test_exportvulnerabilityjob.py b/lib/lp/bugs/tests/test_exportvulnerabilityjob.py
index b0b6dec..98cfc98 100644
--- a/lib/lp/bugs/tests/test_exportvulnerabilityjob.py
+++ b/lib/lp/bugs/tests/test_exportvulnerabilityjob.py
@@ -24,7 +24,7 @@ from lp.services.job.interfaces.job import JobStatus
 from lp.services.job.tests import block_on_job
 from lp.services.librarian.model import LibraryFileAlias
 from lp.testing import TestCaseWithFactory
-from lp.testing.layers import CeleryJobLayer, LaunchpadZopelessLayer
+from lp.testing.layers import CelerySlowJobLayer, LaunchpadZopelessLayer
 
 
 class ExportVulnerabilityJobTests(TestCaseWithFactory):
@@ -323,7 +323,7 @@ class ExportVulnerabilityJobTests(TestCaseWithFactory):
 
 
 class ExportVulnerabilityTestViaCelery(TestCaseWithFactory):
-    layer = CeleryJobLayer
+    layer = CelerySlowJobLayer
 
     def setUp(self):
         super().setUp()