)]}'
{"nova/tests/unit/api/openstack/compute/test_security_groups.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"057bc483ebf4beea88fbbadfaac908288b01d0dc","unresolved":false,"context_lines":[{"line_number":388,"context_line":"        self.server_controller \u003d self.server_secgrp_ctl_cls()"},{"line_number":389,"context_line":"        self.manager \u003d self.secgrp_act_ctl_cls()"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":"        # This needs to be done here to set fake_id because the derived"},{"line_number":392,"context_line":"        # class needs to be called first if it wants to set"},{"line_number":393,"context_line":"        # \u0027security_group_api\u0027 and this setUp method needs to be called."},{"line_number":394,"context_line":"        if self.controller.security_group_api.id_is_uuid:"},{"line_number":395,"context_line":"            self.fake_id \u003d \u002711111111-1111-1111-1111-111111111111\u0027"},{"line_number":396,"context_line":"        else:"},{"line_number":397,"context_line":"            self.fake_id \u003d \u002711111111\u0027"},{"line_number":398,"context_line":""},{"line_number":399,"context_line":"        self.req \u003d fakes.HTTPRequest.blank(\u0027\u0027)"},{"line_number":400,"context_line":"        self.admin_req \u003d fakes.HTTPRequest.blank(\u0027\u0027, use_admin_context\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_af18e075","line":397,"range":{"start_line":391,"start_character":0,"end_line":397,"end_character":37},"updated":"2019-12-03 15:35:53.000000000","message":"I drop this in a later change. Probably should have done it here but I clearly missed it\n\n[1] Ia8dbe8ba61ec6d1b8498918a53a103a6eff4d488","commit_id":"59a7bfd48d9714e62d14f44aa3501c73d3e91485"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"eb7f6abd7a771f8bcef1109f2982cc065a9ce8d4","unresolved":false,"context_lines":[{"line_number":94,"context_line":"            \"instances\": [{\u0027id\u0027: 1, \u0027uuid\u0027: UUID_SERVER}]}"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"class MockClient(object):"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    # Needs to be global to survive multiple calls to get_client."},{"line_number":100,"context_line":"    _fake_security_groups \u003d {}"}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_a222c052","line":97,"updated":"2019-12-11 14:14:47.000000000","message":"Obviously not something to hold this up, but this fake client with it\u0027s state tracking is so detailed I wonder if we could drop this and change the tests to just use the NeutronFixture. When this MockClient was written the NeutronFixture was probably pretty basic but it\u0027s become more advanced over the years. Anyway, just a thought for a future rainy day side project.","commit_id":"39bcf6f02db0dc99fe41d97e94fe2f6543a440e4"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"eb7f6abd7a771f8bcef1109f2982cc065a9ce8d4","unresolved":false,"context_lines":[{"line_number":1481,"context_line":"        req.content_type \u003d self.content_type"},{"line_number":1482,"context_line":"        req.headers[\u0027Accept\u0027] \u003d self.content_type"},{"line_number":1483,"context_line":""},{"line_number":1484,"context_line":"        # NOTE: This \u0027os-security-groups\u0027 is for enabling security_groups"},{"line_number":1485,"context_line":"        #       attribute on response body."},{"line_number":1486,"context_line":"        res \u003d req.get_response(fakes.wsgi_app_v21())"},{"line_number":1487,"context_line":"        return res"},{"line_number":1488,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"3fa7e38b_e2baf8de","line":1485,"range":{"start_line":1484,"start_character":8,"end_line":1485,"end_character":43},"updated":"2019-12-11 14:14:47.000000000","message":"This must be vestigial - we don\u0027t have the extensions anymore for controlling the response.","commit_id":"39bcf6f02db0dc99fe41d97e94fe2f6543a440e4"}]}