)]}'
{"nova/tests/api/openstack/compute/plugins/v3/test_instance_usage_audit_log.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"898fce457ac7603844ab9eefa46564377ae6ba18","unresolved":false,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"    def test_index_with_before(self):"},{"line_number":147,"context_line":"        req \u003d fakes.HTTPRequestV3.blank("},{"line_number":148,"context_line":"            \u0027/os-instance_usage_audit_log?before\u003d2012-07-05 10:00:00\u0027)"},{"line_number":149,"context_line":"        result \u003d self.controller.index(req)"},{"line_number":150,"context_line":"        self.assertIn(\u0027instance_usage_audit_log\u0027, result)"},{"line_number":151,"context_line":"        logs \u003d result[\u0027instance_usage_audit_log\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FXQQ%3D","line":148,"updated":"2013-08-01 11:42:38.000000000","message":"Do we want a format where we force space characters in a URL? Would be good to at least add a check when this is URL encoded that everything still works.","commit_id":"b30e3559cb3b6ebe3f60ca6d845b7906524a3c7a"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"53d70467e7095abb9a48ed61630809a37508660f","unresolved":false,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"    def test_index_with_before(self):"},{"line_number":147,"context_line":"        req \u003d fakes.HTTPRequestV3.blank("},{"line_number":148,"context_line":"            \u0027/os-instance_usage_audit_log?before\u003d2012-07-05 10:00:00\u0027)"},{"line_number":149,"context_line":"        result \u003d self.controller.index(req)"},{"line_number":150,"context_line":"        self.assertIn(\u0027instance_usage_audit_log\u0027, result)"},{"line_number":151,"context_line":"        logs \u003d result[\u0027instance_usage_audit_log\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FUEQ%3D","line":148,"in_reply_to":"AAAAQn%2F%2FXQQ%3D","updated":"2013-08-02 03:20:46.000000000","message":"you are right, I think we should use encoded URL for all the test. thanks","commit_id":"b30e3559cb3b6ebe3f60ca6d845b7906524a3c7a"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"898fce457ac7603844ab9eefa46564377ae6ba18","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    def test_index_with_errors(self):"},{"line_number":178,"context_line":"        req \u003d fakes.HTTPRequestV3.blank("},{"line_number":179,"context_line":"            \u0027/os-instance_usage_audit_log?before\u003d2012-07-07 10:00:00\u0027)"},{"line_number":180,"context_line":"        result \u003d self.controller.index(req)"},{"line_number":181,"context_line":"        self.assertIn(\u0027instance_usage_audit_log\u0027, result)"},{"line_number":182,"context_line":"        logs \u003d result[\u0027instance_usage_audit_log\u0027]"},{"line_number":183,"context_line":"        self.assertEquals(57, logs[\u0027total_instances\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FXQk%3D","line":180,"updated":"2013-08-01 11:42:38.000000000","message":"Can we check both formats and also check errors to both formats?","commit_id":"b30e3559cb3b6ebe3f60ca6d845b7906524a3c7a"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"53d70467e7095abb9a48ed61630809a37508660f","unresolved":false,"context_lines":[{"line_number":177,"context_line":"    def test_index_with_errors(self):"},{"line_number":178,"context_line":"        req \u003d fakes.HTTPRequestV3.blank("},{"line_number":179,"context_line":"            \u0027/os-instance_usage_audit_log?before\u003d2012-07-07 10:00:00\u0027)"},{"line_number":180,"context_line":"        result \u003d self.controller.index(req)"},{"line_number":181,"context_line":"        self.assertIn(\u0027instance_usage_audit_log\u0027, result)"},{"line_number":182,"context_line":"        logs \u003d result[\u0027instance_usage_audit_log\u0027]"},{"line_number":183,"context_line":"        self.assertEquals(57, logs[\u0027total_instances\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2FUEI%3D","line":180,"in_reply_to":"AAAAQn%2F%2FXQk%3D","updated":"2013-08-02 03:20:46.000000000","message":"Not sure I understand your comment. I guess you mean use deferent format for \u0027before\u0027 to test this case? I think it is same for any format of before.\n\nBut I miss some test for format of before, I will add.","commit_id":"b30e3559cb3b6ebe3f60ca6d845b7906524a3c7a"}]}