Harald Welte has submitted this change and it was merged.
Change subject: ctrl_test: add two more current parsing bugs to prep for fix
......................................................................
ctrl_test: add two more current parsing bugs to prep for fix
Change-Id: Id11bc326be2f0bc2746a928354e416495a18baf7
---
M tests/ctrl/ctrl_test.c
M tests/ctrl/ctrl_test.ok
2 files changed, 34 insertions(+), 0 deletions(-)
Approvals:
Harald Welte: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/tests/ctrl/ctrl_test.c b/tests/ctrl/ctrl_test.c
index e25929c..b1d4f23 100644
--- a/tests/ctrl/ctrl_test.c
+++ b/tests/ctrl/ctrl_test.c
@@ -154,6 +154,22 @@
"ERROR 1 Command not found",
},
+ { "GET 1 var\ti\table",
+ {
+ .type = CTRL_TYPE_GET,
+ .id = "1",
+ .variable = "var\ti\table", /* current bug */
+ },
+ "ERROR 1 Command not found",
+ },
+ { "GET 1 var\ri\rable",
+ {
+ .type = CTRL_TYPE_GET,
+ .id = "1",
+ .variable = "var\ri\rable", /* current bug */
+ },
+ "ERROR 1 Command not found",
+ },
{ "GET 1 variable value",
{
.type = CTRL_TYPE_GET,
diff --git a/tests/ctrl/ctrl_test.ok b/tests/ctrl/ctrl_test.ok
index 210c599..4a3a169 100644
--- a/tests/ctrl/ctrl_test.ok
+++ b/tests/ctrl/ctrl_test.ok
@@ -34,6 +34,24 @@
handling:
replied: 'ERROR 1 Command not found'
ok
+test: 'GET 1 var\ti\table'
+parsing:
+id = '1'
+variable = 'var\ti\table'
+value = '(null)'
+reply = '(null)'
+handling:
+replied: 'ERROR 1 Command not found'
+ok
+test: 'GET 1 var\ri\rable'
+parsing:
+id = '1'
+variable = 'var\ri\rable'
+value = '(null)'
+reply = '(null)'
+handling:
+replied: 'ERROR 1 Command not found'
+ok
test: 'GET 1 variable value'
parsing:
id = '1'
--
To view, visit https://gerrit.osmocom.org/5435
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id11bc326be2f0bc2746a928354e416495a18baf7
Gerrit-PatchSet: 5
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofm...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
