)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"a968b5491f928c339117214240ede070f58b0fdf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d4a8f8c7_88144e8e","updated":"2025-10-11 19:01:17.000000000","message":"I don\u0027t understand - Why did you copy \u0026 paste https://review.opendev.org/c/openstack/i18n/+/955982/5/tools/weblate/weblate_tests/README.rst and submit this patch?","commit_id":"3f8e7eabf00b7de418fbe288978dbad9a450fd87"},{"author":{"_account_id":23279,"name":"Seongsoo Cho","display_name":"Seongsoo Cho","email":"ppiyakk2@printf.kr","username":"seongsoo.cho"},"change_message_id":"1e9c8d5b74bb38a62957773b083d22820d16a70c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d3ff4eb3_b1d22885","updated":"2025-10-13 08:41:43.000000000","message":"Please describe in more detail what this patch does.","commit_id":"3f8e7eabf00b7de418fbe288978dbad9a450fd87"},{"author":{"_account_id":23279,"name":"Seongsoo Cho","display_name":"Seongsoo Cho","email":"ppiyakk2@printf.kr","username":"seongsoo.cho"},"change_message_id":"a30d7910a82ec7b3f47c0d6e80643e11747fafa2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f29777ab_8f45fc1b","in_reply_to":"2eda4b97_98cc7b02","updated":"2025-10-14 10:15:10.000000000","message":"In my opinion, you don’t need to separate the files from the existing patch.\n\nMy guess is that you wanted to separate them because both of the following patches include __init__.py.\nHowever, I think the issue will be resolved once either of them is merged.\n\nhttps://review.opendev.org/c/openstack/i18n/+/953745\nhttps://review.opendev.org/c/openstack/i18n/+/953332\n\nMy suggestion is that you leave your review comments for these two patches submitted by the CMU student directly on each patch.","commit_id":"3f8e7eabf00b7de418fbe288978dbad9a450fd87"},{"author":{"_account_id":37971,"name":"Chaemin Lim","display_name":"antraxmin","email":"antraxmin@naver.com","username":"antraxmin"},"change_message_id":"f834749113cb677e62d0580e908fbd029a5ec468","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4846f283_b70bb84d","in_reply_to":"d3ff4eb3_b1d22885","updated":"2025-10-14 09:09:43.000000000","message":"This patch only sets up the minimal test package skeleton under tools/weblate/weblate_tests/.\n\nIt includes __init__.py and README.rst to ensure that the directory is recognized as a Python package and to document the purpose of this test environment. There’s no executable test logic yet — this is just the base infrastructure needed to support future unit tests for Weblate-related modules (such as userinfo, users, and utils).\n\nIt’s part of the work to separate and organize the test framework more cleanly, based on the previous review feedback from https://review.opendev.org/c/openstack/i18n/+/953332","commit_id":"3f8e7eabf00b7de418fbe288978dbad9a450fd87"},{"author":{"_account_id":37971,"name":"Chaemin Lim","display_name":"antraxmin","email":"antraxmin@naver.com","username":"antraxmin"},"change_message_id":"f834749113cb677e62d0580e908fbd029a5ec468","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2eda4b97_98cc7b02","in_reply_to":"d4a8f8c7_88144e8e","updated":"2025-10-14 09:09:43.000000000","message":"Thank you for the comment.\nI believe there may be a small misunderstanding — this patch was not derived from Change #955982\n\nIt continues the unfinished work from https://review.opendev.org/c/openstack/i18n/+/953332, which was originally submitted by CMU students and contained both test infrastructure files and unit tests in a single commit.\nBased on the previous review feedback, I’ve separated the test infrastructure (__init__.py, README.rst) into this new Change (#963703) so that the test framework can be maintained independently and consistently.\n\nThis patch doesn’t introduce new logic — it just restores and organizes the minimal structure needed for future Weblate-related unit tests.\n\nIf I misunderstood the previous review context, please let me know — I’d be happy to adjust the approach accordingly.","commit_id":"3f8e7eabf00b7de418fbe288978dbad9a450fd87"}],"tools/weblate/weblate_tests/README.rst":[{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"a968b5491f928c339117214240ede070f58b0fdf","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"**Core API Communication**: Tests ensure that the tools can properly authenticate with Weblate servers, handle HTTP requests/responses, and manage API errors smoothly. This is critical since translation workflows depend on reliable API connectivity."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"**Configuration Management**: Validates that configuration files (``weblate.ini``) are parsed correctly. Tools configured incorrectly could lead to authentication failures or data corruption."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"**User and Team Management**: Tests verify that user account information is retrieved accurately and that language assignments are handled properly. This ensures translators are properly organized and have correct access rights."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"c377f9be_a9c29e70","line":15,"range":{"start_line":15,"start_character":105,"end_line":15,"end_character":134},"updated":"2025-10-11 19:01:17.000000000","message":"Original word from https://review.opendev.org/c/openstack/i18n/+/955982/5/tools/weblate/weblate_tests/README.rst seems better","commit_id":"3f8e7eabf00b7de418fbe288978dbad9a450fd87"}]}