Attention is currently required from: fixeria, pespin.
1 comment:
File msc/BSC_ConnectionHandler.ttcn:
Patch Set #1, Line 1366: if (g_pars.ran_is_geran) {
Ack, and f_expect_paging can actually be an altstep.
Instead of going into this code again i'd rather drop this mild convenience patch that only helps when osmo-msc is buggy, i.e. quite soon never.
I see how your feedback can make sense but it is simply not worth the effort, let's not idealize our test suite.
On a larger scale:
I don't like that ttcn3 forces us to build up these alt-steps-with-timers and catching non-matching messages, all over the place. Instead we should implement and apply a generalized soultion for this, sort of like f_rua_expect() functions.
So instead of spreading these constructs over and over again, we should find a way to *generally* fix our test suite usability.
So, no, I disagree very much with going into these bike sheds.
To view, visit change 36456. To unsubscribe, or for help writing mail filters, visit settings.