Landscape Client

Code review comment for lp:~therve/landscape-client/fix-xen-report

  1. fix-xen-report
  2. Merge into trunk
Revision history for this message
Björn Tillenius (bjornt) wrote :

+1!

[1]

+ def test_get_vm_info_is_empty_without_xen_devices(self):
+ """
+ L{get_vm_info} returns "xen" if the /sys/bus/xen/devices directory
+ exists and contains file.

The docstring doesn't fit what is actually tested.

review: Approve

« Back to merge proposal