)]}'
{"tempest/api/network/test_networks.py":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"ac898e4c118d200951a955cb6b4fa2c9e9fece94","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":126,"context_line":"    def test_show_non_existent_subnet(self):"},{"line_number":127,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.show_subnet,"},{"line_number":128,"context_line":"                          \u002712345\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FPck%3D","line":128,"updated":"2013-08-04 15:27:08.000000000","message":"This is ok, but there really is no guarantee that a cloud tenant would not have created a tenant network called \"12345\". Seems to me that it would be better to generate a randomized name or a UUID and use that instead of \"12345\"","commit_id":"9fde9babe6162bce077a12af6316b2e1f4b76f3d"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"b430249113beeec1bd62f0dd1927e7b0012ac339","unresolved":false,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":121,"context_line":"    def test_show_non_existent_network(self):"},{"line_number":122,"context_line":"        non_exist_id \u003d rand_name(\u0027network\u0027)"},{"line_number":123,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.show_network,"},{"line_number":124,"context_line":"                          non_exist_id)"},{"line_number":125,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FPQs%3D","line":122,"updated":"2013-08-04 19:41:26.000000000","message":"this variable is only used once. You can skip it and use rand_name() directly in the assert call.","commit_id":"6656f58726194d6f1448d28ce4b00babd1e18c09"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"01b64debeb481c1f8591fe70684f5d24de444f52","unresolved":false,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":121,"context_line":"    def test_show_non_existent_network(self):"},{"line_number":122,"context_line":"        non_exist_id \u003d rand_name(\u0027network\u0027)"},{"line_number":123,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.show_network,"},{"line_number":124,"context_line":"                          non_exist_id)"},{"line_number":125,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FOug%3D","line":122,"in_reply_to":"AAAAQn%2F%2FPQs%3D","updated":"2013-08-05 04:31:59.000000000","message":"In python an additional helper variable is frequently used for readability.","commit_id":"6656f58726194d6f1448d28ce4b00babd1e18c09"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"b430249113beeec1bd62f0dd1927e7b0012ac339","unresolved":false,"context_lines":[{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":127,"context_line":"    def test_show_non_existent_subnet(self):"},{"line_number":128,"context_line":"        non_exist_id \u003d rand_name(\u0027subnet\u0027)"},{"line_number":129,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.show_subnet,"},{"line_number":130,"context_line":"                          non_exist_id)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FPQo%3D","line":128,"updated":"2013-08-04 19:41:26.000000000","message":"see above","commit_id":"6656f58726194d6f1448d28ce4b00babd1e18c09"}]}