)]}'
{"nova/tests/api/ec2/test_admin.py":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"6f1ad58472dbd781ab91f09b73a49b47fe09c023","unresolved":false,"context_lines":[{"line_number":216,"context_line":"            self.assertIn(inst_name, [i[\u0027name\u0027] for i in insts])"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"    def test_register_user(self):"},{"line_number":219,"context_line":"        registered_user \u003d admin.AdminController().register_user(self._c, \u0027bob\u0027)"},{"line_number":220,"context_line":"        self.assertEqual(\u0027bob\u0027, registered_user[\u0027username\u0027])"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def test_describe_user(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACX%2F%2F%2BVs%3D","line":219,"updated":"2011-09-28 06:42:39.000000000","message":"You construct another AdminController here","commit_id":"06a27165dfd940b505df8c9677157b6bf70c0b2e"},{"author":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"change_message_id":"8eeead00b34ffae4f49d2a7ee45beb55367cf925","unresolved":false,"context_lines":[{"line_number":216,"context_line":"            self.assertIn(inst_name, [i[\u0027name\u0027] for i in insts])"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"    def test_register_user(self):"},{"line_number":219,"context_line":"        registered_user \u003d admin.AdminController().register_user(self._c, \u0027bob\u0027)"},{"line_number":220,"context_line":"        self.assertEqual(\u0027bob\u0027, registered_user[\u0027username\u0027])"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    def test_describe_user(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACX%2F%2F%2BU4%3D","line":219,"in_reply_to":"AAAACX%2F%2F%2BVs%3D","updated":"2011-09-28 10:01:55.000000000","message":"I should have paid more attention when refactoring!","commit_id":"06a27165dfd940b505df8c9677157b6bf70c0b2e"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"6f1ad58472dbd781ab91f09b73a49b47fe09c023","unresolved":false,"context_lines":[{"line_number":395,"context_line":"            pass"},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"        def get_fake_instance_func():"},{"line_number":398,"context_line":"            first_call \u003d [True]"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"            def fake_instance_get_all_by_project(self, *args):"},{"line_number":401,"context_line":"                if first_call[0]:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACX%2F%2F%2BVo%3D","line":398,"updated":"2011-09-28 06:42:39.000000000","message":"Any reason that first_call is an array?","commit_id":"06a27165dfd940b505df8c9677157b6bf70c0b2e"},{"author":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"change_message_id":"8eeead00b34ffae4f49d2a7ee45beb55367cf925","unresolved":false,"context_lines":[{"line_number":395,"context_line":"            pass"},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"        def get_fake_instance_func():"},{"line_number":398,"context_line":"            first_call \u003d [True]"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"            def fake_instance_get_all_by_project(self, *args):"},{"line_number":401,"context_line":"                if first_call[0]:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACX%2F%2F%2BVA%3D","line":398,"in_reply_to":"AAAACX%2F%2F%2BVo%3D","updated":"2011-09-28 10:01:55.000000000","message":"Python\u0027s scoping rules mean if I make first_call a boolean it will give me a UnboundLocalError when I attempt to read it inside fake_instance_get_all_by_project. If there\u0027s a more elegant way around this that\u0027d be great :)","commit_id":"06a27165dfd940b505df8c9677157b6bf70c0b2e"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"6f1ad58472dbd781ab91f09b73a49b47fe09c023","unresolved":false,"context_lines":[{"line_number":450,"context_line":"        self._ac.register_project(self._c, \u0027bobs_project\u0027, \u0027bob\u0027)"},{"line_number":451,"context_line":"        vpns \u003d self._ac.describe_vpns(self._c)"},{"line_number":452,"context_line":""},{"line_number":453,"context_line":"        self.assertIn(\u0027items\u0027, vpns)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACX%2F%2F%2BVk%3D","line":453,"updated":"2011-09-28 06:42:39.000000000","message":"Given the rest of the tests, this looks a bit weak. A few more assertions on the returned vpns list?","commit_id":"06a27165dfd940b505df8c9677157b6bf70c0b2e"},{"author":{"_account_id":1610,"name":"Brendan Maguire","email":"B_Maguire@Dell.com","username":"b-maguire"},"change_message_id":"8eeead00b34ffae4f49d2a7ee45beb55367cf925","unresolved":false,"context_lines":[{"line_number":450,"context_line":"        self._ac.register_project(self._c, \u0027bobs_project\u0027, \u0027bob\u0027)"},{"line_number":451,"context_line":"        vpns \u003d self._ac.describe_vpns(self._c)"},{"line_number":452,"context_line":""},{"line_number":453,"context_line":"        self.assertIn(\u0027items\u0027, vpns)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACX%2F%2F%2BU0%3D","line":453,"in_reply_to":"AAAACX%2F%2F%2BVk%3D","updated":"2011-09-28 10:01:55.000000000","message":"Done","commit_id":"06a27165dfd940b505df8c9677157b6bf70c0b2e"}]}