X-Git-Url: https://gerrit.fd.io/r/gitweb?a=blobdiff_plain;f=test%2Ftest_tap.py;h=0e5b315963e2695fe4b8fd79565e9fe6af92ab84;hb=073d74d0b;hp=92589789a95512a4169b44e057299338c3c50208;hpb=5de4fb7076a46ab75e2d3c30079dd6639af16a86;p=vpp.git

diff --git a/test/test_tap.py b/test/test_tap.py
index 92589789a95..0e5b315963e 100644
--- a/test/test_tap.py
+++ b/test/test_tap.py
@@ -6,10 +6,10 @@ from vpp_devices import VppTAPInterface
 
 
 def check_tuntap_driver_access():
-    return os.access("/dev/net/tun", os.R_OK or os.W_OK)
+    return os.access("/dev/net/tun", os.R_OK and os.W_OK)
 
 
-@unittest.skipUnless(check_tuntap_driver_access(), "Permission denied")
+@unittest.skip("Requires root")
 class TestTAP(VppTestCase):
     """ TAP Test Case """
 
@@ -19,6 +19,20 @@ class TestTAP(VppTestCase):
         tap0.add_vpp_config()
         self.assertTrue(tap0.query_vpp_config())
 
+    def test_tap_dump(self):
+        """ Test api dump w/ and w/o sw_if_index filtering"""
+        MAX_INSTANCES = 10
+        tap_instances = []
+        for instance in range(MAX_INSTANCES):
+            i = VppTAPInterface(self, tap_id=instance)
+            i.add_vpp_config()
+            tap_instances.append(i)
+        details = self.vapi.sw_interface_tap_v2_dump()
+        self.assertEqual(MAX_INSTANCES, len(details))
+        details = self.vapi.sw_interface_tap_v2_dump(
+            tap_instances[5].sw_if_index)
+        self.assertEqual(1, len(details))
+
 
 if __name__ == '__main__':
     unittest.main(testRunner=VppTestRunner)