)]}'
{"tempest/api/object_storage/test_account_services.py":[{"author":{"_account_id":20553,"name":"Brian Ober","email":"bober@us.ibm.com","username":"ober37"},"change_message_id":"e2e49bd06de02900c2f866bf1544393db9d5b2c3","unresolved":false,"context_lines":[{"line_number":226,"context_line":"        self.assertEqual(len(container_list),"},{"line_number":227,"context_line":"                         min(limit, self.containers_count - 2))"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"    @test.idempotent_id(\u0027365e6fc7-1cfe-463b-a37c-8bd08d47b6aa\u0027)"},{"line_number":230,"context_line":"    def test_list_containers_with_prefix(self):"},{"line_number":231,"context_line":"        # list containers that have a name that starts with a prefix"},{"line_number":232,"context_line":"        prefix \u003d \u0027{0}-a\u0027.format(CONF.resources_prefix)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_716e9230","line":229,"updated":"2016-05-19 02:20:58.000000000","message":"Nice catch on the test case.   I am wondering if the approach for the test case could be improved.  Isn\u0027t CONF.resources_prefix used for all containers, so in theory all containers would be returned because all containers start with this prefix.\n\nI am wondering if would make more sense to write this case in such a way where a subset of containers are returned, so we know that the prefix logic is working properly.","commit_id":"947d23344c35347d44dc85dbcd699333e3004ad8"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"96d7df691de605294463849a34b2af80c822f051","unresolved":false,"context_lines":[{"line_number":226,"context_line":"        self.assertEqual(len(container_list),"},{"line_number":227,"context_line":"                         min(limit, self.containers_count - 2))"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"    @test.idempotent_id(\u0027365e6fc7-1cfe-463b-a37c-8bd08d47b6aa\u0027)"},{"line_number":230,"context_line":"    def test_list_containers_with_prefix(self):"},{"line_number":231,"context_line":"        # list containers that have a name that starts with a prefix"},{"line_number":232,"context_line":"        prefix \u003d \u0027{0}-a\u0027.format(CONF.resources_prefix)"}],"source_content_type":"text/x-python","patch_set":1,"id":"bab6814e_1546e7be","line":229,"in_reply_to":"bab6814e_716e9230","updated":"2016-05-19 16:47:04.000000000","message":"Actually the intention of the test is that only one container would be returned in the list, effectively filtering out all other containers.\n\nThe list of containers created in the resource_setup if the default value of CONF.resources_prefix is used are these:\ntempest-a--\u003crandom\u003e\ntempest-b--\u003crandom\u003e\n...\ntempest-f--\u003crandom\u003e\n\nI\u0027m assigning the prefix as CONF.resources_prefix plus a \"-a\" to target only the first container in the list.","commit_id":"947d23344c35347d44dc85dbcd699333e3004ad8"}]}