)]}'
{"ironic/drivers/modules/agent_client.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"68b7a469646cac3c96efa07cebc24f053cabf008","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    \"\"\"Client for interacting with nodes via a REST API.\"\"\""},{"line_number":38,"context_line":"    def __init__(self):"},{"line_number":39,"context_line":"        self.session \u003d requests.Session()"},{"line_number":40,"context_line":"        self.session.headers.update({\u0027Content-Type\u0027: \u0027application/json\u0027})"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    def _get_command_url(self, node):"},{"line_number":43,"context_line":"        agent_url \u003d node.driver_internal_info.get(\u0027agent_url\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7aaa499b_91a60ac0","line":40,"updated":"2015-04-13 10:30:18.000000000","message":"nit:\n\n self.session.headers[\u0027Content-Type\u0027] \u003d \u0027application/json\u0027","commit_id":"af090e97559220e948978c499606ae17046c6d6f"}],"ironic/tests/drivers/test_agent_client.py":[{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"bfecf0dfd66e7970c2f9354643fe80c69892268e","unresolved":false,"context_lines":[{"line_number":50,"context_line":"            \u0027instance_info\u0027: self.instance_info"},{"line_number":51,"context_line":"        }"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"class TestAgentClient(base.TestCase):"},{"line_number":55,"context_line":"    def setUp(self):"},{"line_number":56,"context_line":"        super(TestAgentClient, self).setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa53dc9_921b1473","line":53,"updated":"2015-04-10 09:02:55.000000000","message":"Can we have a test case which asserts that we have {\u0027Content-Type\u0027: \u0027application/json\u0027} in self.client.session.headers.  May be in setUp(), something like\n\n  self.client.session \u003d mock.Mock(autospec\u003drequests.Session, headers\u003d{})\n\nAnd then have  test_init() which asserts value of self.client.session.headers","commit_id":"ecb94e5f7a848b0ff22b7c18bc9bfd6aa90c3536"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"d157d116dcc89efb1b35816cbd231249267c810c","unresolved":false,"context_lines":[{"line_number":50,"context_line":"            \u0027instance_info\u0027: self.instance_info"},{"line_number":51,"context_line":"        }"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"class TestAgentClient(base.TestCase):"},{"line_number":55,"context_line":"    def setUp(self):"},{"line_number":56,"context_line":"        super(TestAgentClient, self).setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa53dc9_1ca8368c","line":53,"in_reply_to":"9aa53dc9_921b1473","updated":"2015-04-10 10:34:34.000000000","message":"Not sure how to do it the way you\u0027re suggesting as session is mocked after content-type is added in client\u0027s init, so added a new test case with actual session creation.","commit_id":"ecb94e5f7a848b0ff22b7c18bc9bfd6aa90c3536"}]}