)]}'
{"cinderclient/tests/unit/test_shell.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"de33e4148ef3821e344bdb9f1d55b995062a37bf","unresolved":false,"context_lines":[{"line_number":219,"context_line":"    def test_http_client_with_cert(self, mock_session):"},{"line_number":220,"context_line":"        _shell \u003d shell.OpenStackCinderShell()"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"        # RuntimeError crashs the command avec Session instantiation"},{"line_number":223,"context_line":"        args \u003d \u0027--os-cert\u0027, \u0027minnie\u0027, \u0027list\u0027"},{"line_number":224,"context_line":"        self.assertRaises(RuntimeError, _shell.main, args)"},{"line_number":225,"context_line":"        mock_session.assert_called_once_with(cert\u003d\u0027minnie\u0027, verify\u003dmock.ANY)"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_29808a8a","line":222,"updated":"2016-05-17 12:43:51.000000000","message":"nit: It would be good to have the whole comment in English, and a mention to why we are raising the exception (because we don\u0027t care about the execution of the rest of the method) would be ideal.","commit_id":"a43ee301cc4ffda8c73e4e87e93ee430ea198c8e"},{"author":{"_account_id":8124,"name":"cbrandily","email":"zzelle@gmail.com","username":"cbrandily"},"change_message_id":"bd948806ea80c8250574181c62a0c74e4da8da3a","unresolved":false,"context_lines":[{"line_number":219,"context_line":"    def test_http_client_with_cert(self, mock_session):"},{"line_number":220,"context_line":"        _shell \u003d shell.OpenStackCinderShell()"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"        # RuntimeError crashs the command avec Session instantiation"},{"line_number":223,"context_line":"        args \u003d \u0027--os-cert\u0027, \u0027minnie\u0027, \u0027list\u0027"},{"line_number":224,"context_line":"        self.assertRaises(RuntimeError, _shell.main, args)"},{"line_number":225,"context_line":"        mock_session.assert_called_once_with(cert\u003d\u0027minnie\u0027, verify\u003dmock.ANY)"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_bfacaced","line":222,"in_reply_to":"dab17558_29808a8a","updated":"2016-05-17 13:02:13.000000000","message":"Done","commit_id":"a43ee301cc4ffda8c73e4e87e93ee430ea198c8e"},{"author":{"_account_id":8124,"name":"cbrandily","email":"zzelle@gmail.com","username":"cbrandily"},"change_message_id":"4758c7b858228e97424faf8217fa82f3792aa165","unresolved":false,"context_lines":[{"line_number":219,"context_line":"    def test_http_client_with_cert(self, mock_session):"},{"line_number":220,"context_line":"        _shell \u003d shell.OpenStackCinderShell()"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"        # RuntimeError crashs the command avec Session instantiation"},{"line_number":223,"context_line":"        args \u003d \u0027--os-cert\u0027, \u0027minnie\u0027, \u0027list\u0027"},{"line_number":224,"context_line":"        self.assertRaises(RuntimeError, _shell.main, args)"},{"line_number":225,"context_line":"        mock_session.assert_called_once_with(cert\u003d\u0027minnie\u0027, verify\u003dmock.ANY)"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_840d1d91","line":222,"in_reply_to":"dab17558_29808a8a","updated":"2016-05-17 12:55:50.000000000","message":"oups","commit_id":"a43ee301cc4ffda8c73e4e87e93ee430ea198c8e"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"de33e4148ef3821e344bdb9f1d55b995062a37bf","unresolved":false,"context_lines":[{"line_number":229,"context_line":"    def test_http_client_with_cert_and_key(self, mock_session):"},{"line_number":230,"context_line":"        _shell \u003d shell.OpenStackCinderShell()"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        # RuntimeError crashs the command avec Session instantiation"},{"line_number":233,"context_line":"        args \u003d \u0027--os-cert\u0027, \u0027minnie\u0027, \u0027--os-key\u0027, \u0027mickey\u0027, \u0027list\u0027"},{"line_number":234,"context_line":"        self.assertRaises(RuntimeError, _shell.main, args)"},{"line_number":235,"context_line":"        mock_session.assert_called_once_with(cert\u003d(\u0027minnie\u0027, \u0027mickey\u0027),"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_694c529e","line":232,"updated":"2016-05-17 12:43:51.000000000","message":"ditto","commit_id":"a43ee301cc4ffda8c73e4e87e93ee430ea198c8e"},{"author":{"_account_id":8124,"name":"cbrandily","email":"zzelle@gmail.com","username":"cbrandily"},"change_message_id":"bd948806ea80c8250574181c62a0c74e4da8da3a","unresolved":false,"context_lines":[{"line_number":229,"context_line":"    def test_http_client_with_cert_and_key(self, mock_session):"},{"line_number":230,"context_line":"        _shell \u003d shell.OpenStackCinderShell()"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"        # RuntimeError crashs the command avec Session instantiation"},{"line_number":233,"context_line":"        args \u003d \u0027--os-cert\u0027, \u0027minnie\u0027, \u0027--os-key\u0027, \u0027mickey\u0027, \u0027list\u0027"},{"line_number":234,"context_line":"        self.assertRaises(RuntimeError, _shell.main, args)"},{"line_number":235,"context_line":"        mock_session.assert_called_once_with(cert\u003d(\u0027minnie\u0027, \u0027mickey\u0027),"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_ffb2b447","line":232,"in_reply_to":"dab17558_694c529e","updated":"2016-05-17 13:02:13.000000000","message":"Done","commit_id":"a43ee301cc4ffda8c73e4e87e93ee430ea198c8e"}]}
