fixeria has uploaded this change for review.

View Change

tests/meas: improve logging in test_ts45008_83_is_sub_single()

* Print unexpected TDMA FNs, each on a separate new line.
* Do not abort() on mismatch, keep running.

Change-Id: I32c104b139f32a39c8347de19de6ad7caec811de
Related: SYS#5853
---
M tests/meas/meas_test.c
1 file changed, 3 insertions(+), 8 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/82/28082/1

diff --git a/tests/meas/meas_test.c b/tests/meas/meas_test.c
index 9494495..32f0e7a 100644
--- a/tests/meas/meas_test.c
+++ b/tests/meas/meas_test.c
@@ -464,8 +464,7 @@
 		lchan->tch_mode = GSM48_CMODE_SPEECH_V1;
 	}
 
-	printf(" TS=%u ", ts);
-	printf("SS=%u", ss);
+	printf(" TS=%u SS=%u\n", ts, ss);
 
 	/* Walk trough the first 100 intervals and check for unexpected
 	 * results (false positive and false negative) */
@@ -474,19 +473,15 @@
 		if (rc) {
 			if (!test_ts45008_83_is_sub_is_sacch(i)
 			    && !test_ts45008_83_is_sub_is_sub(i, ss)) {
-				printf("==> Unexpected SUB frame at fn=%u", i);
-				OSMO_ASSERT(false);
+				printf("  ==> Unexpected SUB frame at fn=%u\n", i);
 			}
 		} else {
 			if (test_ts45008_83_is_sub_is_sacch(i)
 			    && test_ts45008_83_is_sub_is_sub(i, ss)) {
-				printf("==> Unexpected non-SUB frame at fn=%u",
-				       i);
-				OSMO_ASSERT(false);
+				printf("  ==> Unexpected non-SUB frame at fn=%u\n", i);
 			}
 		}
 	}
-	printf("\n");
 }
 
 static void test_ts45008_83_is_sub(void)

To view, visit change 28082. To unsubscribe, or for help writing mail filters, visit settings.