)]}'
{"tempest/api/object_storage/test_container_quotas.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"187a739fe29b810d12e83b67a105754192687ee2","unresolved":false,"context_lines":[{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    @testtools.skipIf(not container_quotas_available,"},{"line_number":71,"context_line":"                      \"Container quotas middleware not available\")"},{"line_number":72,"context_line":"    @attr(type\u003d[\"smoke\", ])"},{"line_number":73,"context_line":"    def test_upload_valid_object(self):"},{"line_number":74,"context_line":"        \"\"\"Attempts to uploads an object smaller than the bytes quota.\"\"\""},{"line_number":75,"context_line":"        object_name \u003d rand_name(name\u003d\"TestObject\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAASn%2F%2FFQ0%3D","line":72,"updated":"2013-09-12 10:16:34.000000000","message":"Why do you use an array type and trailing \u0027,\u0027?","commit_id":"a7808d9b2f5018eca42bb18bc0339b4b02a47918"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"709d7e280c7b08526d29f9de08dad3f5ab9387d5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright (C) 2013 Cloudwatt \u003ccontact@cloudwatt.com\u003e"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Author: Sahid Orentino Ferdjaoui \u003csahid.ferdjaoui@cloudwatt.com\u003e"},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAS3%2F%2F%2B%2Fk%3D","line":1,"updated":"2013-09-15 23:35:50.000000000","message":"Most other files starts with\n# vim: tabstop\u003d4 shiftwidth\u003d4 softtabstop\u003d4","commit_id":"b8ebb0cd6d183c8621edaac67bc9e25cfc84bbea"},{"author":{"_account_id":6847,"name":"Martina Kollarova","email":"mkollaro@gmail.com","username":"mkollaro"},"change_message_id":"a6aaa59e2d1995ae6a563f440cef1bd1f515a5a3","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        metadata \u003d {\"quota-bytes\": str(QUOTA_BYTES),"},{"line_number":52,"context_line":"                    \"quota-count\": str(QUOTA_COUNT), }"},{"line_number":53,"context_line":"        cls.container_client.update_container_metadata("},{"line_number":54,"context_line":"            cls.container_name, metadata)"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"    @classmethod"},{"line_number":57,"context_line":"    def tearDownClass(cls):"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2F9NQ%3D","line":54,"updated":"2013-09-16 16:15:11.000000000","message":"I think it would be better if you created the container for testing in setUp, not setUpClass and then just removed it using cls.delete_containers in tearDown. This would make the tests more isolated, plus you wouldn\u0027t need both tearDown and tearDownClass doing almost the same.","commit_id":"3b67059e089cb7312a2ea56d866e8a81bf2a64cc"},{"author":{"_account_id":6847,"name":"Martina Kollarova","email":"mkollaro@gmail.com","username":"mkollaro"},"change_message_id":"a6aaa59e2d1995ae6a563f440cef1bd1f515a5a3","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        super(ContainerQuotasTest, self).tearDown()"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    @testtools.skipIf(not container_quotas_available,"},{"line_number":71,"context_line":"                      \"Container quotas middleware not available\")"},{"line_number":72,"context_line":"    @attr(type\u003d\"smoke\")"},{"line_number":73,"context_line":"    def test_upload_valid_object(self):"},{"line_number":74,"context_line":"        \"\"\"Attempts to uploads an object smaller than the bytes quota.\"\"\""}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2F9MI%3D","line":71,"updated":"2013-09-16 16:15:11.000000000","message":"Put this message into a constant","commit_id":"3b67059e089cb7312a2ea56d866e8a81bf2a64cc"},{"author":{"_account_id":6847,"name":"Martina Kollarova","email":"mkollaro@gmail.com","username":"mkollaro"},"change_message_id":"a6aaa59e2d1995ae6a563f440cef1bd1f515a5a3","unresolved":false,"context_lines":[{"line_number":124,"context_line":"                          self.container_name, \"OverQuotaObject\", \"\")"},{"line_number":125,"context_line":"        resp, _ \u003d self.container_client.list_container_metadata("},{"line_number":126,"context_line":"            self.container_name)"},{"line_number":127,"context_line":"        nafter \u003d int(resp[\"x-container-object-count\"])"},{"line_number":128,"context_line":"        self.assertEqual(nbefore, nafter)"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAS3%2F%2F9Iw%3D","line":127,"updated":"2013-09-16 16:15:11.000000000","message":"How about you create helper methods _get_object_count() and _get_bytes_used()? It gets repeated a lot.","commit_id":"3b67059e089cb7312a2ea56d866e8a81bf2a64cc"}]}