)]}'
{"nova/tests/unit/virt/xenapi/stubs.py":[{"author":{"_account_id":6735,"name":"Bob Ball","email":"bob.ball@citrix.com","username":"bob-ball"},"change_message_id":"0b032f4679e37151072f7ae77bba52a5e1e16612","unresolved":false,"context_lines":[{"line_number":60,"context_line":"                    product_brand\u003d\u0027XenServer\u0027, platform_version\u003d(1, 9, 0),"},{"line_number":61,"context_line":"                    **opt_args):"},{"line_number":62,"context_line":"    \"\"\"Stubs out methods from XenAPISession.\"\"\""},{"line_number":63,"context_line":"    stubs.Set(session.XenAPISession, \u0027_create_session\u0027,"},{"line_number":64,"context_line":"              lambda s, url: cls(url, **opt_args))"},{"line_number":65,"context_line":"    stubs.Set(session.XenAPISession, \u0027_get_product_version_and_brand\u0027,"},{"line_number":66,"context_line":"              lambda s: (product_version, product_brand))"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f287b81_54e2fd30","line":63,"updated":"2017-08-25 08:24:29.000000000","message":"I think this is wrong (there is no _create_session in the current code) - but it\u0027s also indicative that we\u0027re doing the wrong thing here.\nDo we have to use os-xenapi\u0027s XenAPI Session in these test cases?\nCan\u0027t we use Fake Session?\n\nGenerally it concerns me that Nova is relying on internal implementation of an external library here.","commit_id":"5450e64bdbc6b60554e111bb95c53c9a4c674f96"}]}