<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style id="owaParaStyle" type="text/css">P {margin-top:0;margin-bottom:0;}</style>
</head>
<body ocsi="0" fpstyle="1">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">Hello.<br>
<br>
I would like to request a FFE for the Hyper-V unit tests refactoring blueprint: <a href="https://blueprints.launchpad.net/nova/+spec/hyper-v-test-refactoring" target="_blank">
https://blueprints.launchpad.net/nova/+spec/hyper-v-test-refactoring</a><br>
<br>
The point of the blueprint was to get rid of the ancient test_hypervapi.py tests, that use mox, as they prove more and more difficult to maintain, especially when adding new features or fixing bugs. Those tests would be replaced with mock unit tests, per Ops
class.<br>
<br>
There were 11 commits in total, 6 already merged, 5 remain. Out of these 5, the last 2 are trivial:<br>
<br>
[1] <a href="https://review.openstack.org/#/c/138934/" target="_blank">https://review.openstack.org/#/c/138934/</a><br>
[2] <a href="https://review.openstack.org/#/c/139796/" target="_blank">https://review.openstack.org/#/c/139796/</a><br>
[3] <a href="https://review.openstack.org/#/c/139797/" target="_blank">https://review.openstack.org/#/c/139797/</a><br>
<br>
[4] <a href="https://review.openstack.org/148980" target="_blank">https://review.openstack.org/148980</a> - unit tests for methods that have 1 instruction each. Just to have coverage on all the modules.<br>
<br>
[5] <a href="https://review.openstack.org/139798" target="_blank">https://review.openstack.org/139798</a> - just removes test_hypervapi.py<br>
<br>
The commits have been reviewed, already have a couple of +1s.<br>
<br>
<br>
Note: this blueprint is limited to the Hyper-V unit tests and does not change the functionality of the Driver in any way. It is barely worthy of the name blueprint and I consider it more of a bug, rather than a blueprint. This will improve maintainability,
readability and coverage for the Hyper-V classes.<br>
<br>
Best regards,<br>
<br>
Claudiu Belu<br>
</div>
</body>
</html>