launchpad-reviewers team mailing list archive

[Ines Almeida <mp+446164@xxxxxxxxxxxxxxxxxx>]

Ines Almeida has proposed merging ~ines-almeida/launchpad:add-bug-webhooks/fix-unittests into launchpad:master.

Commit message:
Remove webhooks feature flag from tests that no longer need it

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~ines-almeida/launchpad/+git/launchpad/+merge/446164
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~ines-almeida/launchpad:add-bug-webhooks/fix-unittests into launchpad:master.

diff --git a/lib/lp/bugs/tests/test_subscribers.py b/lib/lp/bugs/tests/test_subscribers.py
index 938002d..f6fce0d 100644
--- a/lib/lp/bugs/tests/test_subscribers.py
+++ b/lib/lp/bugs/tests/test_subscribers.py
@@ -9,10 +9,8 @@ from testtools.matchers import Equals, MatchesDict, MatchesStructure
 from zope.event import notify
 
 from lp.bugs.interfaces.bug import IBug
-from lp.bugs.interfaces.bugtarget import DISABLE_BUG_WEBHOOKS_FEATURE_FLAG
 from lp.bugs.interfaces.bugtask import BugTaskStatus
 from lp.bugs.subscribers.bugactivity import what_changed
-from lp.services.features.testing import FeatureFixture
 from lp.services.webapp.publisher import canonical_url
 from lp.services.webapp.snapshot import notify_modified
 from lp.testing import TestCaseWithFactory, person_logged_in
@@ -60,13 +58,6 @@ class TestBugWebhooksTriggered(TestCaseWithFactory):
 
     def setUp(self):
         super().setUp()
-        self.useFixture(
-            FeatureFixture(
-                {
-                    DISABLE_BUG_WEBHOOKS_FEATURE_FLAG: "on",
-                }
-            )
-        )
         self.target = self.factory.makeProduct()
         self.owner = self.target.owner
         self.bugtask = self.factory.makeBugTask(target=self.target)