[taler-merchant] branch master updated: [ci] fixing CI testing, SERVE values set, more error message, WIP

[gnunet]

This is an automated email from the git hooks/post-receive script.

oec pushed a commit to branch master
in repository merchant.

The following commit(s) were added to refs/heads/master by this push:
     new 6efab6ed [ci] fixing CI testing, SERVE values set, more error message, 
WIP
6efab6ed is described below

commit 6efab6eddad09ed8f209bc2f740c7c29ccc1058b
Author: Özgür Kesim <oec-taler@kesim.org>
AuthorDate: Tue Jan 23 11:58:18 2024 +0100

    [ci] fixing CI testing, SERVE values set, more error message, WIP
---
 contrib/ci/jobs/2-test/test.sh                | 2 +-
 src/testing/test.conf                         | 2 +-
 src/testing/test_key_rotation.conf            | 2 +-
 src/testing/test_kyc_api.conf                 | 2 +-
 src/testing/test_merchant_api.conf            | 2 +-
 src/testing/testing_api_cmd_delete_template.c | 7 ++++---
 src/testing/testing_api_cmd_forget_order.c    | 7 ++++---
 src/testing/testing_api_cmd_post_instances.c  | 9 ++++-----
 src/testing/testing_api_cmd_post_orders.c     | 2 ++
 src/testing/testing_api_cmd_post_templates.c  | 7 ++++---
 10 files changed, 23 insertions(+), 19 deletions(-)

diff --git a/contrib/ci/jobs/2-test/test.sh b/contrib/ci/jobs/2-test/test.sh
index 68adb1b7..0c02085f 100755
--- a/contrib/ci/jobs/2-test/test.sh
+++ b/contrib/ci/jobs/2-test/test.sh
@@ -19,7 +19,7 @@ sudo -u postgres createdb -p 5432 -O root talercheck
 check_command()
 {
        # Set LD_LIBRARY_PATH so tests can find the installed libs
-       LD_LIBRARY_PATH=/usr/lib/x86_64-linux-gnu/taler:/usr/lib/taler 
PGPORT=5432 make check
+       LD_LIBRARY_PATH=/usr/lib/x86_64-linux-gnu/taler:/usr/lib:/usr/lib/taler 
PGPORT=5432 make check
 }
 
 print_logs()
diff --git a/src/testing/test.conf b/src/testing/test.conf
index a338e22e..2cfc0418 100644
--- a/src/testing/test.conf
+++ b/src/testing/test.conf
@@ -40,7 +40,7 @@ HTTP_PORT = 8082
 
 # Which port do we run the backend on? (HTTP server)
 PORT = 8080
-SERVE = http
+SERVE = tcp
 
 # Which plugin (backend) do we use for the DB.
 DB = postgres
diff --git a/src/testing/test_key_rotation.conf 
b/src/testing/test_key_rotation.conf
index 7ce0c27e..cc7035b0 100644
--- a/src/testing/test_key_rotation.conf
+++ b/src/testing/test_key_rotation.conf
@@ -78,7 +78,7 @@ HTTP_PORT = 8082
 SUGGESTED_EXCHANGE = http://localhost:8081/
 SUGGESTED_EXCHANGE_PAYTO = payto://x-taler-bank/localhost/2
 ALLOW_REGISTRATIONS = YES
-SERVE = http
+SERVE = tcp
 
 [exchangedb]
 IDLE_RESERVE_EXPIRATION_TIME = 4 weeks
diff --git a/src/testing/test_kyc_api.conf b/src/testing/test_kyc_api.conf
index ce5151a2..bbbfc17c 100644
--- a/src/testing/test_kyc_api.conf
+++ b/src/testing/test_kyc_api.conf
@@ -19,7 +19,7 @@ HTTP_PORT = 8082
 
 [merchant]
 PORT = 8080
-SERVE = http
+SERVE = tcp
 DB = postgres
 
 # This specifies which database the postgres backend uses.
diff --git a/src/testing/test_merchant_api.conf 
b/src/testing/test_merchant_api.conf
index c09bafe0..1ad96ca5 100644
--- a/src/testing/test_merchant_api.conf
+++ b/src/testing/test_merchant_api.conf
@@ -32,7 +32,7 @@ PORT = 8082
 
 [merchant]
 PORT = 8080
-SERVE = http
+SERVE = tcp
 DB = postgres
 
 [merchantdb-postgres]
diff --git a/src/testing/testing_api_cmd_delete_template.c 
b/src/testing/testing_api_cmd_delete_template.c
index a2c7feeb..6227c543 100644
--- a/src/testing/testing_api_cmd_delete_template.c
+++ b/src/testing/testing_api_cmd_delete_template.c
@@ -77,9 +77,10 @@ delete_template_cb (void *cls,
   dis->tdh = NULL;
   if (dis->http_status != hr->http_status)
   {
-    TALER_TESTING_unexpected_status (dis->is,
-                                     hr->http_status,
-                                     dis->http_status);
+    TALER_TESTING_unexpected_status_with_body (dis->is,
+                                               hr->http_status,
+                                               dis->http_status,
+                                               hr->reply);
     return;
   }
   switch (hr->http_status)
diff --git a/src/testing/testing_api_cmd_forget_order.c 
b/src/testing/testing_api_cmd_forget_order.c
index 76d2427b..172ac295 100644
--- a/src/testing/testing_api_cmd_forget_order.c
+++ b/src/testing/testing_api_cmd_forget_order.c
@@ -121,9 +121,10 @@ order_forget_cb (void *cls,
   ofs->ofh = NULL;
   if (ofs->http_status != hr->http_status)
   {
-    TALER_TESTING_unexpected_status (ofs->is,
-                                     hr->http_status,
-                                     ofs->http_status);
+    TALER_TESTING_unexpected_status_with_body (ofs->is,
+                                               hr->http_status,
+                                               ofs->http_status,
+                                               hr->reply);
     return;
   }
   TALER_TESTING_interpreter_next (ofs->is);
diff --git a/src/testing/testing_api_cmd_post_instances.c 
b/src/testing/testing_api_cmd_post_instances.c
index f1b81cbf..0d081026 100644
--- a/src/testing/testing_api_cmd_post_instances.c
+++ b/src/testing/testing_api_cmd_post_instances.c
@@ -112,11 +112,10 @@ post_instances_cb (void *cls,
   pis->iph = NULL;
   if (pis->http_status != hr->http_status)
   {
-    GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-                "Unexpected response code %u (%d) to command %s\n",
-                hr->http_status,
-                (int) hr->ec,
-                TALER_TESTING_interpreter_get_current_label (pis->is));
+    TALER_TESTING_unexpected_status_with_body (pis->is,
+                                               hr->http_status,
+                                               pis->http_status,
+                                               hr->reply);
     TALER_TESTING_interpreter_fail (pis->is);
     return;
   }
diff --git a/src/testing/testing_api_cmd_post_orders.c 
b/src/testing/testing_api_cmd_post_orders.c
index da4aa406..72fe0b11 100644
--- a/src/testing/testing_api_cmd_post_orders.c
+++ b/src/testing/testing_api_cmd_post_orders.c
@@ -264,6 +264,8 @@ order_cb (void *cls,
                                                por->hr.http_status,
                                                ps->http_status,
                                                por->hr.reply);
+    TALER_TESTING_interpreter_fail (ps->is);
+    return;
   }
   switch (por->hr.http_status)
   {
diff --git a/src/testing/testing_api_cmd_post_templates.c 
b/src/testing/testing_api_cmd_post_templates.c
index 0ff0d11a..f0b6d713 100644
--- a/src/testing/testing_api_cmd_post_templates.c
+++ b/src/testing/testing_api_cmd_post_templates.c
@@ -92,9 +92,10 @@ post_templates_cb (void *cls,
   tis->iph = NULL;
   if (tis->http_status != hr->http_status)
   {
-    TALER_TESTING_unexpected_status (tis->is,
-                                     hr->http_status,
-                                     tis->http_status);
+    TALER_TESTING_unexpected_status_with_body (tis->is,
+                                               hr->http_status,
+                                               tis->http_status,
+                                               hr->reply);
     return;
   }
   switch (hr->http_status)

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.