Change in ...osmo-hlr[master]: tests/db_test: close the database when test is finished

2019-07-30 Thread Vadim Yanitskiy 
Vadim Yanitskiy has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-hlr/+/14965 )

Change subject: tests/db_test: close the database when test is finished
..

tests/db_test: close the database when test is finished

Change-Id: I96fedf9181e89e4d68815b04f494a9c2ecc0e057
---
M tests/db/db_test.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/tests/db/db_test.c b/tests/db/db_test.c
index cc299bf..91b32ea 100644
--- a/tests/db/db_test.c
+++ b/tests/db/db_test.c
@@ -979,6 +979,7 @@
test_subscr_sqn();

printf("Done\n");
+   db_close(dbc);
return 0;
 }


--
To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/14965
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-Change-Id: I96fedf9181e89e4d68815b04f494a9c2ecc0e057
Gerrit-Change-Number: 14965
Gerrit-PatchSet: 2
Gerrit-Owner: Vadim Yanitskiy 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Vadim Yanitskiy 
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged

Change in ...osmo-hlr[master]: tests/db_test: close the database when test is finished

2019-07-26 Thread laforge 
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-hlr/+/14965 )

Change subject: tests/db_test: close the database when test is finished
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/14965
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-Change-Id: I96fedf9181e89e4d68815b04f494a9c2ecc0e057
Gerrit-Change-Number: 14965
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Fri, 26 Jul 2019 12:12:26 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in ...osmo-hlr[master]: tests/db_test: close the database when test is finished

2019-07-26 Thread fixeria 
fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-hlr/+/14965


Change subject: tests/db_test: close the database when test is finished
..

tests/db_test: close the database when test is finished

Change-Id: I96fedf9181e89e4d68815b04f494a9c2ecc0e057
---
M tests/db/db_test.c
1 file changed, 1 insertion(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-hlr refs/changes/65/14965/1

diff --git a/tests/db/db_test.c b/tests/db/db_test.c
index fdd62c5..57fbf0c 100644
--- a/tests/db/db_test.c
+++ b/tests/db/db_test.c
@@ -903,6 +903,7 @@
test_subscr_sqn();

printf("Done\n");
+   db_close(dbc);
return 0;
 }


--
To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/14965
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-Change-Id: I96fedf9181e89e4d68815b04f494a9c2ecc0e057
Gerrit-Change-Number: 14965
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-MessageType: newchange