Re: [PATCH 1/2] tests: display info about what failed in media/tree URI tests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2018-11-19 at 11:43 +0000, Daniel P. Berrangé wrote:
> Make it possible to debug failures by displaying the URL,
> curl error code/message and http code.
> 
> Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> ---
>  tests/test-mediauris.c | 4 ++++
>  tests/test-treeuris.c  | 4 ++++
>  2 files changed, 8 insertions(+)
> 
> diff --git a/tests/test-mediauris.c b/tests/test-mediauris.c
> index 443fdc3..c906f08 100644
> --- a/tests/test-mediauris.c
> +++ b/tests/test-mediauris.c
> @@ -46,6 +46,10 @@ static void test_media(OsinfoMediaList *medialist,
> GError **error, CURL *curl)
>          curl_easy_getinfo(curl, CURLINFO_RESPONSE_CODE,
> &response_code);
>  
>          g_test_message("res=%d, %s; code=%ld", res,
> curl_easy_strerror(res), response_code);
> +        if (res != CURLE_OK) {
> +            g_printerr("Failed URI %s res=%d (%s) code=%ld\n",
> +                       url, res, curl_easy_strerror(res), 

We have a small problem on using g_printerr() though. It'll make the
test to abort in the first failure it faces and it's not something we
would like to do.

> response_code);
> +        }
>          g_assert_cmpint(res, ==, CURLE_OK);
>  
>          tmp = tmp->next;
> diff --git a/tests/test-treeuris.c b/tests/test-treeuris.c
> index 9b5c679..b0cddbf 100644
> --- a/tests/test-treeuris.c
> +++ b/tests/test-treeuris.c
> @@ -46,6 +46,10 @@ static void test_tree(OsinfoTreeList *treelist,
> GError **error, CURL *curl)
>          curl_easy_getinfo(curl, CURLINFO_RESPONSE_CODE,
> &response_code);
>  
>          g_test_message("res=%d, %s; code=%ld", res,
> curl_easy_strerror(res), response_code);
> +        if (res != CURLE_OK) {
> +            g_printerr("Failed URI %s res=%d (%s) code=%ld\n",
> +                       url, res, curl_easy_strerror(res),
> response_code);

Same comment here.

> +        }
>          g_assert_cmpint(res, ==, CURLE_OK);
>  
>          tmp = tmp->next;

Would you be okay on changing the g_printerr() to g_test_message() or
similar? If yes, please, just do the change and push the patch without
the need of a v2.

Also, I've talked to Věra and in general we need to improve our tests
in order to have them displaying better info ... mainly in cases where
we go through our database. Thanks for working on this!

Best Regards,
-- 
Fabiano Fidêncio

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux