[MERGED] openbsc[master]: Use ipa.py for ctrl tests

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Max gerrit-no-reply at lists.osmocom.org
Mon May 8 08:31:10 UTC 2017


Max has submitted this change and it was merged.

Change subject: Use ipa.py for ctrl tests
......................................................................


Use ipa.py for ctrl tests

Remove duplicated code and make sure that python code is actively used
as part of test harness

Change-Id: I676390abe2f179df6004cdd33d0eaaf60e18df03
---
M openbsc/tests/ctrl_test_runner.py
1 file changed, 8 insertions(+), 16 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/openbsc/tests/ctrl_test_runner.py b/openbsc/tests/ctrl_test_runner.py
index 02bc28a..0a99c89 100644
--- a/openbsc/tests/ctrl_test_runner.py
+++ b/openbsc/tests/ctrl_test_runner.py
@@ -30,6 +30,11 @@
 import osmopy.obscvty as obscvty
 import osmopy.osmoutil as osmoutil
 
+# add $top_srcdir/contrib to find ipa.py
+sys.path.append(os.path.join(sys.path[0], '..', 'contrib'))
+
+from ipa import Ctrl, IPA
+
 # to be able to find $top_srcdir/doc/...
 confpath = os.path.join(sys.path[0], '..')
 verbose = False
@@ -65,20 +70,6 @@
         self.disconnect()
         osmoutil.end_proc(self.proc)
 
-    def prefix_ipa_ctrl_header(self, data):
-        return struct.pack(">HBB", len(data)+1, 0xee, 0) + data
-
-    def remove_ipa_ctrl_header(self, data):
-        if (len(data) < 4):
-            raise BaseException("Answer too short!")
-        (plen, ipa_proto, osmo_proto) = struct.unpack(">HBB", data[:4])
-        if (plen + 3 > len(data)):
-            print "Warning: Wrong payload length (expected %i, got %i)" % (plen, len(data) - 3)
-        if (ipa_proto != 0xee or osmo_proto != 0):
-            raise BaseException("Wrong protocol in answer!")
-
-        return data[4:plen+3], data[plen+3:]
-
     def disconnect(self):
         if not (self.sock is None):
             self.sock.close()
@@ -106,7 +97,7 @@
     def send(self, data):
         if verbose:
             print "Sending \"%s\"" %(data)
-        data = self.prefix_ipa_ctrl_header(data)
+        data = Ctrl().add_header(data)
         return self.sock.send(data) == len(data)
 
     def send_set(self, var, value, id):
@@ -133,7 +124,8 @@
         responses = {}
         data = self.sock.recv(4096)
         while (len(data)>0):
-            (answer, data) = self.remove_ipa_ctrl_header(data)
+            (head, data) = IPA().split_combined(data)
+            answer = Ctrl().rem_header(head)
             if verbose:
                 print "Got message:", answer
             (mtype, id, msg) = answer.split(None, 2)

-- 
To view, visit https://gerrit.osmocom.org/2466
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I676390abe2f179df6004cdd33d0eaaf60e18df03
Gerrit-PatchSet: 2
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>



More information about the gerrit-log mailing list