)]}'
{"neutron/hacking/checks.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"3b9dd2a880689d18a3736e4195e54042b06bfa15","unresolved":false,"context_lines":[{"line_number":252,"context_line":"def check_no_import_mock(logical_line, filename, noqa):"},{"line_number":253,"context_line":"    \"\"\"N347 Test code must not import mock library.*"},{"line_number":254,"context_line":"    \"\"\""},{"line_number":255,"context_line":"    msg \u003d (\"N347 Test code must not import mock library.*\")"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"    if noqa:"},{"line_number":258,"context_line":"        return"}],"source_content_type":"text/x-python","patch_set":6,"id":"1f493fa4_d8ec9ebc","line":255,"updated":"2020-05-01 16:05:50.000000000","message":"Rodolfo - this actually enforces noone does \u0027import mock\u0027\n\nAnd just being a nitpicker I\u0027m going to fix this message to be like the others - I copied it from above and will fix that too.  Seems silly but it\u0027s Friday and the check queue is fast :)","commit_id":"4647b677ea548cc8d6fdcb95258454f2b0fbb05c"}],"neutron/tests/unit/hacking/test_checks.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"d5fe2a9d5066434c1cbfd60b60e2f2eeaa1ae865","unresolved":false,"context_lines":[{"line_number":218,"context_line":"                    pass_line, \"neutron/tests/test_fake.py\", None))))"},{"line_number":219,"context_line":"        for fail_line in fail_lines:"},{"line_number":220,"context_line":"            self.assertEqual("},{"line_number":221,"context_line":"                0, len(list("},{"line_number":222,"context_line":"                    checks.check_no_import_mock("},{"line_number":223,"context_line":"                        fail_line, \"neutron/common/utils.py\", None))))"},{"line_number":224,"context_line":"            self.assertEqual("}],"source_content_type":"text/x-python","patch_set":6,"id":"1f493fa4_72714744","line":221,"updated":"2020-04-30 08:13:24.000000000","message":"I think we can check that this import is not used anywhere, but that\u0027s ok","commit_id":"4647b677ea548cc8d6fdcb95258454f2b0fbb05c"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"ffc7fca2cd364081689ba01227a84bcf5d8fe535","unresolved":false,"context_lines":[{"line_number":211,"context_line":"    def test_check_no_import_mock(self):"},{"line_number":212,"context_line":"        pass_line \u003d \u0027from unittest import mock\u0027"},{"line_number":213,"context_line":"        fail_lines \u003d (\u0027import mock\u0027,"},{"line_number":214,"context_line":"                      \u0027import mock as mock_lib\u0027)"},{"line_number":215,"context_line":"        self.assertEqual("},{"line_number":216,"context_line":"            0, len(list("},{"line_number":217,"context_line":"                checks.check_no_import_mock("}],"source_content_type":"text/x-python","patch_set":7,"id":"ff570b3c_4fe8ad15","line":214,"updated":"2020-05-09 12:33:52.000000000","message":"Might also want to add the format \"from mock import patch\"","commit_id":"8126f888945b0f3ac01dca1da0dc46d1f9ec3679"}]}