From 22f12a9ebd80726dd43a2f369cd79dcf89dc23fa Mon Sep 17 00:00:00 2001 From: Carlos Garnacho Date: Mon, 11 Jul 2022 20:34:02 +0200 Subject: tests: Invert order of asserts When checking a function with a return value and a GError out argument, it often gives better clues to check the error first, then the return value. --- tests/libtracker-sparql/tracker-deserialize-test.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'tests') diff --git a/tests/libtracker-sparql/tracker-deserialize-test.c b/tests/libtracker-sparql/tracker-deserialize-test.c index cd8c9af87..b378567bd 100644 --- a/tests/libtracker-sparql/tracker-deserialize-test.c +++ b/tests/libtracker-sparql/tracker-deserialize-test.c @@ -153,8 +153,8 @@ deserialize_dbus_cb (GObject *source, retval = tracker_sparql_connection_deserialize_finish (TRACKER_SPARQL_CONNECTION (source), res, &error); - g_assert_true (retval); g_assert_no_error (error); + g_assert_true (retval); g_main_loop_quit (test_fixture->loop); } @@ -195,8 +195,8 @@ deserialize_direct_cb (GObject *source, retval = tracker_sparql_connection_deserialize_finish (TRACKER_SPARQL_CONNECTION (source), res, &error); - g_assert_true (retval); g_assert_no_error (error); + g_assert_true (retval); /* Read RDF data back */ tracker_sparql_connection_serialize_async (test_fixture->direct, -- cgit v1.2.1