Tests: Raise exception, don't raise string. 26/18226/1
authorPaul Vinciguerra <pvinci@vinciconsulting.com>
Wed, 13 Mar 2019 00:41:27 +0000 (17:41 -0700)
committerPaul Vinciguerra <pvinci@vinciconsulting.com>
Wed, 13 Mar 2019 00:48:49 +0000 (17:48 -0700)
This was deprecated ~ python 2.4, and causes a TypeError as sideEffect.

>>> raise "foo"
Traceback (most recent call last):
  File "<input>", line 1, in <module>
TypeError: exceptions must derive from BaseException

Change-Id: I4117b6d60ae896eaa1ef2a73a323d8d241f8c3a7
Signed-off-by: Paul Vinciguerra <pvinci@vinciconsulting.com>
test/test_geneve.py
test/test_gtpu.py
test/test_vxlan.py
test/test_vxlan_gpe.py

index ebe95a0..4d6e3f5 100644 (file)
@@ -113,7 +113,7 @@ class TestGeneve(BridgeDomain, VppTestCase):
                 remote_address=cls.mcast_ip4n, mcast_sw_if_index=1,
                 is_add=is_add, vni=vni)
             if r.sw_if_index == 0xffffffff:
-                raise "bad sw_if_index"
+                raise ValueError("bad sw_if_index: ~0")
 
     @classmethod
     def add_shared_mcast_dst_load(cls):
index f5b1089..4b77628 100644 (file)
@@ -168,7 +168,7 @@ class TestGtpu(BridgeDomain, VppTestCase):
                 teid=teid,
                 is_add=is_add)
             if r.sw_if_index == 0xffffffff:
-                raise "bad sw_if_index"
+                raise ValueError("bad sw_if_index: ~0")
 
     @classmethod
     def add_shared_mcast_dst_load(cls):
index 7e3ea50..5ded725 100644 (file)
@@ -111,7 +111,7 @@ class TestVxlan(BridgeDomain, VppTestCase):
                                               mcast_sw_if_index=1,
                                               is_add=is_add, vni=vni)
             if r.sw_if_index == 0xffffffff:
-                raise "bad sw_if_index"
+                raise ValueError("bad sw_if_index: ~0")
 
     @classmethod
     def add_shared_mcast_dst_load(cls):
index 6ee33d5..36661a9 100644 (file)
@@ -114,7 +114,7 @@ class TestVxlanGpe(BridgeDomain, VppTestCase):
                 vni=vni,
                 is_add=is_add)
             if r.sw_if_index == 0xffffffff:
-                raise "bad sw_if_index"
+                raise ValueError("bad sw_if_index: ~0")
 
     @classmethod
     def add_shared_mcast_dst_load(cls):