On 07:25 pm, itamar@itamarst.org wrote:
I'm working on tests for t.c.s.session.SSHSession, and it's difficult because of the terrible API SSHSession currently has. I've tried a rewrite in branches/session-2710, but I wanted to know if anyone was actually using that class directly? The typical use case is writing an Avatar->ISession adapter, and if you're doing it that way nothing should change for you even with this update.
Rather than removing it, you could e.g. deprecate the old class and then write a new one (using test-driven development) to ensure a decent API and good test coverage.
I tried a while ago to document some parts of the unspoken idioms around compatibility: http://twistedmatrix.com/trac/wiki/CompatibilityPolicy It's still not done, but hey, as long as we're talking about it, Paul, have a look at the number of entries Conch has in that list near the bottom. ;-)