)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"84a6163ffd309f63bc5656d01226ed9fd38045df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2d7c8816_19ce52f0","updated":"2022-03-22 09:08:26.000000000","message":"Thanks Abhishek","commit_id":"77919e15d281e908de7b687e2af927b59d3dc8a8"}],"glance_store/tests/unit/test_cinder_base.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"9d15e6f7f3283dfb28c61e71014ee180c1532e3e","unresolved":true,"context_lines":[{"line_number":127,"context_line":"    def test__check_context(self):"},{"line_number":128,"context_line":"        with mock.patch.object("},{"line_number":129,"context_line":"                cinder.Store, \u0027is_user_overriden\u0027, return_value\u003dTrue):"},{"line_number":130,"context_line":"            self.store._check_context(self.context)"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"    def test__check_context_no_context(self):"},{"line_number":133,"context_line":"        with mock.patch.object("}],"source_content_type":"text/x-python","patch_set":2,"id":"fe2f8e80_48aca005","line":130,"range":{"start_line":130,"start_character":12,"end_line":130,"end_character":51},"updated":"2022-03-22 07:54:27.000000000","message":"no need to assert anything here?","commit_id":"9d08e162ffb9f84346ab8fc2928c9c90aab27d48"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"84a6163ffd309f63bc5656d01226ed9fd38045df","unresolved":false,"context_lines":[{"line_number":127,"context_line":"    def test__check_context(self):"},{"line_number":128,"context_line":"        with mock.patch.object("},{"line_number":129,"context_line":"                cinder.Store, \u0027is_user_overriden\u0027, return_value\u003dTrue):"},{"line_number":130,"context_line":"            self.store._check_context(self.context)"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"    def test__check_context_no_context(self):"},{"line_number":133,"context_line":"        with mock.patch.object("}],"source_content_type":"text/x-python","patch_set":2,"id":"5db2b410_ae5e505b","line":130,"range":{"start_line":130,"start_character":12,"end_line":130,"end_character":51},"in_reply_to":"fe2f8e80_48aca005","updated":"2022-03-22 09:08:26.000000000","message":"The idea is if _check_context returns then we\u0027ve the right context[1] else we will face an exception as seen in other two tests below.\nThe only thing logical to assert here is the ``is_user_overriden`` is called, i will add that assertion in next PS.\n\n[1] https://github.com/openstack/glance_store/blob/master/glance_store/_drivers/cinder.py#L634","commit_id":"9d08e162ffb9f84346ab8fc2928c9c90aab27d48"}]}