Attention is currently required from: fixeria, lynxis lazus.
View Change
1 comment:
Patchset:
Patch Set #4:
Then please at least add unit tests here. I am aiming for good testing coverage.
I'm aiming for it too, but not everything can be done together at the same time.
I'm not aware that you followed yourself the same constrains that you are now applying on me in here while implementing all the stuff you did for osmo-s1gw.
The ttcn3 testsuite is not yet in a state where new tests can easily be added.
To view, visit change 37924. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-MessageType: comment
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: I242e84fb09b00f4794b6e1aa770f348a0e60aea4
Gerrit-Change-Number: 37924
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu>
Gerrit-Attention: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis@fe80.eu>
Gerrit-Comment-Date: Wed, 04 Sep 2024 14:07:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin@sysmocom.de>
Comment-In-Reply-To: fixeria <vyanitskiy@sysmocom.de>