)]}'
{"ironic/tests/unit/drivers/modules/drac/test_management.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"33527615f94fdc85d1625ce59b249844fa071b8b","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                mode[\u0027is_next\u0027] \u003d True"},{"line_number":52,"context_line":"        return [test_utils.dict_to_namedtuple(values\u003dmode) for mode in modes]"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    def setUp(self):"},{"line_number":56,"context_line":"        super(DracManagementInternalMethodsTestCase, self).setUp()"},{"line_number":57,"context_line":"        mgr_utils.mock_the_extension_manager(driver\u003d\u0027fake_drac\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_93678224","line":54,"updated":"2017-01-16 10:44:55.000000000","message":"extra new line is making pep8 to fail :-(\n\nhttp://logs.openstack.org/06/420106/1/check/gate-ironic-pep8-ubuntu-xenial/3ab78e8/console.html#_2017-01-13_18_01_39_600113","commit_id":"9e4ca8c17bd515afa55aee7fef495f1e9a3863d5"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"0282a1b9c952128bf0176bc0b35cf53cbe0c16da","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                mode[\u0027is_next\u0027] \u003d True"},{"line_number":52,"context_line":"        return [test_utils.dict_to_namedtuple(values\u003dmode) for mode in modes]"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    def setUp(self):"},{"line_number":56,"context_line":"        super(DracManagementInternalMethodsTestCase, self).setUp()"},{"line_number":57,"context_line":"        mgr_utils.mock_the_extension_manager(driver\u003d\u0027fake_drac\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a3c09a3_05517d62","line":54,"in_reply_to":"7a3c09a3_93678224","updated":"2017-01-16 13:42:27.000000000","message":"Done","commit_id":"9e4ca8c17bd515afa55aee7fef495f1e9a3863d5"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"f67852fdfab509610e3d4e374fda4f6c0796f36a","unresolved":false,"context_lines":[{"line_number":40,"context_line":"                   autospec\u003dTrue)"},{"line_number":41,"context_line":"class DracManagementInternalMethodsTestCase(db_base.DbTestCase):"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def boot_modes(self, *next_modes):"},{"line_number":44,"context_line":"        modes \u003d ["},{"line_number":45,"context_line":"            {\u0027id\u0027: \u0027IPL\u0027, \u0027name\u0027: \u0027BootSeq\u0027,"},{"line_number":46,"context_line":"             \u0027is_current\u0027: True, \u0027is_next\u0027: False},"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a3c09a3_9efbedc0","line":43,"updated":"2017-01-16 19:04:16.000000000","message":"i thought we tended to use _boot_modes() but I don\u0027t think it matter much for tests. A docstring might be useful too but ...\n\nI\u0027m also not fond of *next_modes, especially w/o a docstring describing what is going on, but as long as others can easily understand the code...","commit_id":"bf8870653bddeefbf194c5258e6b1f54ad47cf54"}]}