)]}'
{"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":"b02cbb8730f3835d52df9246579fa86d7545be66","unresolved":true,"context_lines":[{"line_number":1377,"context_line":"            \"share_group_id\": \"group-1\","},{"line_number":1378,"context_line":"        }"},{"line_number":1379,"context_line":""},{"line_number":1380,"context_line":"        self.mock_object("},{"line_number":1381,"context_line":"            self._driver,"},{"line_number":1382,"context_line":"            \"_get_export_locations\","},{"line_number":1383,"context_line":"            mock.Mock(return_value\u003d[\"export\"]))"}],"source_content_type":"text/x-python","patch_set":1,"id":"cee1ab0f_13c53649","line":1380,"updated":"2026-03-31 21:15:24.000000000","message":"Rather than mocking _get_export_locations, let it run through\nnaturally (like test_create_share at line 285 does). This verifies\nboth rados_command calls: \"fs subvolume create\" AND \"fs subvolume\ngetpath\", both of which should include group_name. Use\nassert_has_calls with full argument dicts including group_name.","commit_id":"33f9e0f8a003adc528ca34dd53365635ce3d31fe"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b02cbb8730f3835d52df9246579fa86d7545be66","unresolved":true,"context_lines":[{"line_number":1384,"context_line":""},{"line_number":1385,"context_line":"        result \u003d self._driver.create_share(None, share)"},{"line_number":1386,"context_line":""},{"line_number":1387,"context_line":"        args, _ \u003d driver.rados_command.call_args"},{"line_number":1388,"context_line":"        argdict \u003d args[2]"},{"line_number":1389,"context_line":""},{"line_number":1390,"context_line":"        self.assertEqual(\"group-1\", argdict[\"group_name\"])"}],"source_content_type":"text/x-python","patch_set":1,"id":"96188eeb_021229cf","line":1387,"updated":"2026-03-31 21:15:24.000000000","message":"Accessing args[2] by positional index is fragile and opaque. The\nexisting test_create_share pattern uses assert_has_calls with\nmock.call(...) which is more readable and validates the full call\nsignature.","commit_id":"33f9e0f8a003adc528ca34dd53365635ce3d31fe"}]}