)]}'
{"test/functional/test_object.py":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"fe88af097d7658e3e64c481c51abc89cb8d8229a","unresolved":true,"context_lines":[{"line_number":942,"context_line":"    @requires_acls"},{"line_number":943,"context_line":"    def test_read_only(self):"},{"line_number":944,"context_line":"        if tf.skip3:"},{"line_number":945,"context_line":"            raise SkipTest"},{"line_number":946,"context_line":""},{"line_number":947,"context_line":"        def get_listing(url, token, parsed, conn):"},{"line_number":948,"context_line":"            conn.request(\u0027GET\u0027, \u0027%s/%s\u0027 % (parsed.path, self.container), \u0027\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3e77c540_40cc107a","line":945,"updated":"2023-02-17 00:22:41.000000000","message":"I was mildly curious, why not self.skipTest()? One less import. You even changed it in test_dlo.py, but not here.","commit_id":"be16d6c4fd7b61cee7c947fad9ef0473cb6661dc"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"8b61c96c5625c3514af9260f62ac9f51c6474f82","unresolved":true,"context_lines":[{"line_number":942,"context_line":"    @requires_acls"},{"line_number":943,"context_line":"    def test_read_only(self):"},{"line_number":944,"context_line":"        if tf.skip3:"},{"line_number":945,"context_line":"            raise SkipTest"},{"line_number":946,"context_line":""},{"line_number":947,"context_line":"        def get_listing(url, token, parsed, conn):"},{"line_number":948,"context_line":"            conn.request(\u0027GET\u0027, \u0027%s/%s\u0027 % (parsed.path, self.container), \u0027\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ecf4fa78_28ba2649","line":945,"in_reply_to":"3e77c540_40cc107a","updated":"2023-02-17 01:27:05.000000000","message":"Just \u0027cause there\u0027s like 10 more of these scattered through the file. I\u0027m not opposed to swapping all\n\n raise SkipTest\n\nfor\n\n self.skipTest()\n\n(or even better, make sure we include messages for all skips!) but I didn\u0027t want to force that much churn on any reviewers ;-)","commit_id":"be16d6c4fd7b61cee7c947fad9ef0473cb6661dc"},{"author":{"_account_id":34892,"name":"ASHWIN A NAIR","display_name":"indianwhocodes","email":"nairashwin952013@gmail.com","username":"indianwhocodes","status":"Nvidia"},"change_message_id":"d23f1597bfd42749afdfc2742146d10624532952","unresolved":false,"context_lines":[{"line_number":942,"context_line":"    @requires_acls"},{"line_number":943,"context_line":"    def test_read_only(self):"},{"line_number":944,"context_line":"        if tf.skip3:"},{"line_number":945,"context_line":"            raise SkipTest"},{"line_number":946,"context_line":""},{"line_number":947,"context_line":"        def get_listing(url, token, parsed, conn):"},{"line_number":948,"context_line":"            conn.request(\u0027GET\u0027, \u0027%s/%s\u0027 % (parsed.path, self.container), \u0027\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f18ae64_1862755a","line":945,"in_reply_to":"ecf4fa78_28ba2649","updated":"2023-08-17 16:15:30.000000000","message":"Ack","commit_id":"be16d6c4fd7b61cee7c947fad9ef0473cb6661dc"}]}
