)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"85d50b2e22d18bd9bd574836b8bd5de51a9b9164","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Replace try/except/else with self.assertRaises."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"File Modified: tempest/tests/compute/admin/test_quotas.py."},{"line_number":10,"context_line":"Test Modified: test_create_server_when_memory_quota_is_full"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This commit modifies the test as per below Blueprint requirements."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAN3%2F%2Fw6k%3D","line":9,"updated":"2013-03-13 11:43:40.000000000","message":"you don\u0027t need these lines, we know that from the commit","commit_id":"90d2ad5151cd6cfcf4b68d56c8289aee58381489"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"85d50b2e22d18bd9bd574836b8bd5de51a9b9164","unresolved":false,"context_lines":[{"line_number":9,"context_line":"File Modified: tempest/tests/compute/admin/test_quotas.py."},{"line_number":10,"context_line":"Test Modified: test_create_server_when_memory_quota_is_full"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This commit modifies the test as per below Blueprint requirements."},{"line_number":13,"context_line":"Blueprint update-expected-exception-tests."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I5a4da788fcf8269d87e1d1e5c30bc8fdbef83aec"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAN3%2F%2Fw6c%3D","line":12,"updated":"2013-03-13 11:43:40.000000000","message":"A better commit message is something like:\n\nas per tempest style guidelines convert a try/except block into\nan assertRaises. Because this used the else block, we also add a cleanup function to ensure we don\u0027t break quotas for all the other tests.\n\nPartially implements blueprint update-expected-exception-tests","commit_id":"90d2ad5151cd6cfcf4b68d56c8289aee58381489"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"5a010f26ecf11ebc78c47976500f5f54d8a6d3a7","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Test Modified: test_create_server_when_memory_quota_is_full."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Modifications:"},{"line_number":12,"context_line":"As per new Tempest style guidelines convert \"try/expect/else\" statements into \"assertRaises\"."},{"line_number":13,"context_line":"In order to ensure other tests remain unaffected because of this change, replace \"finally\" with \"addCleanup\" method."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Blueprint update-expected-exception-tests."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAN3%2F%2FwrI%3D","line":12,"updated":"2013-03-13 15:14:11.000000000","message":"please wrap at \u003c 72 columns","commit_id":"d0c71de77f4d04bc8460076002757fe6a22720b8"}]}