Change in osmo-pcu[master]: EDGE tests: reduce code duplication

[Max]

Max has uploaded this change for review. ( https://gerrit.osmocom.org/13051


Change subject: EDGE tests: reduce code duplication
......................................................................

EDGE tests: reduce code duplication

* move duplicated code into helper function
* use proper parameter types

Change-Id: I8a9528032629e5df629996da5cd8b808efede017
---
M tests/edge/EdgeTest.cpp
1 file changed, 15 insertions(+), 8 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/51/13051/1

diff --git a/tests/edge/EdgeTest.cpp b/tests/edge/EdgeTest.cpp
index 98ca206..6ce2334 100644
--- a/tests/edge/EdgeTest.cpp
+++ b/tests/edge/EdgeTest.cpp
@@ -105,11 +105,22 @@
 
 }
 
+static bool check_strong_monotonicity(const GprsCodingScheme *cs, uint8_t last_UL, uint8_t last_DL)
+{
+	if (cs->maxBytesUL() <= last_UL)
+		return false;
+
+	if (cs->maxBytesDL() <= last_DL)
+		return false;
+
+	return true;
+}
+
 static void test_coding_scheme()
 {
 	unsigned i;
-	unsigned last_size_UL;
-	unsigned last_size_DL;
+	uint8_t last_size_UL;
+	uint8_t last_size_DL;
 	GprsCodingScheme::Scheme gprs_schemes[] = {
 		GprsCodingScheme::CS1,
 		GprsCodingScheme::CS2,
@@ -152,9 +163,7 @@
 		OSMO_ASSERT(GprsCodingScheme::Scheme(current_cs) == gprs_schemes[i]);
 		OSMO_ASSERT(current_cs == GprsCodingScheme(gprs_schemes[i]));
 
-		/* Check strong monotonicity */
-		OSMO_ASSERT(current_cs.maxBytesUL() > last_size_UL);
-		OSMO_ASSERT(current_cs.maxBytesDL() > last_size_DL);
+		OSMO_ASSERT(check_strong_monotonicity(&current_cs, last_size_UL, last_size_DL));
 		last_size_UL = current_cs.maxBytesUL();
 		last_size_DL = current_cs.maxBytesDL();
 
@@ -179,9 +188,7 @@
 		OSMO_ASSERT(GprsCodingScheme::Scheme(current_cs) == egprs_schemes[i].s);
 		OSMO_ASSERT(current_cs == GprsCodingScheme(egprs_schemes[i].s));
 
-		/* Check strong monotonicity */
-		OSMO_ASSERT(current_cs.maxBytesUL() > last_size_UL);
-		OSMO_ASSERT(current_cs.maxBytesDL() > last_size_DL);
+		OSMO_ASSERT(check_strong_monotonicity(&current_cs, last_size_UL, last_size_DL));
 		last_size_UL = current_cs.maxBytesUL();
 		last_size_DL = current_cs.maxBytesDL();
 

-- 
To view, visit https://gerrit.osmocom.org/13051
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8a9528032629e5df629996da5cd8b808efede017
Gerrit-Change-Number: 13051
Gerrit-PatchSet: 1
Gerrit-Owner: Max <msuraev at sysmocom.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190226/57f68608/attachment.htm>