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)
_______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : launchpad-reviewers@lists.launchpad.net Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp