)]}'
{"tempest/api/identity/admin/v2/test_tokens.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"fb6dbbb37a27113d490e73e11b0f6e498608bbe3","unresolved":false,"context_lines":[{"line_number":100,"context_line":""},{"line_number":101,"context_line":"        self.assertEqual(user[\u0027id\u0027], token_body[\u0027user\u0027][\u0027id\u0027])"},{"line_number":102,"context_line":"        self.assertEqual(user[\u0027name\u0027], token_body[\u0027user\u0027][\u0027name\u0027])"},{"line_number":103,"context_line":"        self.assertIsInstance(orig_issued_at, unicode)"},{"line_number":104,"context_line":"        self.assertIsInstance(orig_expires, unicode)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        # Validate unscoped token using GET, timestamps should match."}],"source_content_type":"text/x-python","patch_set":2,"id":"dac4157f_14b549cc","line":103,"updated":"2015-04-20 17:38:34.000000000","message":"This is a bit weird. What are you trying exactly to assert here ? I know we are not there yet but this isn\u0027t  going to work with Py3.","commit_id":"da10ac6875e85779a1414dd46bde212c13184bb0"},{"author":{"_account_id":14345,"name":"Filip Hubík","email":"fhubik@redhat.com","username":"fhubik"},"change_message_id":"f8191f401470cacfab7c34a8e5bf7d48deec2d77","unresolved":false,"context_lines":[{"line_number":100,"context_line":""},{"line_number":101,"context_line":"        self.assertEqual(user[\u0027id\u0027], token_body[\u0027user\u0027][\u0027id\u0027])"},{"line_number":102,"context_line":"        self.assertEqual(user[\u0027name\u0027], token_body[\u0027user\u0027][\u0027name\u0027])"},{"line_number":103,"context_line":"        self.assertIsInstance(orig_issued_at, unicode)"},{"line_number":104,"context_line":"        self.assertIsInstance(orig_expires, unicode)"},{"line_number":105,"context_line":""},{"line_number":106,"context_line":"        # Validate unscoped token using GET, timestamps should match."}],"source_content_type":"text/x-python","patch_set":2,"id":"dac4157f_9445b5b4","line":103,"in_reply_to":"dac4157f_14b549cc","updated":"2015-04-21 14:27:53.000000000","message":"I am trying to check if the timestamp is unicode object in first time, likewise done in https://github.com/openstack/tempest/blob/master/tempest/api/identity/admin/v3/test_tokens.py#L100.","commit_id":"da10ac6875e85779a1414dd46bde212c13184bb0"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"fb6dbbb37a27113d490e73e11b0f6e498608bbe3","unresolved":false,"context_lines":[{"line_number":107,"context_line":"        token_body \u003d self.client.get_token(token_id)"},{"line_number":108,"context_line":"        token_details \u003d token_body[\u0027token\u0027]"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"        self.assertEqual(user[\u0027id\u0027], token_body[\u0027user\u0027][\u0027id\u0027])"},{"line_number":111,"context_line":"        self.assertEqual(user[\u0027name\u0027], token_body[\u0027user\u0027][\u0027name\u0027])"},{"line_number":112,"context_line":"        self.assertIsInstance(orig_issued_at, unicode)"},{"line_number":113,"context_line":"        self.assertIsInstance(orig_expires, unicode)"}],"source_content_type":"text/x-python","patch_set":2,"id":"dac4157f_b449fdd2","line":110,"updated":"2015-04-20 17:38:34.000000000","message":"You have a lot of duplicated code. You should define a helper function that does the two assertEqual and the two assertIsInstance. Then call this new function instead of duplicating these 4 lines.","commit_id":"da10ac6875e85779a1414dd46bde212c13184bb0"}]}