)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"58b38a34c962e51c31f96f720095add35c696bca","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor test_user_update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"test_user_update looks a little mess and not easy to"},{"line_number":10,"context_line":"distinguish the test steps from the continuous lines,"},{"line_number":11,"context_line":"so this is to refactor it to make it more readable."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df3967d1_16dda76f","line":9,"range":{"start_line":9,"start_character":32,"end_line":9,"end_character":36},"updated":"2017-08-01 18:53:26.000000000","message":"messy","commit_id":"bb30391ac75642739abdbd1215cd24df7a3113cd"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"dccfc57f33fcaf0784d2eafd88e35761d958b067","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor test_user_update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"test_user_update looks a little messy and not easy to"},{"line_number":10,"context_line":"distinguish the test steps from the continuous lines,"},{"line_number":11,"context_line":"so this is to refactor it to make it more readable."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df3967d1_8bdba519","line":9,"range":{"start_line":9,"start_character":32,"end_line":9,"end_character":37},"updated":"2017-08-02 00:26:20.000000000","message":"done","commit_id":"d4f6aae3c4be2b892f687214261214ed0ee723db"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"1c095a2e0871baff95f08817f4b92e06e560098c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor test_user_update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"test_user_update looks a little messy and not easy to"},{"line_number":10,"context_line":"distinguish the test steps from the continuous lines,"},{"line_number":11,"context_line":"so this is to refactor it to make it more readable."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f436f4f_5fda5717","line":9,"range":{"start_line":9,"start_character":32,"end_line":9,"end_character":37},"in_reply_to":"9f436f4f_54da43e9","updated":"2017-08-09 03:28:51.000000000","message":"yeah, silly me, I\u0027ll be careful to avoid that:)","commit_id":"d4f6aae3c4be2b892f687214261214ed0ee723db"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"890fcd33adf74b95661945aa4b2a9b28eb32f3fa","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor test_user_update"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"test_user_update looks a little messy and not easy to"},{"line_number":10,"context_line":"distinguish the test steps from the continuous lines,"},{"line_number":11,"context_line":"so this is to refactor it to make it more readable."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9f436f4f_54da43e9","line":9,"range":{"start_line":9,"start_character":32,"end_line":9,"end_character":37},"in_reply_to":"df3967d1_8bdba519","updated":"2017-08-08 22:01:32.000000000","message":"replies in a different patch-set are a bit hard to read :)","commit_id":"d4f6aae3c4be2b892f687214261214ed0ee723db"}],"tempest/api/identity/admin/v3/test_users.py":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"58b38a34c962e51c31f96f720095add35c696bca","unresolved":false,"context_lines":[{"line_number":42,"context_line":"                         \u0027description\u0027: data_utils.rand_name(\u0027desc2\u0027),"},{"line_number":43,"context_line":"                         \u0027project_id\u0027: project[\u0027id\u0027],"},{"line_number":44,"context_line":"                         \u0027email\u0027: \u0027user2@testmail.tm\u0027,"},{"line_number":45,"context_line":"                         \u0027enable\u0027: False}"},{"line_number":46,"context_line":"        updated_user \u003d self.users_client.update_user("},{"line_number":47,"context_line":"            user[\u0027id\u0027], **update_kwargs)[\u0027user\u0027]"},{"line_number":48,"context_line":"        for field in update_kwargs:"}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_b65edb0f","line":45,"range":{"start_line":45,"start_character":26,"end_line":45,"end_character":32},"updated":"2017-08-01 18:53:26.000000000","message":"enabled\n\n(Double-checked here: https://developer.openstack.org/api-ref/identity/v3/index.html#update-user)","commit_id":"bb30391ac75642739abdbd1215cd24df7a3113cd"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"58b38a34c962e51c31f96f720095add35c696bca","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        for field in update_kwargs:"},{"line_number":49,"context_line":"            self.assertEqual(update_kwargs[field], updated_user[field])"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        # GET by id after updation"},{"line_number":52,"context_line":"        new_user_get \u003d self.users_client.show_user(user[\u0027id\u0027])[\u0027user\u0027]"},{"line_number":53,"context_line":"        # Assert response body of GET after updation"},{"line_number":54,"context_line":"        for field in update_kwargs:"}],"source_content_type":"text/x-python","patch_set":1,"id":"df3967d1_b68c9b7f","line":51,"range":{"start_line":51,"start_character":26,"end_line":51,"end_character":34},"updated":"2017-08-01 18:53:26.000000000","message":"updating\n\n(updation is not a word)","commit_id":"bb30391ac75642739abdbd1215cd24df7a3113cd"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"890fcd33adf74b95661945aa4b2a9b28eb32f3fa","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    def test_user_update(self):"},{"line_number":33,"context_line":"        # Test case to check if updating of user attributes is successful."},{"line_number":34,"context_line":"        # Creating first user"},{"line_number":35,"context_line":"        user \u003d self.setup_test_user()"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        # Creating second project for updation"},{"line_number":38,"context_line":"        project \u003d self.setup_test_project()"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f436f4f_f4725711","line":35,"range":{"start_line":35,"start_character":20,"end_line":35,"end_character":35},"updated":"2017-08-08 22:01:32.000000000","message":"This is not the same as L35 to L43 as many for fields were set.\nI think the original version made the code more readable and the assertion slightly stronger as it shows that the update is actually changing all fields.","commit_id":"d4f6aae3c4be2b892f687214261214ed0ee723db"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"1c095a2e0871baff95f08817f4b92e06e560098c","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    def test_user_update(self):"},{"line_number":33,"context_line":"        # Test case to check if updating of user attributes is successful."},{"line_number":34,"context_line":"        # Creating first user"},{"line_number":35,"context_line":"        user \u003d self.setup_test_user()"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"        # Creating second project for updation"},{"line_number":38,"context_line":"        project \u003d self.setup_test_project()"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f436f4f_5f9ed763","line":35,"range":{"start_line":35,"start_character":20,"end_line":35,"end_character":35},"in_reply_to":"9f436f4f_f4725711","updated":"2017-08-09 03:28:51.000000000","message":"Yes I agree to explicitly put all fields.","commit_id":"d4f6aae3c4be2b892f687214261214ed0ee723db"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"dccfc57f33fcaf0784d2eafd88e35761d958b067","unresolved":false,"context_lines":[{"line_number":42,"context_line":"                         \u0027description\u0027: data_utils.rand_name(\u0027desc2\u0027),"},{"line_number":43,"context_line":"                         \u0027project_id\u0027: project[\u0027id\u0027],"},{"line_number":44,"context_line":"                         \u0027email\u0027: \u0027user2@testmail.tm\u0027,"},{"line_number":45,"context_line":"                         \u0027enabled\u0027: False}"},{"line_number":46,"context_line":"        updated_user \u003d self.users_client.update_user("},{"line_number":47,"context_line":"            user[\u0027id\u0027], **update_kwargs)[\u0027user\u0027]"},{"line_number":48,"context_line":"        for field in update_kwargs:"}],"source_content_type":"text/x-python","patch_set":2,"id":"df3967d1_6bd641f4","line":45,"range":{"start_line":45,"start_character":26,"end_line":45,"end_character":33},"updated":"2017-08-02 00:26:20.000000000","message":"done","commit_id":"d4f6aae3c4be2b892f687214261214ed0ee723db"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"dccfc57f33fcaf0784d2eafd88e35761d958b067","unresolved":false,"context_lines":[{"line_number":48,"context_line":"        for field in update_kwargs:"},{"line_number":49,"context_line":"            self.assertEqual(update_kwargs[field], updated_user[field])"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        # GET by id after updating"},{"line_number":52,"context_line":"        new_user_get \u003d self.users_client.show_user(user[\u0027id\u0027])[\u0027user\u0027]"},{"line_number":53,"context_line":"        # Assert response body of GET after updation"},{"line_number":54,"context_line":"        for field in update_kwargs:"}],"source_content_type":"text/x-python","patch_set":2,"id":"df3967d1_2b0a7989","line":51,"range":{"start_line":51,"start_character":26,"end_line":51,"end_character":34},"updated":"2017-08-02 00:26:20.000000000","message":"done","commit_id":"d4f6aae3c4be2b892f687214261214ed0ee723db"}]}