test-roster-subscribe: assert that we don't send useless subscribe requests

1 files changed, 17 insertions, 0 deletions

diff --git a/tests/twisted/roster/test-roster-subscribe.py b/tests/twisted/roster/test-roster-subscribe.py
index 2a0c205c8..d389f6e18 100644
--- a/tests/twisted/roster/test-roster-subscribe.py
+++ b/tests/twisted/roster/test-roster-subscribe.py
@@ -88,6 +88,23 @@ def test(q, bus, conn, stream, modern=True):
                     }, []]),
             )
 
+    send_roster_push(stream, 'bob@foo.com', 'to')
+    q.expect('stream-iq', iq_type='result', iq_id='push')
+
+    # Doing the same again is a successful no-op
+    forbidden = [EventPattern('stream-iq', query_ns=ns.ROSTER),
+            EventPattern('stream-presence')]
+    sync_stream(q, stream)
+    sync_dbus(bus, q, conn)
+    q.forbid_events(forbidden)
+
+    call_async(q, conn.ContactList, 'RequestSubscription', [bob], 'moo')
+    q.expect('dbus-return', method='RequestSubscription')
+
+    sync_stream(q, stream)
+    sync_dbus(bus, q, conn)
+    q.unforbid_events(forbidden)
+
 if __name__ == '__main__':
     exec_test(test_ancient)
     exec_test(test_modern)