)]}'
{"tests/test_base.py":[{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"285772f661c9e04f71ee4d23c783a8f47acc5054","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from glanceclient.common import base"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"class BaseTest(testtools.TestCase):"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    def test_resource_repr(self):"},{"line_number":24,"context_line":"        r \u003d base.Resource(None, dict(foo\u003d\"bar\", baz\u003d\"spam\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F6NM%3D","line":21,"updated":"2013-03-06 21:28:42.000000000","message":"hey,\n\ncould you please invert the class name? TestBase instead of BaseTest?","commit_id":"6f8f345c936171b9ab86eb7fee6522e52f4301ef"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"285772f661c9e04f71ee4d23c783a8f47acc5054","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        # Two resources with no ID: equal if their info is equal"},{"line_number":41,"context_line":"        r1 \u003d base.Resource(None, {\u0027name\u0027: \u0027joe\u0027, \u0027age\u0027: 12})"},{"line_number":42,"context_line":"        r2 \u003d base.Resource(None, {\u0027name\u0027: \u0027joe\u0027, \u0027age\u0027: 12})"},{"line_number":43,"context_line":"        self.assertEqual(r1, r2)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F6M8%3D","line":43,"updated":"2013-03-06 21:28:42.000000000","message":"Could you add a simple test for neq?","commit_id":"6f8f345c936171b9ab86eb7fee6522e52f4301ef"},{"author":{"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},"change_message_id":"22c5cd047bab96bfaa5702ee7f6fa6e1b1d6e297","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright (C) 2013 Yahoo! Inc."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAOX%2F%2F8%2BU%3D","line":1,"updated":"2013-04-04 18:19:45.000000000","message":"There needs to be an Openstack Foundation LLC copyright here.","commit_id":"37611750d0c1145d7956c85353cb864811797638"},{"author":{"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},"change_message_id":"22c5cd047bab96bfaa5702ee7f6fa6e1b1d6e297","unresolved":false,"context_lines":[{"line_number":32,"context_line":"            id \u003d 4"},{"line_number":33,"context_line":"        self.assertEqual(base.getid(TmpObject), 4)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def test_eq(self):"},{"line_number":36,"context_line":"        # Two resources of the same type with the same id: equal"},{"line_number":37,"context_line":"        r1 \u003d base.Resource(None, {\u0027id\u0027: 1, \u0027name\u0027: \u0027hi\u0027})"},{"line_number":38,"context_line":"        r2 \u003d base.Resource(None, {\u0027id\u0027: 1, \u0027name\u0027: \u0027hello\u0027})"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAOX%2F%2F8%2Bc%3D","line":35,"updated":"2013-04-04 18:19:45.000000000","message":"These assertions look good, but when there\u0027s such obvious parallelism they should be split into multiple test functions, four in this case. That way, when an early one fails, the other ones still get checked.","commit_id":"37611750d0c1145d7956c85353cb864811797638"}]}