Calling stopall not necessary in individual unit tests

Bug #1291130 reported by Kevin Benton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Kevin Benton

Bug Description

Once https://bugs.launchpad.net/neutron/+bug/1290550 is closed, calling stopall on mock.patch from individual unit tests is no longer required. Calls to stopall in existing code should be removed to prevent duplication.

Changed in neutron:
assignee: nobody → Kevin Benton (kevinbenton)
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/79830

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/79830
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=3196cef6986b733ff0a101164e5f3a49a10425c0
Submitter: Jenkins
Branch: master

commit 3196cef6986b733ff0a101164e5f3a49a10425c0
Author: Kevin Benton <email address hidden>
Date: Wed Mar 12 02:13:30 2014 +0000

    Removes calls to mock.patch.stopall in unit tests

    Removes unnecessary calls to mock.patch.stopall
    in individual unit tests since it is now
    automatically called from the base test case.

    Closes-Bug: #1291130
    Change-Id: Ia8e82d6a9a9bc9bd5f48146c1be53069ff1cdb01

Changed in neutron:
status: In Progress → Fix Committed
Akihiro Motoki (amotoki)
Changed in neutron:
milestone: none → icehouse-rc1
importance: Undecided → Low
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: icehouse-rc1 → 2014.1
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.