)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"881aa541163832cfb4b86d1b07aef8d4ea5c75f5","unresolved":false,"context_lines":[{"line_number":11,"context_line":"of find_file(), which is always None."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ia5285d252d5636892c4fbeb9191a6c7ed4923b78"},{"line_number":14,"context_line":"Closes-bug: 1236176"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATH%2F%2F1fs%3D","line":14,"updated":"2013-10-09 07:20:21.000000000","message":"Should be Closes-Bug:\n\nSeems like launchpad isn\u0027t updated otherwise","commit_id":"9dda9371978ff5b3ab7d90923ac2d01d794861c8"},{"author":{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},"change_message_id":"aad92172253dff438190d37f4956538cda41b20c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"of find_file(), which is always None."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ia5285d252d5636892c4fbeb9191a6c7ed4923b78"},{"line_number":14,"context_line":"Closes-bug: 1236176"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATH%2F%2F1a4%3D","line":14,"in_reply_to":"AAAATH%2F%2F1fs%3D","updated":"2013-10-09 07:35:53.000000000","message":"Done","commit_id":"9dda9371978ff5b3ab7d90923ac2d01d794861c8"}],"nova/tests/cells/test_cells_state_manager.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"53c9347bc0629cecdbb9c52e9dd65414719355c4","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        self.stubs.Set(db, \u0027flavor_get_all\u0027, _fake_instance_type_all)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def test_cells_config_not_found(self):"},{"line_number":71,"context_line":"        self.flags(cells_config\u003d\u0027no_such_file_exists.conf\u0027, group\u003d\u0027cells\u0027)"},{"line_number":72,"context_line":"        try:"},{"line_number":73,"context_line":"            state.CellStateManager()"},{"line_number":74,"context_line":"        except cfg.ConfigFilesNotFoundError as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2F%2BSg%3D","line":71,"updated":"2013-10-07 08:22:33.000000000","message":"Thanks!\n\nplease use: self.assertRaises(cfg.ConfigFilesNotFoundError, state.CellStateManager)","commit_id":"f55f7d8064a16e2ba92f992ddbccf5faeb37f5ff"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"29123d3799ce3f69774068b5baf0db606c3178b2","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        self.stubs.Set(db, \u0027flavor_get_all\u0027, _fake_instance_type_all)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def test_cells_config_not_found(self):"},{"line_number":71,"context_line":"        self.flags(cells_config\u003d\u0027no_such_file_exists.conf\u0027, group\u003d\u0027cells\u0027)"},{"line_number":72,"context_line":"        try:"},{"line_number":73,"context_line":"            state.CellStateManager()"},{"line_number":74,"context_line":"        except cfg.ConfigFilesNotFoundError as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2F9Xs%3D","line":71,"in_reply_to":"AAAATH%2F%2F%2BBY%3D","updated":"2013-10-07 13:50:25.000000000","message":"Please see:\nhttp://docs.python.org/2/library/unittest.html#unittest.TestCase.assertRaises\nYou can invoke the assertRaises and then check the exception value afterwards.","commit_id":"f55f7d8064a16e2ba92f992ddbccf5faeb37f5ff"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"d607394cf912afa1a31fd2f841430918a32f6af4","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        self.stubs.Set(db, \u0027flavor_get_all\u0027, _fake_instance_type_all)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def test_cells_config_not_found(self):"},{"line_number":71,"context_line":"        self.flags(cells_config\u003d\u0027no_such_file_exists.conf\u0027, group\u003d\u0027cells\u0027)"},{"line_number":72,"context_line":"        try:"},{"line_number":73,"context_line":"            state.CellStateManager()"},{"line_number":74,"context_line":"        except cfg.ConfigFilesNotFoundError as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2F%2BBY%3D","line":71,"in_reply_to":"AAAATH%2F%2F%2BRY%3D","updated":"2013-10-07 10:16:09.000000000","message":"Not sure","commit_id":"f55f7d8064a16e2ba92f992ddbccf5faeb37f5ff"},{"author":{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},"change_message_id":"4e143c95a6d7a50efcc01024e5f780718d334375","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        self.stubs.Set(db, \u0027flavor_get_all\u0027, _fake_instance_type_all)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def test_cells_config_not_found(self):"},{"line_number":71,"context_line":"        self.flags(cells_config\u003d\u0027no_such_file_exists.conf\u0027, group\u003d\u0027cells\u0027)"},{"line_number":72,"context_line":"        try:"},{"line_number":73,"context_line":"            state.CellStateManager()"},{"line_number":74,"context_line":"        except cfg.ConfigFilesNotFoundError as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2F%2BRY%3D","line":71,"in_reply_to":"AAAATH%2F%2F%2BSg%3D","updated":"2013-10-07 08:27:42.000000000","message":"I want to check e.config_files holds the expected value. Can I do it with assertRaises?","commit_id":"f55f7d8064a16e2ba92f992ddbccf5faeb37f5ff"},{"author":{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},"change_message_id":"9bd8de0c5cd9b5a92856575c130ec6cb1ba8e51e","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        self.stubs.Set(db, \u0027flavor_get_all\u0027, _fake_instance_type_all)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def test_cells_config_not_found(self):"},{"line_number":71,"context_line":"        self.flags(cells_config\u003d\u0027no_such_file_exists.conf\u0027, group\u003d\u0027cells\u0027)"},{"line_number":72,"context_line":"        try:"},{"line_number":73,"context_line":"            state.CellStateManager()"},{"line_number":74,"context_line":"        except cfg.ConfigFilesNotFoundError as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2F6ok%3D","line":71,"in_reply_to":"AAAATH%2F%2F6p8%3D","updated":"2013-10-08 04:20:33.000000000","message":"Oh, testtools\u0027s assertRaises() returns the raised exception. I\u0027ll use it.","commit_id":"f55f7d8064a16e2ba92f992ddbccf5faeb37f5ff"},{"author":{"_account_id":3068,"name":"Arata Notsu","email":"notsu@virtualtech.jp","username":"arata776"},"change_message_id":"4de5e34cc96ca68a8d8e83f1d146823861f04e48","unresolved":false,"context_lines":[{"line_number":68,"context_line":"        self.stubs.Set(db, \u0027flavor_get_all\u0027, _fake_instance_type_all)"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def test_cells_config_not_found(self):"},{"line_number":71,"context_line":"        self.flags(cells_config\u003d\u0027no_such_file_exists.conf\u0027, group\u003d\u0027cells\u0027)"},{"line_number":72,"context_line":"        try:"},{"line_number":73,"context_line":"            state.CellStateManager()"},{"line_number":74,"context_line":"        except cfg.ConfigFilesNotFoundError as e:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATH%2F%2F6p8%3D","line":71,"in_reply_to":"AAAATH%2F%2F9Xs%3D","updated":"2013-10-08 04:08:27.000000000","message":"Thanks! But, assertRaises(exception) is not supported by testtools.TestCase, which is test.TestCase\u0027s parent.","commit_id":"f55f7d8064a16e2ba92f992ddbccf5faeb37f5ff"}]}