yahoo-eng-team team mailing list archive

[Ryan Rossiter <rlrossit@xxxxxxxxxx>]

Public bug reported:

The neutron.common.rpc module has no unit test coverage. The following
change was made to neutron/common/rpc.py:

 68 def cleanup():
 69     global TRANSPORT, NOTIFIER
 70     assert TRANSPORT is not None
 71     assert NOTIFIER is not None
 72 #    TRANSPORT.cleanup()
 73 #    TRANSPORT = NOTIFIER = None

No unit tests failed as a result of L72-73 being commented out, meaning
no unit tests are covering some of the main functionality of this
module. At a minimum, the public functions and classes should be unit
tested.

** Affects: neutron
     Importance: Undecided
         Status: New


** Tags: unittest

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

Title:
  neutron.common.rpc has no unit test coverage

Status in neutron:
  New

Bug description:
  The neutron.common.rpc module has no unit test coverage. The following
  change was made to neutron/common/rpc.py:

   68 def cleanup():
   69     global TRANSPORT, NOTIFIER
   70     assert TRANSPORT is not None
   71     assert NOTIFIER is not None
   72 #    TRANSPORT.cleanup()
   73 #    TRANSPORT = NOTIFIER = None

  No unit tests failed as a result of L72-73 being commented out,
  meaning no unit tests are covering some of the main functionality of
  this module. At a minimum, the public functions and classes should be
  unit tested.

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