)]}'
{"tests/compute/test_manager.py":[{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"7ab20eda0c029474d8504795c85219bf76593da1","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        # instance_get_all_by_host() so when the manager gets the list"},{"line_number":82,"context_line":"        # of instances to poll we can control the results."},{"line_number":83,"context_line":"        self.instance \u003d self.fake_instance(\u0027faux\u0027, \u0027active\u0027)"},{"line_number":84,"context_line":"        stillborn_instance \u003d self.fake_instance(\u0027stillborn\u0027, \u0027error\u0027)"},{"line_number":85,"context_line":"        self.stubs.Set(nova_client.Client, \u0027instance_get_all_by_host\u0027,"},{"line_number":86,"context_line":"                       lambda *x: [self.instance, stillborn_instance])"},{"line_number":87,"context_line":"        self.mox.ReplayAll()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2Bshk%3D","line":84,"updated":"2013-02-21 11:20:56.000000000","message":"Would it be more consistent with the other tests to use a Mock here, as opposed to a handcrafted class?\n\ne.g.:\n\n        def _fake_instance(name, state):\n            instance \u003d mock.MagicMock()\n            instance.name \u003d name\n            setattr(instance, \u0027OS-EXT-STS:vm_state\u0027, state)\n            return instance\n       \n        self.instance \u003d _fake_instance(\u0027faux\u0027, \u0027active\u0027)\n        stillborn_instance\u003d _fake_instance(\u0027stillborn\u0027, \u0027error\u0027)","commit_id":"a45094f6634db982179a1c104fa8e18b0bd8f7ef"},{"author":{"_account_id":4573,"name":"yunhong-jiang","email":"yunhong.jiang@intel.com","username":"yunhong-jiang"},"change_message_id":"16cd2af1833525cdd089b8789d3360f39ff93d0d","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        # instance_get_all_by_host() so when the manager gets the list"},{"line_number":82,"context_line":"        # of instances to poll we can control the results."},{"line_number":83,"context_line":"        self.instance \u003d self.fake_instance(\u0027faux\u0027, \u0027active\u0027)"},{"line_number":84,"context_line":"        stillborn_instance \u003d self.fake_instance(\u0027stillborn\u0027, \u0027error\u0027)"},{"line_number":85,"context_line":"        self.stubs.Set(nova_client.Client, \u0027instance_get_all_by_host\u0027,"},{"line_number":86,"context_line":"                       lambda *x: [self.instance, stillborn_instance])"},{"line_number":87,"context_line":"        self.mox.ReplayAll()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2BsfE%3D","line":84,"in_reply_to":"AAAAM3%2F%2Bshk%3D","updated":"2013-02-21 11:30:51.000000000","message":"Done","commit_id":"a45094f6634db982179a1c104fa8e18b0bd8f7ef"}]}