)]}'
{"cinder/tests/unit/volume/drivers/synology/test_synology_common.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"7b665033791ebc00fe63a25c2a605f6e60efcc24","unresolved":false,"context_lines":[{"line_number":446,"context_line":"                          version)"},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"    @mock.patch.object(common.LOG, \u0027debug\u0027)"},{"line_number":449,"context_line":"    def test_request_auth_error_session_missing(self, _log):"},{"line_number":450,"context_line":"        version \u003d 1"},{"line_number":451,"context_line":""},{"line_number":452,"context_line":"        self.request._start \u003d mock.Mock(return_value\u003d\u0027fake.cgi\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_df9eaa04","line":449,"range":{"start_line":449,"start_character":8,"end_line":449,"end_character":47},"updated":"2019-11-01 16:35:56.000000000","message":"Please, use ddt library here instead of implementing two almost the same tests","commit_id":"12c1018637314ce49b26946c4fd0306401bf55a1"},{"author":{"_account_id":31216,"name":"Chih Yu Wu","email":"chihyuwu@synology.com","username":"chihyu"},"change_message_id":"a88ba1216b1d2778fbec0f55116f986d3b53bf2c","unresolved":false,"context_lines":[{"line_number":446,"context_line":"                          version)"},{"line_number":447,"context_line":""},{"line_number":448,"context_line":"    @mock.patch.object(common.LOG, \u0027debug\u0027)"},{"line_number":449,"context_line":"    def test_request_auth_error_session_missing(self, _log):"},{"line_number":450,"context_line":"        version \u003d 1"},{"line_number":451,"context_line":""},{"line_number":452,"context_line":"        self.request._start \u003d mock.Mock(return_value\u003d\u0027fake.cgi\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_fe91089b","line":449,"range":{"start_line":449,"start_character":8,"end_line":449,"end_character":47},"in_reply_to":"3fa7e38b_df9eaa04","updated":"2019-11-26 06:00:07.000000000","message":"Done","commit_id":"12c1018637314ce49b26946c4fd0306401bf55a1"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"61c440a9fcab24a20960aa6311fe81f771eee29a","unresolved":false,"context_lines":[{"line_number":25,"context_line":"from cryptography.hazmat.primitives.asymmetric import rsa"},{"line_number":26,"context_line":"from ddt import data"},{"line_number":27,"context_line":"from ddt import ddt"},{"line_number":28,"context_line":"import mock"},{"line_number":29,"context_line":"from oslo_utils import units"},{"line_number":30,"context_line":"import requests"},{"line_number":31,"context_line":"from six.moves import http_client"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff570b3c_8cc9be79","line":28,"updated":"2020-05-29 12:42:42.000000000","message":"pep8: C337: Unit tests should use the standard library \"mock\" module, not the third party mock lib.","commit_id":"6329fa013ccfd97c7e504ec04caa230f7d82f27b"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"61c440a9fcab24a20960aa6311fe81f771eee29a","unresolved":false,"context_lines":[{"line_number":25,"context_line":"from cryptography.hazmat.primitives.asymmetric import rsa"},{"line_number":26,"context_line":"from ddt import data"},{"line_number":27,"context_line":"from ddt import ddt"},{"line_number":28,"context_line":"import mock"},{"line_number":29,"context_line":"from oslo_utils import units"},{"line_number":30,"context_line":"import requests"},{"line_number":31,"context_line":"from six.moves import http_client"}],"source_content_type":"text/x-python","patch_set":8,"id":"ff570b3c_6cc42a60","line":28,"updated":"2020-05-29 12:42:42.000000000","message":"pep8: F811 redefinition of unused \u0027mock\u0027 from line 21","commit_id":"6329fa013ccfd97c7e504ec04caa230f7d82f27b"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"ef5b49847c5937b11adf5b9f1c1166f958127206","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from cryptography.hazmat.backends import default_backend"},{"line_number":24,"context_line":"from cryptography.hazmat.primitives.asymmetric import padding"},{"line_number":25,"context_line":"from cryptography.hazmat.primitives.asymmetric import rsa"},{"line_number":26,"context_line":"from ddt import data"},{"line_number":27,"context_line":"from ddt import ddt"},{"line_number":28,"context_line":"from oslo_utils import units"},{"line_number":29,"context_line":"import requests"},{"line_number":30,"context_line":"from six.moves import http_client"}],"source_content_type":"text/x-python","patch_set":9,"id":"ff570b3c_b27dc91d","line":27,"range":{"start_line":26,"start_character":0,"end_line":27,"end_character":19},"updated":"2020-06-02 16:56:56.000000000","message":"According to [1], please import modules, not objects.\n\n[1] https://docs.openstack.org/hacking/latest/user/hacking.html#imports","commit_id":"2761ed7111db6df94a0f835452ec1b67ca50b4e9"}]}