)]}'
{"quantum/tests/unit/test_db_plugin.py":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"a39e1426c6caad4a23d79bc86fc2b6c48d1bd5de","unresolved":false,"context_lines":[{"line_number":384,"context_line":"        res \u003d self._list(\u0027%ss\u0027 % collection,"},{"line_number":385,"context_line":"                         quantum_context\u003dquantum_context,"},{"line_number":386,"context_line":"                         query_params\u003dquery_params)"},{"line_number":387,"context_line":"        self.assertItemsEqual([i[\u0027id\u0027] for i in res[\u0027%ss\u0027 % collection]],"},{"line_number":388,"context_line":"                              [i[collection][\u0027id\u0027] for i in items])"},{"line_number":389,"context_line":""},{"line_number":390,"context_line":"    @contextlib.contextmanager"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMH%2F%2Fxiw%3D","line":387,"updated":"2012-11-26 10:39:58.000000000","message":"Another assert for element count might be welcome here.\nIn some other modules when we tests lists we also check for element count","commit_id":"b01b837fc633233f9e1819c72017f0def36e9555"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"d0484514f6254e81004a83eccf6cf279b38cc7c4","unresolved":false,"context_lines":[{"line_number":384,"context_line":"        res \u003d self._list(\u0027%ss\u0027 % collection,"},{"line_number":385,"context_line":"                         quantum_context\u003dquantum_context,"},{"line_number":386,"context_line":"                         query_params\u003dquery_params)"},{"line_number":387,"context_line":"        self.assertItemsEqual([i[\u0027id\u0027] for i in res[\u0027%ss\u0027 % collection]],"},{"line_number":388,"context_line":"                              [i[collection][\u0027id\u0027] for i in items])"},{"line_number":389,"context_line":""},{"line_number":390,"context_line":"    @contextlib.contextmanager"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAMH%2F%2FxeU%3D","line":387,"in_reply_to":"AAAAMH%2F%2Fxiw%3D","updated":"2012-11-26 14:01:27.000000000","message":"\u0027assertItemsEqual\u0027 will failed when element count was different. So it cover the case of element count was different.","commit_id":"b01b837fc633233f9e1819c72017f0def36e9555"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"unresolved":false,"context_lines":[{"line_number":379,"context_line":"        self.assertEqual(items[0][\u0027name\u0027], \u0027test_0\u0027)"},{"line_number":380,"context_line":"        self.assertEqual(items[1][\u0027name\u0027], \u0027test_1\u0027)"},{"line_number":381,"context_line":""},{"line_number":382,"context_line":"    def _test_list_collections(self, collection, items, quantum_context\u003dNone,"},{"line_number":383,"context_line":"                               query_params\u003dNone):"},{"line_number":384,"context_line":"        res \u003d self._list(\u0027%ss\u0027 % collection,"},{"line_number":385,"context_line":"                         quantum_context\u003dquantum_context,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMH%2F%2FwAA%3D","line":382,"updated":"2012-11-27 11:33:33.000000000","message":"collection -\u003e resource? Maybe function\u0027s name can be _check_list_resources_against_wanted(self, resource, wanted_results, quantum_context\u003dNone, query_params\u003dNone):","commit_id":"99c04316f6149e9ee134eecc302efc3f588bbe52"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"d0484514f6254e81004a83eccf6cf279b38cc7c4","unresolved":false,"context_lines":[{"line_number":379,"context_line":"        self.assertEqual(items[0][\u0027name\u0027], \u0027test_0\u0027)"},{"line_number":380,"context_line":"        self.assertEqual(items[1][\u0027name\u0027], \u0027test_1\u0027)"},{"line_number":381,"context_line":""},{"line_number":382,"context_line":"    def _test_list_collections(self, collection, items, quantum_context\u003dNone,"},{"line_number":383,"context_line":"                               query_params\u003dNone):"},{"line_number":384,"context_line":"        res \u003d self._list(\u0027%ss\u0027 % collection,"},{"line_number":385,"context_line":"                         quantum_context\u003dquantum_context,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMH%2F%2Fu60%3D","line":382,"in_reply_to":"AAAAMH%2F%2FwAA%3D","updated":"2012-11-28 05:38:41.000000000","message":"Thanks. And test_list_(collection|resource) should be enough for express the meaning.","commit_id":"99c04316f6149e9ee134eecc302efc3f588bbe52"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"unresolved":false,"context_lines":[{"line_number":757,"context_line":"\"\"\".strip() % (fixed_ips[\u0027ip_address\u0027],"},{"line_number":758,"context_line":"               \u0027192.168.126.5\u0027,"},{"line_number":759,"context_line":"               fixed_ips[\u0027subnet_id\u0027])"},{"line_number":760,"context_line":"            self._test_list_collections(\u0027port\u0027, [port1],"},{"line_number":761,"context_line":"                                        query_params\u003dquery_params)"},{"line_number":762,"context_line":""},{"line_number":763,"context_line":"    def test_list_ports_public_network(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMH%2F%2Fv%2F8%3D","line":760,"updated":"2012-11-27 11:33:33.000000000","message":"list_collectoins -\u003e list_resources?","commit_id":"99c04316f6149e9ee134eecc302efc3f588bbe52"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"d0484514f6254e81004a83eccf6cf279b38cc7c4","unresolved":false,"context_lines":[{"line_number":757,"context_line":"\"\"\".strip() % (fixed_ips[\u0027ip_address\u0027],"},{"line_number":758,"context_line":"               \u0027192.168.126.5\u0027,"},{"line_number":759,"context_line":"               fixed_ips[\u0027subnet_id\u0027])"},{"line_number":760,"context_line":"            self._test_list_collections(\u0027port\u0027, [port1],"},{"line_number":761,"context_line":"                                        query_params\u003dquery_params)"},{"line_number":762,"context_line":""},{"line_number":763,"context_line":"    def test_list_ports_public_network(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAMH%2F%2Fu6w%3D","line":760,"in_reply_to":"AAAAMH%2F%2Fv%2F8%3D","updated":"2012-11-28 05:38:41.000000000","message":"Done","commit_id":"99c04316f6149e9ee134eecc302efc3f588bbe52"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"unresolved":false,"context_lines":[{"line_number":2494,"context_line":"                                  network[\u0027network\u0027][\u0027id\u0027])"},{"line_number":2495,"context_line":""},{"line_number":2496,"context_line":"    def test_list_subnets(self):"},{"line_number":2497,"context_line":"        # NOTE(jkoelker) This would be a good place to use contextlib.nested"},{"line_number":2498,"context_line":"        #                or just drop 2.6 support ;)"},{"line_number":2499,"context_line":"        with self.network() as network:"},{"line_number":2500,"context_line":"            with contextlib.nested(self.subnet(network\u003dnetwork,"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAMX%2F%2F%2BZg%3D","line":2497,"updated":"2012-12-04 10:25:05.000000000","message":"Can we remove the comment?","commit_id":"8223c1d5a4c3b6bde2688403915375cd3ca52d3e"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"d0484514f6254e81004a83eccf6cf279b38cc7c4","unresolved":false,"context_lines":[{"line_number":2494,"context_line":"                                  network[\u0027network\u0027][\u0027id\u0027])"},{"line_number":2495,"context_line":""},{"line_number":2496,"context_line":"    def test_list_subnets(self):"},{"line_number":2497,"context_line":"        # NOTE(jkoelker) This would be a good place to use contextlib.nested"},{"line_number":2498,"context_line":"        #                or just drop 2.6 support ;)"},{"line_number":2499,"context_line":"        with self.network() as network:"},{"line_number":2500,"context_line":"            with contextlib.nested(self.subnet(network\u003dnetwork,"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAMX%2F%2F%2BNI%3D","line":2497,"in_reply_to":"AAAAMX%2F%2F%2BZg%3D","updated":"2012-12-04 13:22:11.000000000","message":"Done","commit_id":"8223c1d5a4c3b6bde2688403915375cd3ca52d3e"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"unresolved":false,"context_lines":[{"line_number":2531,"context_line":"                    self.assertIn(priv_subnet[\u0027subnet\u0027][\u0027cidr\u0027], cidrs)"},{"line_number":2532,"context_line":""},{"line_number":2533,"context_line":"    def test_list_subnets_with_parameter(self):"},{"line_number":2534,"context_line":"        # NOTE(jkoelker) This would be a good place to use contextlib.nested"},{"line_number":2535,"context_line":"        #                or just drop 2.6 support ;)"},{"line_number":2536,"context_line":"        with self.network() as network:"},{"line_number":2537,"context_line":"            with contextlib.nested(self.subnet(network\u003dnetwork,"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAMX%2F%2F%2BZc%3D","line":2534,"updated":"2012-12-04 10:25:05.000000000","message":"Can we remove the comment?","commit_id":"8223c1d5a4c3b6bde2688403915375cd3ca52d3e"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"d0484514f6254e81004a83eccf6cf279b38cc7c4","unresolved":false,"context_lines":[{"line_number":2531,"context_line":"                    self.assertIn(priv_subnet[\u0027subnet\u0027][\u0027cidr\u0027], cidrs)"},{"line_number":2532,"context_line":""},{"line_number":2533,"context_line":"    def test_list_subnets_with_parameter(self):"},{"line_number":2534,"context_line":"        # NOTE(jkoelker) This would be a good place to use contextlib.nested"},{"line_number":2535,"context_line":"        #                or just drop 2.6 support ;)"},{"line_number":2536,"context_line":"        with self.network() as network:"},{"line_number":2537,"context_line":"            with contextlib.nested(self.subnet(network\u003dnetwork,"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAMX%2F%2F%2BNE%3D","line":2534,"in_reply_to":"AAAAMX%2F%2F%2BZc%3D","updated":"2012-12-04 13:22:11.000000000","message":"Done","commit_id":"8223c1d5a4c3b6bde2688403915375cd3ca52d3e"}]}