)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6f093f7a8d55b5a6534627a339e7a4dffc38dbe0","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     judemin \u003cjudemin2087@naver.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-08-20 21:14:09 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"title: Add functional tests for \u0027token issue\u0027"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added following three test cases."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bb519d40_04fc5608","line":7,"updated":"2025-08-20 16:47:59.000000000","message":"For your next commit message, you can drop `title:` from the subject","commit_id":"0708dd97c7413935977ea0f45cee8119127dd394"},{"author":{"_account_id":38255,"name":"SanyYeon Min","display_name":"judemin","email":"judemin2087@naver.com","username":"judemin"},"change_message_id":"f4d947449137c9a986fcd9c5704d91c80266c71e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     judemin \u003cjudemin2087@naver.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2025-08-20 21:14:09 +0900"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"title: Add functional tests for \u0027token issue\u0027"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Added following three test cases."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"07a5d872_13fd8e6a","line":7,"in_reply_to":"bb519d40_04fc5608","updated":"2025-08-30 05:15:30.000000000","message":"Thank you for the feedback, I will be sure to omit it going forward.","commit_id":"0708dd97c7413935977ea0f45cee8119127dd394"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3bd736fb5aec7ca2bf30a4999c4290e85bd000f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b2146fb9_a2959cb9","updated":"2025-08-20 16:28:14.000000000","message":"Thank you for the change. Unfortunately we can\u0027t merge this and this isn\u0027t the kind of thing we\u0027d like to test with functional tests. You will note that many OSC commands provide the same options you are testing here. As a result, we are not really testing something specific to the identity tokens API and its command, but something specific to osc-lib and cliff. We already have unit tests for that, which are sufficient here.\n\nRather than add this kind of test, I\u0027d encourage you to find commands that have no test coverage at all. Avoid modifying anything with global state (hint: if there isn\u0027t a `create` command for the resource, you probably don\u0027t want to modify it. You should not modify `quota` in a function test, for example, or disable and re-enable a service) but otherwise you can pick anything.\n\nPlease ask if you need more guidance.","commit_id":"0708dd97c7413935977ea0f45cee8119127dd394"},{"author":{"_account_id":38255,"name":"SanyYeon Min","display_name":"judemin","email":"judemin2087@naver.com","username":"judemin"},"change_message_id":"f4d947449137c9a986fcd9c5704d91c80266c71e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2ec5f3f4_e7a48b7c","in_reply_to":"b2146fb9_a2959cb9","updated":"2025-08-30 05:15:30.000000000","message":"Thank you for taking the time to provide such clear guidance. I apologize for the misinterpreting the test\u0027s scope. Following your suggestion, I will now focus on identifying commands without test coverage. I look forward to making a more meaningful contribution next time.","commit_id":"0708dd97c7413935977ea0f45cee8119127dd394"}]}