[merge] FTP now has and passes regression tests
John Arbash Meinel
john at arbash-meinel.com
Tue May 16 22:39:46 BST 2006
Attached is the diff for review. Branch available from:
http://bzr.arbash-meinel.com/branches/bzr/ftp-server/
Basically, I check to see if 'medusa' is installed, and if so, I create
a FtpServer which can be used for testing the FtpTransport.
It turns out that we have to do a lot of semi-bogus error string
parsing, in order to satisfy all of Transport's requirements. For
example Transport.has() on a directory. We don't use it, because http
gives weird responses for it. But it does exist in the test suite (as
part of the mkdir() tests).
Anyway, I would guess that our FtpTransport won't give proper exceptions
when coupled with just any ftp server, since it is parsing the server's
error message. But hopefully that won't matter too much. (And people can
give us bug reports when it doesn't work with *their* ftp server).
So mostly what I did was clean up exception handling, and merge the last
of the append() changes.
If anyone actually using ftp could give it a try, I would appreciate it.
John
=:->
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ftp-server-test.diff
Url: https://lists.ubuntu.com/archives/bazaar/attachments/20060516/38b9b81b/attachment.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 254 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060516/38b9b81b/attachment.pgp
More information about the bazaar
mailing list