)]}'
{"climate/tests/plugins/oshosts/test_reservation_pool.py":[{"author":{"_account_id":3012,"name":"Dina Belova","email":"dbelova@mirantis.com","username":"dbelova"},"change_message_id":"901762643894b7aaaadd322d609e0044b36e98c1","unresolved":false,"context_lines":[{"line_number":183,"context_line":""},{"line_number":184,"context_line":"    def test_add_computehost_with_host_id(self):"},{"line_number":185,"context_line":"        # NOTE(sbauza): Freepool.hosts only contains names of hosts, not UUIDs"},{"line_number":186,"context_line":"        self._patch_get_agg_from_whatever()"},{"line_number":187,"context_line":"        self.assertRaises(rp.HostNotInFreePool,"},{"line_number":188,"context_line":"                          self.pool.add_computehost, \u0027pool\u0027, \u00273\u0027)"},{"line_number":189,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAT3%2F%2F9hI%3D","line":186,"updated":"2013-12-26 08:59:07.000000000","message":"We\u0027ve fixed name for this function from get_agg_from_whatever to get_aggregate_from_name_or_id in the testing class itself. May you rename this patching method also here? I see we missed that, but that naming looks not really good :) That\u0027s quite cosmetic change, that\u0027s why I suppose it would be normal to fix it if this CR.\n\nThanks!","commit_id":"bce819244d138e9a7c38b9a441daff91ad5a0e36"}]}