[Pkg-telepathy-commits] [telepathy-mission-control-6] 43/280: already-has-obsolete test: remove

Simon McVittie smcv at debian.org
Thu Mar 27 20:07:03 UTC 2014


This is an automated email from the git hooks/post-receive script.

smcv pushed a commit to branch debian
in repository telepathy-mission-control-6.

commit f2eee77e70c9e165cb7ad2c86fdac52fcd9e8bb3
Author: Simon McVittie <simon.mcvittie at collabora.co.uk>
Date:   Thu Oct 3 17:50:19 2013 +0100

    already-has-obsolete test: remove
    
    This didn't test what it was meant to test (see the next commit
    for details) which made it basically the same as dispatch-obsolete.
    In Telepathy 1.0 the "no Requests" case won't exist anyway.
    
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=54879
    Reviewed-by: Guillaume Desmottes <guillaume.desmottes at collabora.co.uk>
---
 tests/twisted/Makefile.am                        |   1 -
 tests/twisted/dispatcher/already-has-obsolete.py | 133 -----------------------
 2 files changed, 134 deletions(-)

diff --git a/tests/twisted/Makefile.am b/tests/twisted/Makefile.am
index e47c070..00e6729 100644
--- a/tests/twisted/Makefile.am
+++ b/tests/twisted/Makefile.am
@@ -27,7 +27,6 @@ TWISTED_BASIC_TESTS = \
 	account/addressing.py \
 	capabilities/contact-caps.py \
 	dispatcher/already-has-channel.py \
-	dispatcher/already-has-obsolete.py \
 	dispatcher/approver-fails.py \
 	dispatcher/bypass-approval.py \
 	dispatcher/bypass-observers.py \
diff --git a/tests/twisted/dispatcher/already-has-obsolete.py b/tests/twisted/dispatcher/already-has-obsolete.py
deleted file mode 100644
index 9eafa7c..0000000
--- a/tests/twisted/dispatcher/already-has-obsolete.py
+++ /dev/null
@@ -1,133 +0,0 @@
-# Copyright (C) 2009 Nokia Corporation
-# Copyright (C) 2009 Collabora Ltd.
-#
-# This library is free software; you can redistribute it and/or
-# modify it under the terms of the GNU Lesser General Public
-# License as published by the Free Software Foundation; either
-# version 2.1 of the License, or (at your option) any later version.
-#
-# This library is distributed in the hope that it will be useful, but
-# WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
-# Lesser General Public License for more details.
-#
-# You should have received a copy of the GNU Lesser General Public
-# License along with this library; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
-# 02110-1301 USA
-
-import dbus
-"""Regression test for dispatching an incoming Text channel that was already
-there before the Connection became ready.
-"""
-
-import dbus
-import dbus.service
-
-from servicetest import EventPattern, tp_name_prefix, tp_path_prefix, \
-        call_async
-from mctest import exec_test, SimulatedConnection, SimulatedClient, \
-        create_fakecm_account, SimulatedChannel, expect_client_setup
-import constants as cs
-
-def test(q, bus, mc):
-    params = dbus.Dictionary({"account": "someguy at example.com",
-        "password": "secrecy"}, signature='sv')
-    cm_name_ref, account = create_fakecm_account(q, bus, mc, params)
-
-    text_fixed_properties = dbus.Dictionary({
-        cs.CHANNEL + '.TargetHandleType': cs.HT_CONTACT,
-        cs.CHANNEL + '.ChannelType': cs.CHANNEL_TYPE_TEXT,
-        }, signature='sv')
-
-    # Two clients want to observe, approve and handle channels
-    empathy = SimulatedClient(q, bus, 'Empathy',
-            observe=[text_fixed_properties], approve=[text_fixed_properties],
-            handle=[text_fixed_properties], bypass_approval=False)
-    kopete = SimulatedClient(q, bus, 'Kopete',
-            observe=[text_fixed_properties], approve=[text_fixed_properties],
-            handle=[text_fixed_properties], bypass_approval=False)
-
-    # wait for MC to download the properties
-    expect_client_setup(q, [empathy, kopete])
-
-    # Enable the account
-    account.Set(cs.ACCOUNT, 'Enabled', True,
-            dbus_interface=cs.PROPERTIES_IFACE)
-
-    requested_presence = dbus.Struct((dbus.UInt32(2L),
-        dbus.String(u'available'), dbus.String(u'')))
-    account.Set(cs.ACCOUNT,
-            'RequestedPresence', requested_presence,
-            dbus_interface=cs.PROPERTIES_IFACE)
-
-    e = q.expect('dbus-method-call', method='RequestConnection',
-            args=['fakeprotocol', params],
-            destination=cs.tp_name_prefix + '.ConnectionManager.fakecm',
-            path=cs.tp_path_prefix + '/ConnectionManager/fakecm',
-            interface=cs.tp_name_prefix + '.ConnectionManager',
-            handled=False)
-
-    # Don't allow the Connection to become ready until we want it to, by
-    # avoiding a return from Get(Interfaces) or GetInterfaces.
-    conn = SimulatedConnection(q, bus, 'fakecm', 'fakeprotocol', '_',
-            'myself', implement_get_interfaces=False, has_requests=False)
-
-    q.dbus_return(e.message, conn.bus_name, conn.object_path, signature='so')
-
-    # This is the pre-Connect one, from Mission Control.
-    e = q.expect('dbus-method-call', method='Get',
-            args=[cs.CONN, 'Interfaces'],
-            path=conn.object_path, handled=False)
-    q.dbus_raise(e.message, cs.DISCONNECTED, 'Not connected yet')
-
-    q.expect('dbus-method-call', method='Connect',
-            path=conn.object_path, handled=True)
-    conn.StatusChanged(cs.CONN_STATUS_CONNECTED, cs.CSR_NONE_SPECIFIED)
-
-    # This is from TpConnection. It's an implementation detail that it calls
-    # GetInterfaces at this point; in 'next' it will call GetAll (but
-    # this test will be obsolete anyway, so it doesn't really matter).
-    get_interfaces_call = q.expect('dbus-method-call', method='GetInterfaces',
-            args=[],
-            path=conn.object_path, handled=False)
-
-    # subscribe to the OperationList interface (MC assumes that until this
-    # property has been retrieved once, nobody cares)
-
-    cd = bus.get_object(cs.CD, cs.CD_PATH)
-    cd_props = dbus.Interface(cd, cs.PROPERTIES_IFACE)
-    assert cd_props.Get(cs.CD_IFACE_OP_LIST, 'DispatchOperations') == []
-
-    # Before returning from Get(Interfaces), make a Channel spring into
-    # existence
-
-    channel_properties = dbus.Dictionary(text_fixed_properties,
-            signature='sv')
-    channel_properties[cs.CHANNEL + '.TargetID'] = 'juliet'
-    channel_properties[cs.CHANNEL + '.TargetHandle'] = \
-            conn.ensure_handle(cs.HT_CONTACT, 'juliet')
-    channel_properties[cs.CHANNEL + '.InitiatorID'] = 'juliet'
-    channel_properties[cs.CHANNEL + '.InitiatorHandle'] = \
-            conn.ensure_handle(cs.HT_CONTACT, 'juliet')
-    channel_properties[cs.CHANNEL + '.Requested'] = False
-    channel_properties[cs.CHANNEL + '.Interfaces'] = dbus.Array(signature='s')
-
-    chan = SimulatedChannel(conn, channel_properties)
-    chan.announce()
-
-    # Now reply to Get(Interfaces) and say we don't have Requests
-    conn.Get_Interfaces(get_interfaces_call)
-
-    # MC shoots down the connection. Goodbye!
-    q.expect_many(
-            EventPattern('dbus-signal', signal='AccountPropertyChanged',
-                predicate=lambda e:
-                    e.args[0].get('ConnectionError') ==
-                        cs.SOFTWARE_UPGRADE_REQUIRED),
-            EventPattern('dbus-method-call', method='Disconnect',
-                handled=True),
-            )
-
-if __name__ == '__main__':
-    exec_test(test, {})

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-telepathy/telepathy-mission-control-6.git



More information about the Pkg-telepathy-commits mailing list