)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"997a6f67f4bb9b17a26f4580a723bf694245ff4c","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add unit test for delete share clone"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug #1989472"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I7514f1ec18e5a91883ba4d033805c137e0df2dba"},{"line_number":12,"context_line":"Signed-off-by: olamidepeterojo \u003cpeterojoolamide@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"76ceebd1_6714b8fe","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":6},"updated":"2026-04-06 21:34:15.000000000","message":"Same thing as the others, if all test coverage gaps haven\u0027t been closed with this, this should be \"Partial-Bug\"","commit_id":"a4bd0ef29d23dae24fe4ad3e243ab58e210a0ade"}],"manila/tests/share/drivers/cephfs/test_driver.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"4bc89de5ca8357f02e9a5ce50ce4995265ce9a9f","unresolved":true,"context_lines":[{"line_number":1339,"context_line":"            self._driver, \"_need_to_cancel_clone\", return_value\u003dTrue"},{"line_number":1340,"context_line":"        ):"},{"line_number":1341,"context_line":""},{"line_number":1342,"context_line":"            self._driver.delete_share(None, share)"},{"line_number":1343,"context_line":""},{"line_number":1344,"context_line":"            args, _ \u003d mock_rados.call_args"},{"line_number":1345,"context_line":"            argdict \u003d args[2]"}],"source_content_type":"text/x-python","patch_set":6,"id":"ad066a4e_cd8f5f89","line":1342,"updated":"2026-03-31 21:15:22.000000000","message":"Consider asserting mock_rados.call_count (expected: 2) to confirm\nboth clone cancel and subvolume rm were invoked.","commit_id":"b4e6751f28da6dc7b5c4c2728ac83cfa890870fb"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"4bc89de5ca8357f02e9a5ce50ce4995265ce9a9f","unresolved":true,"context_lines":[{"line_number":1341,"context_line":""},{"line_number":1342,"context_line":"            self._driver.delete_share(None, share)"},{"line_number":1343,"context_line":""},{"line_number":1344,"context_line":"            args, _ \u003d mock_rados.call_args"},{"line_number":1345,"context_line":"            argdict \u003d args[2]"},{"line_number":1346,"context_line":""},{"line_number":1347,"context_line":"            self.assertEqual(\"group-1\", argdict[\"group_name\"])"}],"source_content_type":"text/x-python","patch_set":6,"id":"37fc178d_4428d1f8","line":1344,"updated":"2026-03-31 21:15:22.000000000","message":"mock_rados.call_args returns the arguments of the last call to\nrados_command, which is the \"fs subvolume rm\" call (line 822),\nnot the \"fs clone cancel\" call (line 806) that the test name and\ncommit message claim to cover. Use call_args_list to inspect the\nfirst call instead.","commit_id":"b4e6751f28da6dc7b5c4c2728ac83cfa890870fb"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"4bc89de5ca8357f02e9a5ce50ce4995265ce9a9f","unresolved":true,"context_lines":[{"line_number":1344,"context_line":"            args, _ \u003d mock_rados.call_args"},{"line_number":1345,"context_line":"            argdict \u003d args[2]"},{"line_number":1346,"context_line":""},{"line_number":1347,"context_line":"            self.assertEqual(\"group-1\", argdict[\"group_name\"])"},{"line_number":1348,"context_line":""},{"line_number":1349,"context_line":""},{"line_number":1350,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":6,"id":"e0c7d3dc_49c72b68","line":1347,"updated":"2026-03-31 21:15:22.000000000","message":"The test only asserts group_name but does not verify other arguments\n(vol_name, clone_name, force). Adding complete argdict assertions\nwould make the test more robust.","commit_id":"b4e6751f28da6dc7b5c4c2728ac83cfa890870fb"}]}