Attention is currently required from: fixeria.

pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34255 )

Change subject: dia2gsup: Introduce new error scenario tests
......................................................................


Patch Set 1:

(1 comment)

File dia2gsup/DIA2GSUP_Tests.ttcn:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34255/comment/52a388de_2a9b5b0a
PS1, Line 345: TC_authinfo_gsup_err_imsi_unknown
> Having separate testcases (and even separate implementations) for testing the 
> same scenario looks re […]
Because it's not the same scenario. If they were, the same error cause would be 
returned, they all would have the same test name and I wouldn't have catch that 
different errors are not translated properly.



--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34255
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ic870a2c1de74bee3f7d8e26874635d58469ccbea
Gerrit-Change-Number: 34255
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Thu, 31 Aug 2023 08:04:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <[email protected]>
Gerrit-MessageType: comment

Reply via email to