commit 85bdafacbdd25f88c06076ca7208ff14fe10b74c
author Ole Troan <ot@cisco.com> Wed Dec 21 11:51:05 2016 +0100
committer Damjan Marion <dmarion.lists@gmail.com> Wed Dec 21 17:05:05 2016 +0000
tree 4cf10e8085f70cbc2c4a97ba2268488b1b989548
parent ee49bf82f5548959e0489e9dfac3a071c99f70de

Python API: Incorrect error for empty VLA list.

Change-Id: Ie3e9a54ca23af6b9ee43615f59a48cb05e3906d4
Signed-off-by: Ole Troan <ot@cisco.com>

vpp-api/python/tests/test_vpp_papi2.py

diff --git a/vpp-api/python/tests/test_vpp_papi2.py b/vpp-api/python/tests/test_vpp_papi2.py
index 18f6b68..f45f791 100755
--- a/vpp-api/python/tests/test_vpp_papi2.py
+++ b/vpp-api/python/tests/test_vpp_papi2.py
@@ -434,7 +434,7 @@
 
         rv = vpp.acl_plugin_get_version()
         print('RV', rv)
-        self.assertEqual(rv.major, 0)
+        self.assertEqual(rv.major, 1)
         self.assertEqual(rv.minor, 1)
 
         rv = vpp.acl_add_replace(acl_index = 0xFFFFFFFF,
@@ -472,6 +472,16 @@
         vpp = VPP(jsonfiles)
         vpp.status()
 
+    def test_acl_interface_get(self):
+        vpp = VPP(jsonfiles)
+
+        vpp.connect('test_vpp_papi2')
+
+        rv = vpp.macip_acl_interface_get()
+
+        print('RV', rv)
+
+        vpp.disconnect()
 
 if __name__ == '__main__':
     unittest.main()

vpp-api/python/vpp_papi/vpp_papi.py

diff --git a/vpp-api/python/vpp_papi/vpp_papi.py b/vpp-api/python/vpp_papi/vpp_papi.py
index 02fe245..6b6b79f 100644
--- a/vpp-api/python/vpp_papi/vpp_papi.py
+++ b/vpp-api/python/vpp_papi/vpp_papi.py
@@ -206,9 +206,6 @@
                     e = v[0] if type(v[0]) is int else res[v[2]]
                     if e == -1:
                         e = (len(buf) - off) / v[1].size
-                    if e == 0:
-                        raise ValueError(1,
-                                         'Variable length array, empty length: ' + k)
                     lst = []
                     res.append(lst)
                     size = 0