diff options
| author | Carlos Garnacho <carlosg@gnome.org> | 2022-07-11 20:34:02 +0200 |
|---|---|---|
| committer | Carlos Garnacho <carlosg@gnome.org> | 2022-07-13 11:09:45 +0200 |
| commit | 22f12a9ebd80726dd43a2f369cd79dcf89dc23fa (patch) | |
| tree | 8a1f6b1deb00587f49c4936c10b7daeb827f1377 /tests | |
| parent | 142b15c332ccc7e68c8ad7b235ff8dacfa5cbe1d (diff) | |
| download | tracker-22f12a9ebd80726dd43a2f369cd79dcf89dc23fa.tar.gz | |
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.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/libtracker-sparql/tracker-deserialize-test.c | 4 |
1 files changed, 2 insertions, 2 deletions
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, |