diff options
author | Neels Hofmeyr <nhofmeyr@sysmocom.de> | 2016-09-26 12:59:36 +0200 |
---|---|---|
committer | Neels Hofmeyr <nhofmeyr@sysmocom.de> | 2016-09-26 13:00:35 +0200 |
commit | 89d20b60efd408d79328869f97a8103c776bf0ef (patch) | |
tree | d47095a98a072195f8719692a2fc383ca1617fd6 | |
parent | 23d37c91aff55a741d62b5d348cf791a1df6a6c8 (diff) |
vty_test_runner.py: raise exception when MSC socket connection fails
Instead of below error, raise an exception to describe what's happening.
Seen in a jenkins run on https://gerrit.osmocom.org/#/c/945/2:
ERROR: testBSCreload (__main__.TestVTYNAT)
----------------------------------------------------------------------
Traceback (most recent call last):
File "./vty_test_runner.py", line 787, in testBSCreload
msc = nat_msc_test(self, ip, port)
File "./vty_test_runner.py", line 1251, in nat_msc_test
return conn
UnboundLocalError: local variable 'conn' referenced before assignment
Change-Id: Iae26e7345267a21aed0b108b089453832889c9fa
-rw-r--r-- | openbsc/tests/vty_test_runner.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/openbsc/tests/vty_test_runner.py b/openbsc/tests/vty_test_runner.py index 9fe4c3e75..f624fc96d 100644 --- a/openbsc/tests/vty_test_runner.py +++ b/openbsc/tests/vty_test_runner.py @@ -1233,10 +1233,14 @@ def nat_msc_test(x, ip, port, verbose = False): msc.listen(5) if (verbose): print "MSC is ready at " + ip + conn = None while "MSC is connected: 0" == x.vty.command("show msc connection"): conn, addr = msc.accept() if (verbose): print "MSC got connection from ", addr + if not conn: + raise Exception("VTY reports MSC is connected, but I haven't" + " connected yet: %r %r" % (ip, port)) return conn def ipa_handle_small(x, verbose = False): |