diff options
| author | Will Thompson <will.thompson@collabora.co.uk> | 2012-02-01 13:05:08 +0000 |
|---|---|---|
| committer | Will Thompson <will.thompson@collabora.co.uk> | 2012-02-01 13:09:53 +0000 |
| commit | eecef2669555c65c20a01c55b7166593bac1d96b (patch) | |
| tree | aa7a381f52e81621f95323fddd6f37959c7dca15 | |
| parent | 43d93377facac1a7b9bae8e898c29b245ec476f5 (diff) | |
porter-test: fix /handler-stanza
test_close_porter() assumes test->sched_in has not been started, and
calls wocky_xmpp_connection_recv_stanza_async() on test->in; the
callback assumes that the call has failed with an error indicating that
the stream has been closed. This would be fine if the test was sure
there were no stanzas in the queue in test->in, but since my change to
make WockyC2SPorter send back an error in response to unhandled IQs,
this assumption is no longer true: there are error replies sitting
around waiting to be handled.
So this patch just makes this test turn both porters on and then use
test_close_both_porters().
| -rw-r--r-- | tests/wocky-porter-test.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/wocky-porter-test.c b/tests/wocky-porter-test.c index 4cd533b..662428a 100644 --- a/tests/wocky-porter-test.c +++ b/tests/wocky-porter-test.c @@ -1086,6 +1086,7 @@ test_handler_stanza (void) ')', NULL); wocky_porter_start (test->sched_out); + wocky_porter_start (test->sched_in); /* Send a not jingle IQ */ iq = wocky_stanza_build (WOCKY_STANZA_TYPE_IQ, @@ -1175,7 +1176,7 @@ test_handler_stanza (void) ')', NULL); send_stanza (test, iq, TRUE); - test_close_porter (test); + test_close_both_porters (test); teardown_test (test); } |