)]}'
{"tempest/api/identity/admin/v3/test_credentials.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"d8fbbcfb038db001ea93a446bccbec70a1edf4fa","unresolved":false,"context_lines":[{"line_number":80,"context_line":"                         get_body[\u0027blob\u0027][\u0027secret\u0027])"},{"line_number":81,"context_line":"        self.assertEqual(update_body[\u0027project_id\u0027],"},{"line_number":82,"context_line":"                         get_body[\u0027project_id\u0027])"},{"line_number":83,"context_line":"        self.assertEqual(user_body[\u0027id\u0027], get_body[\u0027user_id\u0027])"},{"line_number":84,"context_line":"        self.addCleanup(self._delete_credential, cred[\u0027id\u0027])"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    @attr(type\u003d\u0027smoke\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F4ow%3D","line":83,"updated":"2013-07-19 20:06:21.000000000","message":"can we try to get this denser and loop through a set of keys we should compare?","commit_id":"bb0e94686b26ee10893e2054be52adbfd64db849"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"d8fbbcfb038db001ea93a446bccbec70a1edf4fa","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        self.assertEqual(update_body[\u0027project_id\u0027],"},{"line_number":82,"context_line":"                         get_body[\u0027project_id\u0027])"},{"line_number":83,"context_line":"        self.assertEqual(user_body[\u0027id\u0027], get_body[\u0027user_id\u0027])"},{"line_number":84,"context_line":"        self.addCleanup(self._delete_credential, cred[\u0027id\u0027])"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"    @attr(type\u003d\u0027smoke\u0027)"},{"line_number":87,"context_line":"    def test_credentials_list_delete(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F4os%3D","line":84,"updated":"2013-07-19 20:06:21.000000000","message":"this needs to happen earlier, otherwise if the asserts fail we could leave this around.","commit_id":"bb0e94686b26ee10893e2054be52adbfd64db849"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"d8fbbcfb038db001ea93a446bccbec70a1edf4fa","unresolved":false,"context_lines":[{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        for i in creds:"},{"line_number":115,"context_line":"            fetched_cred_ids.append(i[\u0027id\u0027])"},{"line_number":116,"context_line":"        missing_creds \u003d\\"},{"line_number":117,"context_line":"            [c for c in created_cred_ids"},{"line_number":118,"context_line":"             if c not in fetched_cred_ids]"},{"line_number":119,"context_line":"        self.assertEqual(0, len(missing_creds),"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAQn%2F%2F4oo%3D","line":116,"updated":"2013-07-19 20:06:21.000000000","message":"you should be able to fit this on one line","commit_id":"bb0e94686b26ee10893e2054be52adbfd64db849"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"ba1dd9c925599dfbba0c19bd247a3425d9abcdc8","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        u_password \u003d rand_name(\u0027pass-\u0027)"},{"line_number":36,"context_line":"        for i in range(2):"},{"line_number":37,"context_line":"            _, cls.project \u003d cls.v3_client.create_project("},{"line_number":38,"context_line":"                rand_name(\u0027project-\u0027), description\u003drand_name(\u0027project-desc-\u0027))"},{"line_number":39,"context_line":"            cls.projects.append(cls.project[\u0027id\u0027])"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        _, cls.user_body \u003d cls.v3_client.create_user("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2FrBY%3D","line":38,"updated":"2013-07-25 11:29:11.000000000","message":"I think not checking the status codes in a setupclass can be OK.\nBut looks like it is ignored in another keystone test cases as well.\n\nIt should be fixed in a different change.","commit_id":"5ab3944cccb7d40641b39070875daea8a82a0bc8"},{"author":{"_account_id":6455,"name":"Harika Vakadi","email":"harika_vakadi@persistent.co.in","username":"harika-vakadi"},"change_message_id":"6d98bc30886c721c0a6060aed060e10d6b60eb54","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        u_password \u003d rand_name(\u0027pass-\u0027)"},{"line_number":36,"context_line":"        for i in range(2):"},{"line_number":37,"context_line":"            _, cls.project \u003d cls.v3_client.create_project("},{"line_number":38,"context_line":"                rand_name(\u0027project-\u0027), description\u003drand_name(\u0027project-desc-\u0027))"},{"line_number":39,"context_line":"            cls.projects.append(cls.project[\u0027id\u0027])"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        _, cls.user_body \u003d cls.v3_client.create_user("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2Fj4Y%3D","line":38,"in_reply_to":"AAAAQn%2F%2FniY%3D","updated":"2013-07-29 07:01:02.000000000","message":"This is the regular practice followed.","commit_id":"5ab3944cccb7d40641b39070875daea8a82a0bc8"},{"author":{"_account_id":6455,"name":"Harika Vakadi","email":"harika_vakadi@persistent.co.in","username":"harika-vakadi"},"change_message_id":"df15ce804829449bc7b68f0dcfa3a93bbf95d041","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        u_password \u003d rand_name(\u0027pass-\u0027)"},{"line_number":36,"context_line":"        for i in range(2):"},{"line_number":37,"context_line":"            _, cls.project \u003d cls.v3_client.create_project("},{"line_number":38,"context_line":"                rand_name(\u0027project-\u0027), description\u003drand_name(\u0027project-desc-\u0027))"},{"line_number":39,"context_line":"            cls.projects.append(cls.project[\u0027id\u0027])"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        _, cls.user_body \u003d cls.v3_client.create_user("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2FTHg%3D","line":38,"in_reply_to":"AAAAQn%2F%2Fnro%3D","updated":"2013-08-02 10:35:23.000000000","message":"Done","commit_id":"5ab3944cccb7d40641b39070875daea8a82a0bc8"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"f156c07f99fb9b023435c1066943f2dd25a77796","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        u_password \u003d rand_name(\u0027pass-\u0027)"},{"line_number":36,"context_line":"        for i in range(2):"},{"line_number":37,"context_line":"            _, cls.project \u003d cls.v3_client.create_project("},{"line_number":38,"context_line":"                rand_name(\u0027project-\u0027), description\u003drand_name(\u0027project-desc-\u0027))"},{"line_number":39,"context_line":"            cls.projects.append(cls.project[\u0027id\u0027])"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        _, cls.user_body \u003d cls.v3_client.create_user("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2FniY%3D","line":38,"in_reply_to":"AAAAQn%2F%2Fnro%3D","updated":"2013-07-26 12:40:52.000000000","message":"They throw an exception if the response code is \u003e\u003d400 anyway.","commit_id":"5ab3944cccb7d40641b39070875daea8a82a0bc8"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"ac7ad18ddf0fb982a100d0547d489af4770261cb","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        u_password \u003d rand_name(\u0027pass-\u0027)"},{"line_number":36,"context_line":"        for i in range(2):"},{"line_number":37,"context_line":"            _, cls.project \u003d cls.v3_client.create_project("},{"line_number":38,"context_line":"                rand_name(\u0027project-\u0027), description\u003drand_name(\u0027project-desc-\u0027))"},{"line_number":39,"context_line":"            cls.projects.append(cls.project[\u0027id\u0027])"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        _, cls.user_body \u003d cls.v3_client.create_user("}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2Fnro%3D","line":38,"in_reply_to":"AAAAQn%2F%2FrBY%3D","updated":"2013-07-26 11:38:55.000000000","message":"honestly, we should guaruntee that we setup correctly, so I\u0027d rather have those checked for success.","commit_id":"5ab3944cccb7d40641b39070875daea8a82a0bc8"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"ac7ad18ddf0fb982a100d0547d489af4770261cb","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        resp, get_body \u003d self.creds_client.get_credential(cred[\u0027id\u0027])"},{"line_number":81,"context_line":"        self.assertEqual(resp[\u0027status\u0027], \u0027200\u0027)"},{"line_number":82,"context_line":"        for value1 in self.creds_list[0]:"},{"line_number":83,"context_line":"            self.assertEqual(update_body[value1],"},{"line_number":84,"context_line":"                             get_body[value1])"},{"line_number":85,"context_line":"        for value2 in self.creds_list[1]:"},{"line_number":86,"context_line":"            self.assertEqual(update_body[\u0027blob\u0027][value2],"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2Fnrk%3D","line":83,"updated":"2013-07-26 11:38:55.000000000","message":"really need to be wrapped?","commit_id":"5ab3944cccb7d40641b39070875daea8a82a0bc8"},{"author":{"_account_id":6455,"name":"Harika Vakadi","email":"harika_vakadi@persistent.co.in","username":"harika-vakadi"},"change_message_id":"6d98bc30886c721c0a6060aed060e10d6b60eb54","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        resp, get_body \u003d self.creds_client.get_credential(cred[\u0027id\u0027])"},{"line_number":81,"context_line":"        self.assertEqual(resp[\u0027status\u0027], \u0027200\u0027)"},{"line_number":82,"context_line":"        for value1 in self.creds_list[0]:"},{"line_number":83,"context_line":"            self.assertEqual(update_body[value1],"},{"line_number":84,"context_line":"                             get_body[value1])"},{"line_number":85,"context_line":"        for value2 in self.creds_list[1]:"},{"line_number":86,"context_line":"            self.assertEqual(update_body[\u0027blob\u0027][value2],"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2Fj44%3D","line":83,"in_reply_to":"AAAAQn%2F%2Fnrk%3D","updated":"2013-07-29 07:01:02.000000000","message":"I have implemented this based on your suggestion in previous patch1.","commit_id":"5ab3944cccb7d40641b39070875daea8a82a0bc8"}]}