summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorIWASE Yusuke <iwase.yusuke0@gmail.com>2015-10-09 16:51:25 +0900
committerFUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>2015-10-11 22:10:24 +0900
commit72a5b442b1192c3e94cee9f48df265a95e72ebfe (patch)
tree4c054fe6126b114e367efaec7611a46fb0d24220
parentc9783734b46b566c14d8b4fc4dc8061d6f78fac1 (diff)
tester: Remove too broad exception handling
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 <iwase.yusuke0@gmail.com> Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
-rw-r--r--ryu/tests/switch/tester.py3
1 files changed, 0 insertions, 3 deletions
diff --git a/ryu/tests/switch/tester.py b/ryu/tests/switch/tester.py
index f5bbd53b..3af7f767 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: