diff options
author | Ole Troan <ot@cisco.com> | 2016-12-21 11:51:05 +0100 |
---|---|---|
committer | Damjan Marion <dmarion.lists@gmail.com> | 2016-12-21 17:05:05 +0000 |
commit | 85bdafacbdd25f88c06076ca7208ff14fe10b74c (patch) | |
tree | 4cf10e8085f70cbc2c4a97ba2268488b1b989548 | |
parent | ee49bf82f5548959e0489e9dfac3a071c99f70de (diff) |
Python API: Incorrect error for empty VLA list.
Change-Id: Ie3e9a54ca23af6b9ee43615f59a48cb05e3906d4
Signed-off-by: Ole Troan <ot@cisco.com>
-rwxr-xr-x | vpp-api/python/tests/test_vpp_papi2.py | 12 | ||||
-rw-r--r-- | vpp-api/python/vpp_papi/vpp_papi.py | 3 |
2 files changed, 11 insertions, 4 deletions
diff --git a/vpp-api/python/tests/test_vpp_papi2.py b/vpp-api/python/tests/test_vpp_papi2.py index 18f6b6866a8..f45f791e627 100755 --- a/vpp-api/python/tests/test_vpp_papi2.py +++ b/vpp-api/python/tests/test_vpp_papi2.py @@ -434,7 +434,7 @@ class TestACL(unittest.TestCase): 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 @@ class TestACL(unittest.TestCase): 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() diff --git a/vpp-api/python/vpp_papi/vpp_papi.py b/vpp-api/python/vpp_papi/vpp_papi.py index 02fe2457ef0..6b6b79fd70b 100644 --- a/vpp-api/python/vpp_papi/vpp_papi.py +++ b/vpp-api/python/vpp_papi/vpp_papi.py @@ -206,9 +206,6 @@ class VPP(): 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 |