)]}'
{"keystone/tests/unit/identity/test_core.py":[{"author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"change_message_id":"2462ecff1cf49751702b07868f023ce128dd0aa6","unresolved":false,"context_lines":[{"line_number":198,"context_line":"        self.assertIsNotNone(user[\u0027name\u0027])"},{"line_number":199,"context_line":"        self.assertIsNone(user[\u0027password_expires_at\u0027])"},{"line_number":200,"context_line":"        self.assertIsNone(user[\u0027domain_id\u0027])"},{"line_number":201,"context_line":"        self.assertEqual(True, user[\u0027enabled\u0027])"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    def test_shadow_existing_federated_user(self):"},{"line_number":204,"context_line":"        fed_user \u003d unit.new_federated_user_ref()"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_dfb5ae31","side":"PARENT","line":201,"range":{"start_line":201,"start_character":31,"end_line":201,"end_character":45},"updated":"2016-11-24 09:32:12.000000000","message":"I think it\u0027s better to explicitly assert the value is \"True\".","commit_id":"54d683889401571aab0e116cec61ce693f81dcae"},{"author":{"_account_id":13063,"name":"WEI CHEN","email":"dave.jungler@gmail.com","username":"wei.d.chen"},"change_message_id":"2462ecff1cf49751702b07868f023ce128dd0aa6","unresolved":false,"context_lines":[{"line_number":198,"context_line":"        self.assertIsNotNone(user[\u0027name\u0027])"},{"line_number":199,"context_line":"        self.assertIsNone(user[\u0027password_expires_at\u0027])"},{"line_number":200,"context_line":"        self.assertIsNone(user[\u0027domain_id\u0027])"},{"line_number":201,"context_line":"        self.assertTrue(user[\u0027enabled\u0027])"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    def test_shadow_existing_federated_user(self):"},{"line_number":204,"context_line":"        fed_user \u003d unit.new_federated_user_ref()"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a74a57a_1fd48645","line":201,"range":{"start_line":201,"start_character":24,"end_line":201,"end_character":39},"updated":"2016-11-24 09:32:12.000000000","message":"How about the user[\u0027enabled\u0027] is defined as the value other that \"True\", \u00272\u0027 for example? I will check if there is convertion between int and bool before I giving any score on this change.","commit_id":"19ff7cd8f1aeda7c21c6eb1fd57cc2434e3fbd46"}]}