)]}'
{"tempest/api/object_storage/test_object_version.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"910850ada44ed6c47628aa7794ed08e7f025de2a","unresolved":false,"context_lines":[{"line_number":48,"context_line":"    @testtools.skipIf("},{"line_number":49,"context_line":"        not CONF.object_storage_feature_enabled.object_versioning,"},{"line_number":50,"context_line":"        \u0027Object-versioning is disabled\u0027)"},{"line_number":51,"context_line":"    def test_versioned_container(self):"},{"line_number":52,"context_line":"        # create container"},{"line_number":53,"context_line":"        vers_container_name \u003d self.create_container()"},{"line_number":54,"context_line":"        resp, body \u003d self.container_client.create_container("}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_7d48872d","line":51,"updated":"2016-06-14 15:57:38.000000000","message":"The original test created only 1 container. Now you have 2, right ? One created L53 and another l54, is that normal ?","commit_id":"a4fe5d1b1b792037091692567408945d48549b26"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"11fc3076b46033385f94568b5dfa67fbe5776fca","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        \u0027Object-versioning is disabled\u0027)"},{"line_number":51,"context_line":"    def test_versioned_container(self):"},{"line_number":52,"context_line":"        # create container"},{"line_number":53,"context_line":"        vers_container_name \u003d data_utils.rand_name(name\u003d\u0027TestVersionContainer\u0027)"},{"line_number":54,"context_line":"        resp, body \u003d self.container_client.create_container("},{"line_number":55,"context_line":"            vers_container_name)"},{"line_number":56,"context_line":"        self.containers.append(vers_container_name)"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a9d85d2_38b4e0cf","line":53,"updated":"2016-06-21 08:55:51.000000000","message":"Why not:\n  vers_container_name \u003d self.create_container()\n\nThis way you reuse your new wrapper (which is the goal of the patch) and remove L54 and L56.","commit_id":"6bdc3ed6d4147e230718e2101d701870d9051382"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"594906490eb1301761a064c7c8c6b3b128d73472","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        \u0027Object-versioning is disabled\u0027)"},{"line_number":51,"context_line":"    def test_versioned_container(self):"},{"line_number":52,"context_line":"        # create container"},{"line_number":53,"context_line":"        vers_container_name \u003d data_utils.rand_name(name\u003d\u0027TestVersionContainer\u0027)"},{"line_number":54,"context_line":"        resp, body \u003d self.container_client.create_container("},{"line_number":55,"context_line":"            vers_container_name)"},{"line_number":56,"context_line":"        self.containers.append(vers_container_name)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1aa78d24_1a318f5a","line":53,"in_reply_to":"5a9d85d2_38b4e0cf","updated":"2016-07-07 09:50:25.000000000","message":"Ok, got it. You need to have the \"resp\" back to assert on the HTTP method (PUT).","commit_id":"6bdc3ed6d4147e230718e2101d701870d9051382"}]}