yahoo-eng-team team mailing list archive

[Terry Wilson <1878708@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

After I876919dfc1fa0ae36bd99e3d760e38d207ee6ef3, two test classes that
inherit from both neutron's oslotest-based base classes and ovsdbapp's
unittest-based base class would fail due to mock.patch.stopall being
called twice. This appears to be because of some special handling in
oslotest addCleanup that checks a private _cleanups variable before
adding a cleanup to stopall. Changing the order of the imports so that
neutron can register its cleanups first seems to fix the issue.

** Affects: neutron
     Importance: Undecided
     Assignee: Terry Wilson (otherwiseguy)
         Status: In Progress


** Tags: ovn

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1878708

Title:
  mock.patch.stopall called twice on tests inheriting from ovsdbapp

Status in neutron:
  In Progress

Bug description:
  After I876919dfc1fa0ae36bd99e3d760e38d207ee6ef3, two test classes that
  inherit from both neutron's oslotest-based base classes and ovsdbapp's
  unittest-based base class would fail due to mock.patch.stopall being
  called twice. This appears to be because of some special handling in
  oslotest addCleanup that checks a private _cleanups variable before
  adding a cleanup to stopall. Changing the order of the imports so that
  neutron can register its cleanups first seems to fix the issue.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1878708/+subscriptions