)]}'
{"cinder/tests/unit/test_api_urlmap.py":[{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"92009631049316927d1e42454e57ac13140dd3f4","unresolved":false,"context_lines":[{"line_number":138,"context_line":"        expected_urlmap \u003d urlmap.URLMap(not_found_app\u003dNone)"},{"line_number":139,"context_line":"        expected_urlmap[\u0027http://foobar.com:20\u0027] \u003d \u0027some_app_returned\u0027\\"},{"line_number":140,"context_line":"                                                  \u0027_by_loader\u0027"},{"line_number":141,"context_line":"        self.assertEqual(expected_urlmap,"},{"line_number":142,"context_line":"                         urlmap.urlmap_factory(self.loader, self.global_conf,"},{"line_number":143,"context_line":"                                               **local_conf))"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3aaa91ec_b5b782f7","line":141,"updated":"2016-07-01 14:00:49.000000000","message":"Please assert for self.loader.get_app call.","commit_id":"5550921b268ce2ddc9f465c2714b4225b3d09c46"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"c03a123deb71b310236fc32d9413224bb610013e","unresolved":false,"context_lines":[{"line_number":138,"context_line":"        expected_urlmap \u003d urlmap.URLMap(not_found_app\u003dNone)"},{"line_number":139,"context_line":"        expected_urlmap[\u0027http://foobar.com:20\u0027] \u003d \u0027some_app_returned\u0027\\"},{"line_number":140,"context_line":"                                                  \u0027_by_loader\u0027"},{"line_number":141,"context_line":"        self.assertEqual(expected_urlmap,"},{"line_number":142,"context_line":"                         urlmap.urlmap_factory(self.loader, self.global_conf,"},{"line_number":143,"context_line":"                                               **local_conf))"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3aaa91ec_55e0de96","line":141,"in_reply_to":"3aaa91ec_b5b782f7","updated":"2016-07-01 14:06:18.000000000","message":"-1: I agree with Vipin, if you have added the call check in the other tests you should add it here as well.","commit_id":"5550921b268ce2ddc9f465c2714b4225b3d09c46"}]}