)]}'
{"nova/tests/unit/api/openstack/compute/test_attach_interfaces.py":[{"author":{"_account_id":5538,"name":"Rushi Agrawal","email":"rushi.openstack@gmail.com","username":"rushiagr"},"change_message_id":"d13bb8c93e8561b28c89a19d39a6b1585381d262","unresolved":false,"context_lines":[{"line_number":187,"context_line":"                          FAKE_PORT_ID1)"},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"    def test_delete(self):"},{"line_number":190,"context_line":"        self.stubs.Set(compute_api.API, \u0027detach_interface\u0027,"},{"line_number":191,"context_line":"                       fake_detach_interface)"},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"        result \u003d self.attachments.delete(self.req, FAKE_UUID1, FAKE_PORT_ID1)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a68dd71_df8271c8","line":190,"updated":"2016-01-20 16:34:16.000000000","message":"Found this instance, and many more instances of stubs.Set in this file. If it\u0027s intentional, I would prefer if a more descriptive commit message is written as to why only some parts of this file are changed and some not. My feeling was that if one file is chosen, all instances of stubs.Set from that file is being removed.","commit_id":"4f474994c7d485b7ae4a798cffb7dfcdad932a8c"},{"author":{"_account_id":16907,"name":"Diana Clarke","email":"diana.joan.clarke@gmail.com","username":"diana-clarke"},"change_message_id":"dd938d6283b8fefdcfeee44d32c5c408a4d3dcd8","unresolved":false,"context_lines":[{"line_number":187,"context_line":"                          FAKE_PORT_ID1)"},{"line_number":188,"context_line":""},{"line_number":189,"context_line":"    def test_delete(self):"},{"line_number":190,"context_line":"        self.stubs.Set(compute_api.API, \u0027detach_interface\u0027,"},{"line_number":191,"context_line":"                       fake_detach_interface)"},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"        result \u003d self.attachments.delete(self.req, FAKE_UUID1, FAKE_PORT_ID1)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a68dd71_d39eea06","line":190,"in_reply_to":"9a68dd71_df8271c8","updated":"2016-01-20 17:45:04.000000000","message":"There are over 300 calls to compute API stubs, so I did those in a different patch to take advantage of search \u0026 replace, and narrow the scope of the patch (make reviews easier). Once these high-churn patches are merged, I will go back to removing mox entirely on a per file basis.","commit_id":"4f474994c7d485b7ae4a798cffb7dfcdad932a8c"}]}