)]}'
{"quantum/tests/unit/test_db_plugin.py":[{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"70d1154ecc241da84c83f240653fa9a540fa8290","unresolved":false,"context_lines":[{"line_number":2300,"context_line":"            for k in keys:"},{"line_number":2301,"context_line":"                self.assertIn(k, subnet[\u0027subnet\u0027])"},{"line_number":2302,"context_line":"                if k \u003d\u003d \u0027host_routes\u0027:"},{"line_number":2303,"context_line":"                    self.assertEqual(sorted(subnet[\u0027subnet\u0027][k]),"},{"line_number":2304,"context_line":"                                     sorted(keys[k]))"},{"line_number":2305,"context_line":"                else:"},{"line_number":2306,"context_line":"                    self.assertEqual(subnet[\u0027subnet\u0027][k], keys[k])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOn%2F%2FX%2Fk%3D","line":2303,"updated":"2013-05-13 07:33:20.000000000","message":"in fact, we have allocation_pools, dns_nameservers which are all list, so can we use \u0027 if isinstance(xxx, list):\n    ....\u0027\n?","commit_id":"388f111da6af33e9f9140a3356133a571a733dc7"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"7ab1b3213e22dbe579e9641678e9796333975022","unresolved":false,"context_lines":[{"line_number":2300,"context_line":"            for k in keys:"},{"line_number":2301,"context_line":"                self.assertIn(k, subnet[\u0027subnet\u0027])"},{"line_number":2302,"context_line":"                if k \u003d\u003d \u0027host_routes\u0027:"},{"line_number":2303,"context_line":"                    self.assertEqual(sorted(subnet[\u0027subnet\u0027][k]),"},{"line_number":2304,"context_line":"                                     sorted(keys[k]))"},{"line_number":2305,"context_line":"                else:"},{"line_number":2306,"context_line":"                    self.assertEqual(subnet[\u0027subnet\u0027][k], keys[k])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAOn%2F%2FX9g%3D","line":2303,"in_reply_to":"AAAAOn%2F%2FX%2Fk%3D","updated":"2013-05-13 07:58:49.000000000","message":"Yes, we can. thanks.","commit_id":"388f111da6af33e9f9140a3356133a571a733dc7"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"unresolved":false,"context_lines":[{"line_number":2308,"context_line":"            if expected:"},{"line_number":2309,"context_line":"                for k in expected:"},{"line_number":2310,"context_line":"                    self.assertIn(k, subnet[\u0027subnet\u0027])"},{"line_number":2311,"context_line":"                    self.assertEqual(subnet[\u0027subnet\u0027][k], expected[k])"},{"line_number":2312,"context_line":"            return subnet"},{"line_number":2313,"context_line":""},{"line_number":2314,"context_line":"    def test_create_subnet(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAOn%2F%2FX0w%3D","line":2311,"updated":"2013-05-13 10:07:55.000000000","message":"what about this line? I think we should deal it with the same way.","commit_id":"5dd20fcc65bf5f52ce0546183326a19992c60e9a"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"7ab1b3213e22dbe579e9641678e9796333975022","unresolved":false,"context_lines":[{"line_number":2308,"context_line":"            if expected:"},{"line_number":2309,"context_line":"                for k in expected:"},{"line_number":2310,"context_line":"                    self.assertIn(k, subnet[\u0027subnet\u0027])"},{"line_number":2311,"context_line":"                    self.assertEqual(subnet[\u0027subnet\u0027][k], expected[k])"},{"line_number":2312,"context_line":"            return subnet"},{"line_number":2313,"context_line":""},{"line_number":2314,"context_line":"    def test_create_subnet(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAOn%2F%2FXyY%3D","line":2311,"in_reply_to":"AAAAOn%2F%2FX0w%3D","updated":"2013-05-13 10:39:21.000000000","message":"you are right, I missed that","commit_id":"5dd20fcc65bf5f52ce0546183326a19992c60e9a"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"unresolved":false,"context_lines":[{"line_number":2300,"context_line":"            for k in keys:"},{"line_number":2301,"context_line":"                self.assertIn(k, subnet[\u0027subnet\u0027])"},{"line_number":2302,"context_line":"                if isinstance(keys[k], list):"},{"line_number":2303,"context_line":"                    self.assertEqual(sorted(subnet[\u0027subnet\u0027][k]),"},{"line_number":2304,"context_line":"                                     sorted(keys[k]))"},{"line_number":2305,"context_line":"                else:"},{"line_number":2306,"context_line":"                    self.assertEqual(subnet[\u0027subnet\u0027][k], keys[k])"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAOn%2F%2FXWk%3D","line":2303,"updated":"2013-05-13 15:11:41.000000000","message":"FYI... self.assertItemsEqual() would be the preferred way to do this in the future.  It\u0027s not necessary to revise this change as it sorts before comparing.","commit_id":"8aaf0bd736839a1c562d60a0a1fdd825e6c553b0"}]}