Split up the wocky XMPP stanza tests so they each test one thing. This is one of the best practices of testing, although obviously shouldn't be a hard and fast rule.
Fixed here: http://git.collabora.co.uk/?p=user/stefw/wocky.git;a=commit;h=02f235609a9cfc4df076c3d4fc1d01e39c91b9e5
static void -test_build_iq_result (void) +test_build_iq_result_simple_ack (void) { WockyStanza *iq, *reply, *expected; @@ -37,6 +37,20 @@ test_build_iq_result (void) g_object_unref (reply); g_object_unref (expected); +} I think this will leak 'iq'. Otherwise looks good.
Merged; thanks!
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.