[Spice-commits] server/tests

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Mon Jul 8 07:16:09 UTC 2019

 server/tests/test-codecs-parsing.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit fdf104fe28aeefa8e0d01fb42f15ea5c6495cbd7
Author: Frediano Ziglio <fziglio at redhat.com>
Date:   Fri Jul 5 13:56:23 2019 +0100

    test-codecs-parsing: On bad codec string spice_server_set_video_codecs can fail
    The 0 result means success however the function (correctly) could
    report a failure if the string is incorrect.
    This fixes the test after commit b4150de3cd0e56d4ce43a99ef5c3c5f5cbdfc4a3
    ("spice_server_set_video_codecs: fail when no codec can be installed").
    Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
    Acked-by: Snir Sheriber <ssheribe at redhat.com

diff --git a/server/tests/test-codecs-parsing.c b/server/tests/test-codecs-parsing.c
index 7998501c..0037c058 100644
--- a/server/tests/test-codecs-parsing.c
+++ b/server/tests/test-codecs-parsing.c
@@ -140,7 +140,7 @@ static void codecs_bad(void)
         g_test_expect_message(G_LOG_DOMAIN, test_cases[i].log_level, test_cases[i].error_message);
         if (test_cases[i].default_err_message)
             g_test_expect_message(G_LOG_DOMAIN, G_LOG_LEVEL_WARNING, "*Failed to set video codecs*");
-        g_assert_cmpint(spice_server_set_video_codecs(server, test_cases[i].codecs), ==, 0);
+        g_assert_cmpint(spice_server_set_video_codecs(server, test_cases[i].codecs), !=, 0);

More information about the Spice-commits mailing list