in some tests in L3SchedulerTestBaseMixin plugin is mocked without any reason

Bug #1425988 reported by Rossella Sblendido
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Rossella Sblendido

Bug Description

Plugin shouldn't be mocked since it's defined by the classes that inherits from L3SchedulerTestBaseMixin

Changed in neutron:
assignee: nobody → Rossella Sblendido (rossella-o)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/159505

Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/159505
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=b5f7ad766346dc28fc612352ead616e8fce32469
Submitter: Jenkins
Branch: master

commit b5f7ad766346dc28fc612352ead616e8fce32469
Author: rossella <email address hidden>
Date: Thu Feb 26 15:23:32 2015 +0000

    Don't mock plugin in L3SchedulerTestBaseMixin

    There's no need to mock the plugin in L3SchedulerTestBaseMixin, since
    the plugin is defined by the classes that mix in L3SchedulerTestBaseMixin.

    Change-Id: Ia447936ba0d6696968acaf0edc26c7194be8d4d8
    Closes-bug: 1425988

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → kilo-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: kilo-3 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.