)]}'
{"nova/tests/unit/objects/test_compute_node.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"6a2371a0c22ebeea6dfc0be23b6ff6578e134ca6","unresolved":false,"context_lines":[{"line_number":155,"context_line":"    @mock.patch.object(db, \u0027compute_node_get\u0027)"},{"line_number":156,"context_line":"    def test_get_by_id(self, get_mock):"},{"line_number":157,"context_line":"        get_mock.return_value \u003d fake_compute_node"},{"line_number":158,"context_line":"        compute \u003d compute_node.ComputeNode.get_by_id(self.context, 123)"},{"line_number":159,"context_line":"        self.compare_obj(compute, fake_compute_node,"},{"line_number":160,"context_line":"                         subs\u003dself.subs(),"},{"line_number":161,"context_line":"                         comparators\u003dself.comparators())"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_63f15201","line":158,"updated":"2016-04-19 11:05:22.000000000","message":"To get the same coverage that in the old test you shall check if the mock was called with 123","commit_id":"9f2ec1ab3fe31c522c9e3b1d1523c3539123e271"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"6a2371a0c22ebeea6dfc0be23b6ff6578e134ca6","unresolved":false,"context_lines":[{"line_number":182,"context_line":""},{"line_number":183,"context_line":"    @mock.patch.object(db, \u0027compute_nodes_get_by_service_id\u0027)"},{"line_number":184,"context_line":"    def test_get_by_service_id(self, get_mock):"},{"line_number":185,"context_line":"        get_mock.return_value \u003d [fake_compute_node]"},{"line_number":186,"context_line":"        compute \u003d compute_node.ComputeNode.get_by_service_id(self.context, 456)"},{"line_number":187,"context_line":"        self.compare_obj(compute, fake_compute_node,"},{"line_number":188,"context_line":"                         subs\u003dself.subs(),"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_83dc6e74","line":185,"updated":"2016-04-19 11:05:22.000000000","message":"ditto","commit_id":"9f2ec1ab3fe31c522c9e3b1d1523c3539123e271"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"6a2371a0c22ebeea6dfc0be23b6ff6578e134ca6","unresolved":false,"context_lines":[{"line_number":227,"context_line":"    @mock.patch.object(db, \u0027compute_node_create\u0027)"},{"line_number":228,"context_line":"    @mock.patch(\u0027nova.db.compute_node_get\u0027, return_value\u003dfake_compute_node)"},{"line_number":229,"context_line":"    def test_create(self, mock_get, mock_create):"},{"line_number":230,"context_line":"        mock_create.return_value \u003d fake_compute_node"},{"line_number":231,"context_line":"        compute \u003d compute_node.ComputeNode(context\u003dself.context)"},{"line_number":232,"context_line":"        compute.service_id \u003d 456"},{"line_number":233,"context_line":"        compute.uuid \u003d uuidsentinel.fake_compute_node"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a122d0e_e3e56238","line":230,"updated":"2016-04-19 11:05:22.000000000","message":"ditto","commit_id":"9f2ec1ab3fe31c522c9e3b1d1523c3539123e271"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"84e2408c51088e948e44fbc9cd401ea40627b3cb","unresolved":false,"context_lines":[{"line_number":360,"context_line":""},{"line_number":361,"context_line":"    @mock.patch.object(db, \u0027compute_node_get_all\u0027)"},{"line_number":362,"context_line":"    def test_get_all(self, mock_get_all):"},{"line_number":363,"context_line":"        mock_get_all.return_value \u003d [fake_compute_node]"},{"line_number":364,"context_line":"        computes \u003d compute_node.ComputeNodeList.get_all(self.context)"},{"line_number":365,"context_line":"        self.assertEqual(1, len(computes))"},{"line_number":366,"context_line":"        self.compare_obj(computes[0], fake_compute_node,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_e9740f69","line":363,"range":{"start_line":363,"start_character":8,"end_line":363,"end_character":20},"updated":"2016-04-19 16:42:22.000000000","message":"check if the mock object is called once.","commit_id":"edb85d6c39118fcf890297c8ab1f4243a1e4afc4"},{"author":{"_account_id":20217,"name":"Anusha Unnam","email":"anusha.unnam25@gmail.com","username":"aunnam"},"change_message_id":"84e2408c51088e948e44fbc9cd401ea40627b3cb","unresolved":false,"context_lines":[{"line_number":369,"context_line":""},{"line_number":370,"context_line":"    @mock.patch.object(db, \u0027compute_node_search_by_hypervisor\u0027)"},{"line_number":371,"context_line":"    def test_get_by_hypervisor(self, mock_search):"},{"line_number":372,"context_line":"        mock_search.return_value \u003d [fake_compute_node]"},{"line_number":373,"context_line":"        computes \u003d compute_node.ComputeNodeList.get_by_hypervisor(self.context,"},{"line_number":374,"context_line":"                                                                  \u0027hyper\u0027)"},{"line_number":375,"context_line":"        self.assertEqual(1, len(computes))"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a122d0e_490c23ed","line":372,"range":{"start_line":372,"start_character":8,"end_line":372,"end_character":19},"updated":"2016-04-19 16:42:22.000000000","message":"ditto","commit_id":"edb85d6c39118fcf890297c8ab1f4243a1e4afc4"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"493ef06fd213b0475962f5b76a3356e8ecdfd596","unresolved":false,"context_lines":[{"line_number":264,"context_line":"        mock_create.assert_called_once_with("},{"line_number":265,"context_line":"            self.context, {\u0027uuid\u0027: fake_compute_node[\u0027uuid\u0027]})"},{"line_number":266,"context_line":""},{"line_number":267,"context_line":"    @mock.patch(\u0027nova.db.compute_node_create\u0027)"},{"line_number":268,"context_line":"    @mock.patch(\u0027nova.db.compute_node_get\u0027, return_value\u003dfake_compute_node)"},{"line_number":269,"context_line":"    def test_recreate_fails(self, mock_get, mock_create):"},{"line_number":270,"context_line":"        mock_create.return_value \u003d fake_compute_node"},{"line_number":271,"context_line":"        compute \u003d compute_node.ComputeNode(context\u003dself.context)"},{"line_number":272,"context_line":"        compute.service_id \u003d 456"},{"line_number":273,"context_line":"        compute.uuid \u003d uuidsentinel.fake_compute_node"}],"source_content_type":"text/x-python","patch_set":4,"id":"bab6814e_33eeeb8b","line":270,"range":{"start_line":267,"start_character":0,"end_line":270,"end_character":52},"updated":"2016-05-18 15:05:57.000000000","message":"nit: let\u0027s use the return_value consistently, either both mock should get the return value at the decorator or both get it in the test code","commit_id":"860e2572b1c3fcb2163c13f706d2f32b58df7a1d"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"7d849121a5b6ea8f30687a7b2c7cba2f8049b505","unresolved":false,"context_lines":[{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    @mock.patch.object(db, \u0027compute_node_get\u0027)"},{"line_number":157,"context_line":"    def test_get_by_id(self, get_mock):"},{"line_number":158,"context_line":"        get_mock.return_value \u003d fake_compute_node"},{"line_number":159,"context_line":"        compute \u003d compute_node.ComputeNode.get_by_id(self.context, 123)"},{"line_number":160,"context_line":"        self.compare_obj(compute, fake_compute_node,"},{"line_number":161,"context_line":"                         subs\u003dself.subs(),"}],"source_content_type":"text/x-python","patch_set":21,"id":"1a6eadb0_d38254c8","line":158,"range":{"start_line":158,"start_character":8,"end_line":158,"end_character":49},"updated":"2016-12-12 15:13:52.000000000","message":"I\u0027d personally prefer to see this as an argument to the decorator, but I won\u0027t block on that","commit_id":"e5c1762020205394df9c1826ce66f898e4a63a1f"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"61ccb19ab800a67c9acb3183238a2a7696d81e7a","unresolved":false,"context_lines":[{"line_number":244,"context_line":"        self.compare_obj(compute, fake_compute_node,"},{"line_number":245,"context_line":"                         subs\u003dself.subs(),"},{"line_number":246,"context_line":"                         comparators\u003dself.comparators())"},{"line_number":247,"context_line":"        param_dict \u003d {"},{"line_number":248,"context_line":"            \u0027service_id\u0027: 456,"},{"line_number":249,"context_line":"            \u0027stats\u0027: fake_stats_db_format,"},{"line_number":250,"context_line":"            \u0027host_ip\u0027: fake_host_ip,"}],"source_content_type":"text/x-python","patch_set":21,"id":"1a6eadb0_b4163053","line":247,"range":{"start_line":247,"start_character":8,"end_line":247,"end_character":18},"updated":"2016-12-14 14:46:16.000000000","message":"Nit: I suspect that argument has a name.","commit_id":"e5c1762020205394df9c1826ce66f898e4a63a1f"}]}