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@xxxxxxxxxx> --- server/tests/test-codecs-parsing.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Changes since v1: - check always error setting codecs. diff --git a/server/tests/test-codecs-parsing.c b/server/tests/test-codecs-parsing.c index 7998501c9..0037c058f 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); g_test_assert_expected_messages(); } -- 2.20.1 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel