Because too broad exception handling obscures errors while testing,
it makes difficult to detect the bugs in tester.py or the test
pattern files.
This patch removes an except statement and avoid this problem.

Signed-off-by: IWASE Yusuke <[email protected]>
---
 ryu/tests/switch/tester.py | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py
index f5bbd53..3af7f76 100644
--- a/ryu/tests/switch/tester.py
+++ b/ryu/tests/switch/tester.py
@@ -529,9 +529,6 @@ class OfTester(app_manager.RyuApp):
                 TestTimeout, TestReceiveError) as err:
             result = [TEST_ERROR, str(err)]
             result_type = str(err).split(':', 1)[0]
-        except Exception:
-            result = [TEST_ERROR, RYU_INTERNAL_ERROR]
-            result_type = RYU_INTERNAL_ERROR
         finally:
             self.ingress_event = None
             for tid in self.ingress_threads:
-- 
1.9.1


------------------------------------------------------------------------------
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to