commit e30455b947669e3dd9adc9f2e591a0c5f3b97d7f
author Rich Lane <rich.lane@bigswitch.com> Thu Jan 03 16:24:44 2013 -0800
committer Rich Lane <rich.lane@bigswitch.com> Thu Jan 03 18:03:45 2013 -0800
tree e1b5155c05c4e978c78b8b51297e00ba8429b478
parent 80efd65292c3d4f4fb81fb0c4c0bc5b8724b86e7

action_list: throw exception if value is not an action

This is a programmer error, not a switch issue.

tests/flow_stats.py

diff --git a/tests/flow_stats.py b/tests/flow_stats.py
index 040fec8..da12071 100644
--- a/tests/flow_stats.py
+++ b/tests/flow_stats.py
@@ -144,7 +144,7 @@
         flow_mod_msg.hard_timeout = 65000
         flow_mod_msg.priority = 100
         act.port = egress_port
-        self.assertTrue(flow_mod_msg.actions.add(act), "Could not add action")
+        flow_mod_msg.actions.add(act)
        
         # send flow
         logging.info("Inserting flow")
@@ -197,7 +197,7 @@
         flow_mod_msg.hard_timeout = 0
         act = action.action_output()
         act.port = egress_port
-        self.assertTrue(flow_mod_msg.actions.add(act), "Could not add action")
+        flow_mod_msg.actions.add(act)
 
         logging.info("Ingress " + str(ingress_port) + 
                        " to egress " + str(egress_port))
@@ -325,7 +325,7 @@
         flow_mod_msg.hard_timeout = 0
         act = action.action_output()
         act.port = egress_port
-        self.assertTrue(flow_mod_msg.actions.add(act), "Could not add action")
+        flow_mod_msg.actions.add(act)
 
         logging.info("Ingress " + str(ingress_port) + 
                        " to egress " + str(egress_port))
@@ -504,7 +504,7 @@
         flow_mod_msg.priority = 100
         flow_mod_msg.flags = ofp.OFPFF_SEND_FLOW_REM
         act.port = egress_port
-        self.assertTrue(flow_mod_msg.actions.add(act), "Could not add action")
+        flow_mod_msg.actions.add(act)
 
         # send flow
         logging.info("Inserting flow")