From beca26622983018255d87b8031e39e517e4e4242 Mon Sep 17 00:00:00 2001 From: Andrew Yourtchenko Date: Fri, 23 Mar 2018 12:20:56 +0100 Subject: [PATCH] acl-plugin: make test: do not use a debug CLI where API can be used Parsing the list of applied ACLs from the debug CLI output is not a good idea, since the debug CLI can and will change. Fix that. Change-Id: Ia47214863f2658a54219f59cb974b5cbc536f862 Signed-off-by: Andrew Yourtchenko --- test/test_acl_plugin_macip.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/test/test_acl_plugin_macip.py b/test/test_acl_plugin_macip.py index bf87e02ecd9..b009fb4598b 100644 --- a/test/test_acl_plugin_macip.py +++ b/test/test_acl_plugin_macip.py @@ -287,13 +287,10 @@ class MethodHolder(VppTestCase): reply = self.vapi.macip_acl_dump() self.assertEqual(len(reply), 0) - intf_acls = self.vapi.ppcli("sh acl-plugin interface").split( - "\nsw_if_index") + intf_acls = self.vapi.acl_interface_list_dump() for i_a in intf_acls: - ia = i_a.split(":") - if len(ia) == 3: - sw_if_index = int(ia[0]) - acl_index = int(ia[2]) + sw_if_index = i_a.sw_if_index + for acl_index in i_a.acls: self.vapi.acl_interface_add_del(sw_if_index, acl_index, 0) self.vapi.acl_del(acl_index) -- 2.16.6