yahoo-eng-team team mailing list archive

[Ryan Rossiter <rlrossit@xxxxxxxxxx>]

Public bug reported:

It seems that there are no unit tests for the nova.rpc module. Because
nova.rpc is used by many other modules, changes to it do cause UT
failures. But it is possible to remove essential parts without failing
any unit tests. An example of this is removing TRANSPORT.cleanup() from
the cleanup function.

Tests should be added for all of the public functions, along with tests
for RequestContextSerializer for both a given base, and a null base.

** Affects: nova
     Importance: Undecided
     Assignee: Ryan Rossiter (rlrossit)
         Status: New

** Changed in: nova
     Assignee: (unassigned) => Ryan Rossiter (rlrossit)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1533741

Title:
  nova.rpc module is only indirectly unit tested

Status in OpenStack Compute (nova):
  New

Bug description:
  It seems that there are no unit tests for the nova.rpc module. Because
  nova.rpc is used by many other modules, changes to it do cause UT
  failures. But it is possible to remove essential parts without failing
  any unit tests. An example of this is removing TRANSPORT.cleanup()
  from the cleanup function.

  Tests should be added for all of the public functions, along with
  tests for RequestContextSerializer for both a given base, and a null
  base.

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