acl-plugin: make test: do not use a debug CLI where API can be used 32/11332/2
authorAndrew Yourtchenko <ayourtch@gmail.com>
Fri, 23 Mar 2018 11:20:56 +0000 (12:20 +0100)
committerDamjan Marion <dmarion.lists@gmail.com>
Fri, 23 Mar 2018 13:00:52 +0000 (13:00 +0000)
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 <ayourtch@gmail.com>
test/test_acl_plugin_macip.py

index bf87e02..b009fb4 100644 (file)
@@ -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)