Merge lp:~mwhudson/launchpad/twisted-x into lp:launchpad

Proposed by Michael Hudson-Doyle
Status: Merged
Approved by: Tim Penhey
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp:~mwhudson/launchpad/twisted-x
Merge into: lp:launchpad
Diff against target: 110 lines (+4/-34)
6 files modified
lib/canonical/twistedsupport/tests/test_processmonitor.py (+0/-7)
lib/lp/buildmaster/tests/test_manager.py (+0/-5)
lib/lp/codehosting/codeimport/tests/test_workermonitor.py (+0/-5)
lib/lp/codehosting/puller/tests/test_scheduler.py (+0/-5)
lib/lp/codehosting/sshserver/service.py (+3/-11)
versions.cfg (+1/-1)
To merge this branch: bzr merge lp:~mwhudson/launchpad/twisted-x
Reviewer Review Type Date Requested Status
Tim Penhey (community) Approve
Review via email: mp+20611@code.launchpad.net

Commit message

Upgrade to Twisted 10.0.0

Description of the change

Let's use Twisted 10.

To post a comment you must log in.
Revision history for this message
Tim Penhey (thumper) wrote :

I love the RED in the diff :-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/canonical/twistedsupport/tests/test_processmonitor.py'
--- lib/canonical/twistedsupport/tests/test_processmonitor.py 2009-11-22 20:17:36 +0000
+++ lib/canonical/twistedsupport/tests/test_processmonitor.py 2010-03-10 01:30:47 +0000
@@ -318,13 +318,6 @@
318318
319 layer = TwistedLayer319 layer = TwistedLayer
320320
321 def setUp(self):
322 # XXX 2009-11-23, MichaelHudson,
323 # bug=http://twistedmatrix.com/trac/ticket/2078: This is a hack to
324 # make sure the reactor is running when the test method is executed to
325 # work around the linked Twisted bug.
326 return task.deferLater(reactor, 0, lambda: None)
327
328 def test_run_process_with_timeout_invalid_args(self):321 def test_run_process_with_timeout_invalid_args(self):
329 # `run_process_with_timeout` expects the process 'args' to be a322 # `run_process_with_timeout` expects the process 'args' to be a
330 # tuple.323 # tuple.
331324
=== modified file 'lib/lp/buildmaster/tests/test_manager.py'
--- lib/lp/buildmaster/tests/test_manager.py 2010-03-06 00:32:52 +0000
+++ lib/lp/buildmaster/tests/test_manager.py 2010-03-10 01:30:47 +0000
@@ -43,11 +43,6 @@
43 TrialTestCase.setUp(self)43 TrialTestCase.setUp(self)
44 self.slave = RecordingSlave(44 self.slave = RecordingSlave(
45 'foo', 'http://foo:8221/rpc', 'foo.host')45 'foo', 'http://foo:8221/rpc', 'foo.host')
46 # XXX 2009-11-23, MichaelHudson,
47 # bug=http://twistedmatrix.com/trac/ticket/2078: This is a hack to
48 # make sure the reactor is running when the test method is executed to
49 # work around the linked Twisted bug.
50 return task.deferLater(reactor, 0, lambda: None)
5146
52 def test_representation(self):47 def test_representation(self):
53 """`RecordingSlave` has a custom representation.48 """`RecordingSlave` has a custom representation.
5449
=== modified file 'lib/lp/codehosting/codeimport/tests/test_workermonitor.py'
--- lib/lp/codehosting/codeimport/tests/test_workermonitor.py 2010-02-24 10:18:16 +0000
+++ lib/lp/codehosting/codeimport/tests/test_workermonitor.py 2010-03-10 01:30:47 +0000
@@ -695,11 +695,6 @@
695 def setUp(self):695 def setUp(self):
696 TestWorkerMonitorIntegration.setUp(self)696 TestWorkerMonitorIntegration.setUp(self)
697 self._protocol = None697 self._protocol = None
698 # XXX 2009-11-23, MichaelHudson,
699 # bug=http://twistedmatrix.com/trac/ticket/2078: This is a hack to
700 # make sure the reactor is running when the test method is executed to
701 # work around the linked Twisted bug.
702 return task.deferLater(reactor, 0, lambda: None)
703698
704 def performImport(self, job_id):699 def performImport(self, job_id):
705 """Perform the import job with ID job_id.700 """Perform the import job with ID job_id.
706701
=== modified file 'lib/lp/codehosting/puller/tests/test_scheduler.py'
--- lib/lp/codehosting/puller/tests/test_scheduler.py 2010-02-24 04:24:01 +0000
+++ lib/lp/codehosting/puller/tests/test_scheduler.py 2010-03-10 01:30:47 +0000
@@ -662,11 +662,6 @@
662 self.bzr_tree.commit('rev1')662 self.bzr_tree.commit('rev1')
663 self.pushToBranch(self.db_branch, self.bzr_tree)663 self.pushToBranch(self.db_branch, self.bzr_tree)
664 self.client = FakePullerEndpointProxy()664 self.client = FakePullerEndpointProxy()
665 # XXX 2009-11-23, MichaelHudson,
666 # bug=http://twistedmatrix.com/trac/ticket/2078: This is a hack to
667 # make sure the reactor is running when the test method is executed to
668 # work around the linked Twisted bug.
669 return task.deferLater(reactor, 0, lambda: None)
670665
671 def run(self, result):666 def run(self, result):
672 # We want to use Trial's run() method so we can return Deferreds.667 # We want to use Trial's run() method so we can return Deferreds.
673668
=== modified file 'lib/lp/codehosting/sshserver/service.py'
--- lib/lp/codehosting/sshserver/service.py 2009-06-25 04:06:00 +0000
+++ lib/lp/codehosting/sshserver/service.py 2010-03-10 01:30:47 +0000
@@ -46,6 +46,8 @@
46 to and disconnection from the SSH server.46 to and disconnection from the SSH server.
47 """47 """
4848
49 protocol = KeepAliveSettingSSHServerTransport
50
49 def __init__(self, portal):51 def __init__(self, portal):
50 # Although 'portal' isn't part of the defined interface for52 # Although 'portal' isn't part of the defined interface for
51 # `SSHFactory`, defining it here is how the `SSHUserAuthServer` gets53 # `SSHFactory`, defining it here is how the `SSHUserAuthServer` gets
@@ -60,17 +62,7 @@
60 The protocol object we return is slightly modified so that we can hook62 The protocol object we return is slightly modified so that we can hook
61 into the 'connectionLost' event and log the disconnection.63 into the 'connectionLost' event and log the disconnection.
62 """64 """
63 # If Conch let us customize the protocol class, we wouldn't need this.65 transport = SSHFactory.buildProtocol(self, address)
64 # See http://twistedmatrix.com/trac/ticket/3443.
65 transport = KeepAliveSettingSSHServerTransport()
66 transport.supportedPublicKeys = self.privateKeys.keys()
67 if not self.primes:
68 log.msg('disabling diffie-hellman-group-exchange because we '
69 'cannot find moduli file')
70 ske = transport.supportedKeyExchanges[:]
71 ske.remove('diffie-hellman-group-exchange-sha1')
72 transport.supportedKeyExchanges = ske
73 transport.factory = self
74 transport._realConnectionLost = transport.connectionLost66 transport._realConnectionLost = transport.connectionLost
75 transport.connectionLost = (67 transport.connectionLost = (
76 lambda reason: self.connectionLost(transport, reason))68 lambda reason: self.connectionLost(transport, reason))
7769
=== modified file 'versions.cfg'
--- versions.cfg 2010-03-02 17:24:50 +0000
+++ versions.cfg 2010-03-10 01:30:47 +0000
@@ -61,7 +61,7 @@
61storm = 0.15danilo-storm-launchpad-r34261storm = 0.15danilo-storm-launchpad-r342
62testtools = 0.9.262testtools = 0.9.2
63transaction = 1.0.063transaction = 1.0.0
64Twisted = 9.0.064Twisted = 10.0.0
65uuid = 1.3065uuid = 1.30
66van.testing = 2.0.166van.testing = 2.0.1
67wadllib = 1.1.467wadllib = 1.1.4