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/osmocom-net-gprs@lists.osmocom.org/.
Holger Freyther holger at freyther.de> On 11 Apr 2016, at 06:45, Aravind Sirsikar <Arvind.Sirsikar at radisys.com> wrote: > > From: Aravind Sirsikar <arvind.sirsikar at radisys.com> > > Add test cases to verify the MCS change functionality. > During MCS upgradation such as MCS6->MCS9, 2 blocks > which were sent separately as MCS6, will be clubbed into > one MCS9 block during retransmission. Same holds good for > MCS5->MCS7 transistion. During MCS reduction such as > MCS9->MCS6,2 blocks which were sent together will be > sent separately during the retransmission case. Same is > verified through the generated log file. Currently > MCS8->MCS6 transition is not supported. The retransmission > MCS is being calculated from Table 8.1.1.2 of TS 44.060. We do testing in two ways: 1.) Use OSMO_ASSERT to assert a certain side-effect/result 2.) Capture the output to catch accidental changes In your case only 2nd is done. It is good as it establishes a baseline but it is not sufficient as no actual side-effect/result is being asserted. E.g. as a reviewer it is very hard/time consuming to go through the debug log to actually find the re-transmission. So please add more strong post conditions to the test. kind regards holger