)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"e7875ce6a3d6171f98a1c65c687743a17428f12c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use assertEqual() instead of assertDictEqual()"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In unittest2, we have:"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"    def assertEqual(self, first, second, msg\u003dNone):"},{"line_number":12,"context_line":"        assertion_func \u003d self._getAssertEqualityFunc(first, second)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5ac665da_45c14a2e","line":9,"updated":"2016-08-13 16:39:44.000000000","message":"I think all of this code in a commit message is super excessive.","commit_id":"53a5bacb157606145d540483d773f4711dcf3220"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"e7875ce6a3d6171f98a1c65c687743a17428f12c","unresolved":false,"context_lines":[{"line_number":55,"context_line":""},{"line_number":56,"context_line":"So we can simply use assertEqual() instead of assertDictEqual()."},{"line_number":57,"context_line":"This will unify the unit test code, and will also simplify the"},{"line_number":58,"context_line":"development of unit tests."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Change-Id: Ifdf2b89555f41a2cb069a7d7048e5a284f460713"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5ac665da_a5dc7e19","line":58,"updated":"2016-08-13 16:39:44.000000000","message":"If the only argument for this patch is that it\u0027s easier to read then I\u0027m not sure I agree. I don\u0027t mind being more expressive here. Pythonistas have a saying about being explicit.","commit_id":"53a5bacb157606145d540483d773f4711dcf3220"}]}