)]}'
{"nova/tests/api/ec2/test_cinder_cloud.py":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"10532e88a177e8e390b6823bdb3e3248f27ab2f2","unresolved":false,"context_lines":[{"line_number":18,"context_line":"#    under the License."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"import copy"},{"line_number":21,"context_line":"import fixtures"},{"line_number":22,"context_line":"import tempfile"},{"line_number":23,"context_line":"import uuid"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Feko%3D","line":21,"updated":"2013-01-20 06:48:23.000000000","message":"fixtures isn\u0027t stdlib, so this should go in a new section - add another blank line after \u0027import uuid\u0027, then \u0027import fixtures\u0027","commit_id":"fbc294cc18f9d6cb29f34b1de5194a7b302074d4"}],"nova/tests/network/test_manager.py":[{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"10532e88a177e8e390b6823bdb3e3248f27ab2f2","unresolved":false,"context_lines":[{"line_number":20,"context_line":"import tempfile"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"import mox"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"from nova import context"},{"line_number":26,"context_line":"from nova import db"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Fek4%3D","line":23,"updated":"2013-01-20 06:48:23.000000000","message":"This is uncessary vertical whitespace - the PEP on imports says one line between groups, and three groups - stdlib, third party, local. So - fixtures should be added just before mox, not up top, and the number and placement of blank lines should not change.","commit_id":"fbc294cc18f9d6cb29f34b1de5194a7b302074d4"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"10532e88a177e8e390b6823bdb3e3248f27ab2f2","unresolved":false,"context_lines":[{"line_number":153,"context_line":"                                              is_admin\u003dFalse)"},{"line_number":154,"context_line":""},{"line_number":155,"context_line":"    def tearDown(self):"},{"line_number":156,"context_line":"        shutil.rmtree(self.tempdir)"},{"line_number":157,"context_line":"        super(FlatNetworkTestCase, self).tearDown()"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"    def test_get_instance_nw_info(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Fekw%3D","line":156,"updated":"2013-01-20 06:48:23.000000000","message":"You don\u0027t need this rmtree anymore, the fixture will delete it.","commit_id":"fbc294cc18f9d6cb29f34b1de5194a7b302074d4"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"10532e88a177e8e390b6823bdb3e3248f27ab2f2","unresolved":false,"context_lines":[{"line_number":1640,"context_line":"            is_admin\u003dFalse)"},{"line_number":1641,"context_line":""},{"line_number":1642,"context_line":"    def tearDown(self):"},{"line_number":1643,"context_line":"        shutil.rmtree(self.tempdir)"},{"line_number":1644,"context_line":"        super(FloatingIPTestCase, self).tearDown()"},{"line_number":1645,"context_line":""},{"line_number":1646,"context_line":"    def test_disassociate_floating_ip_multi_host_calls(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Fek0%3D","line":1643,"updated":"2013-01-20 06:48:23.000000000","message":"You don\u0027t need the rmtree, the fixture will cleanup after itself.","commit_id":"fbc294cc18f9d6cb29f34b1de5194a7b302074d4"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"10532e88a177e8e390b6823bdb3e3248f27ab2f2","unresolved":false,"context_lines":[{"line_number":2139,"context_line":"            is_admin\u003dFalse)"},{"line_number":2140,"context_line":""},{"line_number":2141,"context_line":"    def tearDown(self):"},{"line_number":2142,"context_line":"        shutil.rmtree(self.tempdir)"},{"line_number":2143,"context_line":"        super(InstanceDNSTestCase, self).tearDown()"},{"line_number":2144,"context_line":""},{"line_number":2145,"context_line":"    def test_dns_domains_private(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Feks%3D","line":2142,"updated":"2013-01-20 06:48:23.000000000","message":"Ditto.","commit_id":"fbc294cc18f9d6cb29f34b1de5194a7b302074d4"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"be6c094718a02f92c5105f5801a3bd8e15e35e19","unresolved":false,"context_lines":[{"line_number":152,"context_line":"                                              is_admin\u003dFalse)"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"    def tearDown(self):"},{"line_number":155,"context_line":"        super(FlatNetworkTestCase, self).tearDown()"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def test_get_instance_nw_info(self):"},{"line_number":158,"context_line":"        fake_get_instance_nw_info \u003d fake_network.fake_get_instance_nw_info"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Feag%3D","line":155,"updated":"2013-01-20 22:27:14.000000000","message":"And this is now an empty method that can just be deleted - no more tearDown \\o/. Sorry for not noticing that earlier.","commit_id":"73e7403fea906b0cf27416dabc71bf483157bb10"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"be6c094718a02f92c5105f5801a3bd8e15e35e19","unresolved":false,"context_lines":[{"line_number":1638,"context_line":"            is_admin\u003dFalse)"},{"line_number":1639,"context_line":""},{"line_number":1640,"context_line":"    def tearDown(self):"},{"line_number":1641,"context_line":"        super(FloatingIPTestCase, self).tearDown()"},{"line_number":1642,"context_line":""},{"line_number":1643,"context_line":"    def test_disassociate_floating_ip_multi_host_calls(self):"},{"line_number":1644,"context_line":"        floating_ip \u003d {"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2Feac%3D","line":1641,"updated":"2013-01-20 22:27:14.000000000","message":"Ditto.","commit_id":"73e7403fea906b0cf27416dabc71bf483157bb10"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"be6c094718a02f92c5105f5801a3bd8e15e35e19","unresolved":false,"context_lines":[{"line_number":2136,"context_line":"            is_admin\u003dFalse)"},{"line_number":2137,"context_line":""},{"line_number":2138,"context_line":"    def tearDown(self):"},{"line_number":2139,"context_line":"        super(InstanceDNSTestCase, self).tearDown()"},{"line_number":2140,"context_line":""},{"line_number":2141,"context_line":"    def test_dns_domains_private(self):"},{"line_number":2142,"context_line":"        zone1 \u003d \u0027testzone\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAM3%2F%2FeaU%3D","line":2139,"updated":"2013-01-20 22:27:14.000000000","message":"Ditto.","commit_id":"73e7403fea906b0cf27416dabc71bf483157bb10"}]}