)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"e30852b4e4356283eed4822b20f3e553fe3c6ce8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0c2d8cf0_2419bb7b","updated":"2024-05-07 07:21:49.000000000","message":"I would just follow this pattern if you aren’t making use of the arguments.\n\n@mock.patch.object(rpc, \u0027get_client\u0027, mock.Mock())","commit_id":"e10da1c3b5db040fe7ae3377e96bdab5ab6998b7"},{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"51c234392ca61f79a5aff28ef61477fe49699810","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"29bdcdc8_139ef256","updated":"2024-05-21 13:19:01.000000000","message":"Looks good to me","commit_id":"71bffee11f9adfb196e4bffb322e369830254645"}],"designate/tests/unit/mdns/test_handler.py":[{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"b125dcd0e38075cc25ea7f22e9c56f589868ae4a","unresolved":true,"context_lines":[{"line_number":55,"context_line":"        self.assertIsInstance(self.handler.worker_api,"},{"line_number":56,"context_line":"                              worker_rpcapi.WorkerAPI)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    @mock.patch.object(dns.resolver.BaseResolver, \"read_registry\")"},{"line_number":59,"context_line":"    @mock.patch.object(dns.resolver.BaseResolver, \"read_resolv_conf\")"},{"line_number":60,"context_line":"    @mock.patch.object(rpc, \u0027get_client\u0027, mock.Mock())"},{"line_number":61,"context_line":"    @mock.patch.object(dns.resolver.Resolver, \u0027resolve\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"08352ab8_24dbfffb","line":58,"updated":"2024-05-07 09:50:46.000000000","message":"Would prefer something like, since we don\u0027t make use of these mocks.\n: @mock.patch.object(dns.resolver.BaseResolver, \"read_registry\", mock.Mock())","commit_id":"e10da1c3b5db040fe7ae3377e96bdab5ab6998b7"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"633929e7fc0906e03a45fddfea7f486091e9a4b7","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        self.assertIsInstance(self.handler.worker_api,"},{"line_number":56,"context_line":"                              worker_rpcapi.WorkerAPI)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    @mock.patch.object(dns.resolver.BaseResolver, \"read_registry\")"},{"line_number":59,"context_line":"    @mock.patch.object(dns.resolver.BaseResolver, \"read_resolv_conf\")"},{"line_number":60,"context_line":"    @mock.patch.object(rpc, \u0027get_client\u0027, mock.Mock())"},{"line_number":61,"context_line":"    @mock.patch.object(dns.resolver.Resolver, \u0027resolve\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9bf1588c_9a19c89d","line":58,"in_reply_to":"08352ab8_24dbfffb","updated":"2024-05-21 13:22:40.000000000","message":"Done","commit_id":"e10da1c3b5db040fe7ae3377e96bdab5ab6998b7"}]}