)]}'
{"patrole_tempest_plugin/tests/unit/test_hacking.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"208d580b981086e54212d98c0017ee46073c5df4","unresolved":false,"context_lines":[{"line_number":273,"context_line":"            \"class FakePluginRbacTest\","},{"line_number":274,"context_line":"            \"./patrole_tempest_plugin/tests/api/fake_test_rbac.py\"))"},{"line_number":275,"context_line":"        self.assertFalse(check("},{"line_number":276,"context_line":"            \"class FakePluginRbacTest(base.BaseFakeRbacTest)\","},{"line_number":277,"context_line":"            \"./patrole_tempest_plugin/tests/api/fake_test_rbac.py\"))"},{"line_number":278,"context_line":""},{"line_number":279,"context_line":"        # Failing cases: these **do not** end in correct test class suffix."}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_aaf80dc1","side":"PARENT","line":276,"range":{"start_line":276,"start_character":12,"end_line":276,"end_character":62},"updated":"2018-11-01 10:40:39.000000000","message":"this case should also fail right, we do not wan to have any class have suffix as PluginRbacTest which is not derived from PluginRbacTest base.\n\nCurrent hacking check does not check this otherway around so how about adding this too.","commit_id":"904a02bef8af64c2a6fb1e0aca67ea9db3f9883f"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"7951dec6a093202897a9c755c19d33134c8df688","unresolved":false,"context_lines":[{"line_number":273,"context_line":"            \"class FakePluginRbacTest\","},{"line_number":274,"context_line":"            \"./patrole_tempest_plugin/tests/api/fake_test_rbac.py\"))"},{"line_number":275,"context_line":"        self.assertFalse(check("},{"line_number":276,"context_line":"            \"class FakePluginRbacTest(base.BaseFakeRbacTest)\","},{"line_number":277,"context_line":"            \"./patrole_tempest_plugin/tests/api/fake_test_rbac.py\"))"},{"line_number":278,"context_line":""},{"line_number":279,"context_line":"        # Failing cases: these **do not** end in correct test class suffix."}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_debbc70d","side":"PARENT","line":276,"range":{"start_line":276,"start_character":12,"end_line":276,"end_character":62},"in_reply_to":"3f79a3b5_aaf80dc1","updated":"2018-11-01 14:06:16.000000000","message":"Yeah, I thought about that. I can add that logic too.","commit_id":"904a02bef8af64c2a6fb1e0aca67ea9db3f9883f"}]}