jenkins-bot has submitted this change and it was merged. Change subject: tests: zuul config is no more a class propery ......................................................................
tests: zuul config is no more a class propery It is solely used in setUpClass, no need to have it as a property. Change-Id: I82f589b1333299d6e1fc2236a2c3d8824f4a376e --- M tests/test_zuul_layout.py 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Hashar: Looks good to me, approved jenkins-bot: Verified diff --git a/tests/test_zuul_layout.py b/tests/test_zuul_layout.py index 449bdc8..391f636 100644 --- a/tests/test_zuul_layout.py +++ b/tests/test_zuul_layout.py @@ -31,7 +31,6 @@ class TestZuulLayout(unittest.TestCase): - zuul_config = None sched = None @classmethod @@ -43,7 +42,6 @@ cfg = ConfigParser.ConfigParser() cfg.add_section('zuul') cfg.set('zuul', 'layout_config', wmf_zuul_layout) - cls.zuul_config = cfg cls.sched = zuul.scheduler.Scheduler() # Reporters and Triggers are registered by zuul-server, not the @@ -51,7 +49,7 @@ cls.sched.registerTrigger(FakeTrigger(), 'gerrit') cls.sched.registerTrigger(FakeTrigger(), 'timer') cls.sched.registerTrigger(FakeTrigger(), 'zuul') - cls.sched._doReconfigureEvent(ReconfigureEvent(cls.zuul_config)) + cls.sched._doReconfigureEvent(ReconfigureEvent(cfg)) @classmethod def tearDownClass(cls): -- To view, visit https://gerrit.wikimedia.org/r/233710 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82f589b1333299d6e1fc2236a2c3d8824f4a376e Gerrit-PatchSet: 1 Gerrit-Project: integration/config Gerrit-Branch: master Gerrit-Owner: Hashar <has...@free.fr> Gerrit-Reviewer: Hashar <has...@free.fr> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits