pespin has submitted this change and it was merged. (
https://gerrit.osmocom.org/c/libosmocore/+/15563 )
Change subject: tdef_test: verify case where osmo_tdef_set returns -EEXIST
......................................................................
tdef_test: verify case where osmo_tdef_set returns -EEXIST
Change-Id: I436daa804aac11622fde24afe9ea35193d9e9beb
---
M tests/tdef/tdef_test.c
M tests/tdef/tdef_test.ok
2 files changed, 4 insertions(+), 0 deletions(-)
Approvals:
laforge: Looks good to me, but someone else must approve
fixeria: Looks good to me, approved
osmith: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/tests/tdef/tdef_test.c b/tests/tdef/tdef_test.c
index 60066b1..276330a 100644
--- a/tests/tdef/tdef_test.c
+++ b/tests/tdef/tdef_test.c
@@ -179,6 +179,9 @@
print_tdef_get_short(tdefs, 7, OSMO_TDEF_M);
print_tdef_get_short(tdefs, 7, OSMO_TDEF_CUSTOM);
+ printf("setting 23 = 50 (EEXIST)\n");
+ OSMO_ASSERT(osmo_tdef_set(tdefs, 23, 50, OSMO_TDEF_S) == -EEXIST);
+
printf("resetting\n");
osmo_tdefs_reset(tdefs);
print_tdef_info(7);
diff --git a/tests/tdef/tdef_test.ok b/tests/tdef/tdef_test.ok
index 2a3617e..3c4a093 100644
--- a/tests/tdef/tdef_test.ok
+++ b/tests/tdef/tdef_test.ok
@@ -117,6 +117,7 @@
osmo_tdef_get(7, s) = 420
osmo_tdef_get(7, m) = 7
osmo_tdef_get(7, custom-unit) = 420
+setting 23 = 50 (EEXIST)
resetting
T7=50s
osmo_tdef_get(7, s) = 50
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/15563
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I436daa804aac11622fde24afe9ea35193d9e9beb
Gerrit-Change-Number: 15563
Gerrit-PatchSet: 7
Gerrit-Owner: pespin <pes...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilira...@gmail.com>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: osmith <osm...@sysmocom.de>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
Gerrit-MessageType: merged
