/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/tests/ftp_server.py

  • Committer: Martin Pool
  • Date: 2008-04-24 07:22:53 UTC
  • mto: This revision was merged to the branch mainline in revision 3415.
  • Revision ID: mbp@sourcefrog.net-20080424072253-opmjij7xfy38w27f
Remove every assert statement from bzrlib!

Depending on the context they are:

 * turned into an explicit if/raise of either AssertionError 
   or something more specific -- particularly where they protect
   programming interfaces, complex invariants, or data file integrity
 * removed, if they're redundant with a later check, not protecting
   a meaningful invariant
 * turned into a selftest method on tests

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/ftp_server.py
201
201
 
202
202
    def setUp(self, vfs_server=None):
203
203
        from bzrlib.transport.local import LocalURLServer
204
 
        assert vfs_server is None or isinstance(vfs_server, LocalURLServer), \
205
 
            "FTPServer currently assumes local transport, got %s" % vfs_server
206
 
 
 
204
        if not (vfs_server is None or isinstance(vfs_server, LocalURLServer)):
 
205
            raise AssertionError(
 
206
                "FTPServer currently assumes local transport, got %s" % vfs_server)
207
207
        self._root = os.getcwdu()
208
208
        self._ftp_server = ftp_server(
209
209
            authorizer=test_authorizer(root=self._root),