← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/blackbox/test_aliases.py

  • Committer: Vincent Ladeuil
  • Date: 2009-04-27 16:10:10 UTC
  • mto: (4310.1.1 integration)
  • mto: This revision was merged to the branch mainline in revision 4311.
  • Revision ID: v.ladeuil+lp@free.fr-20090427161010-7swfzeagf63cpixd
Fix bug #367726 by reverting some default user handling introduced
while fixing bug #256612.

* bzrlib/transport/ssh.py:
(_paramiko_auth): Explicitly use getpass.getuser() as default
user.

* bzrlib/transport/ftp/_gssapi.py:
(GSSAPIFtpTransport._create_connection): Explicitly use
getpass.getuser() as default user.

* bzrlib/transport/ftp/__init__.py:
(FtpTransport._create_connection): Explicitly use
getpass.getuser() as default user.

* bzrlib/tests/test_sftp_transport.py:
(TestUsesAuthConfig.test_sftp_is_none_if_no_config)
(TestUsesAuthConfig.test_sftp_doesnt_prompt_username): Revert to
None as the default user.

* bzrlib/tests/test_remote.py:
(TestRemoteSSHTransportAuthentication): The really offending one:
revert to None as the default user.

* bzrlib/tests/test_config.py:
(TestAuthenticationConfig.test_username_default_no_prompt): Update
test (and some PEP8).

* bzrlib/smtp_connection.py:
(SMTPConnection._authenticate): Revert to None as the default
user.

* bzrlib/plugins/launchpad/account.py:
(_get_auth_user): Revert default value handling.

* bzrlib/config.py:
(AuthenticationConfig.get_user): Fix doc-string. Leave default
value handling to callers.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_aliases.py
1
 
# Copyright (C) 2006, 2007, 2009, 2010 Canonical Ltd
 
1
# Copyright (C) 2005, 2006 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
18
18
 
19
19
import os
20
20
 
21
 
from bzrlib import config
22
21
from bzrlib.branch import Branch
23
 
from bzrlib.tests import TestCaseWithTransport
 
22
from bzrlib.tests.blackbox import ExternalBase
24
23
from bzrlib.trace import mutter
25
 
 
26
 
 
27
 
class TestAliases(TestCaseWithTransport):
 
24
from bzrlib.config import (ensure_config_dir_exists, config_filename)
 
25
 
 
26
 
 
27
class TestAliases(ExternalBase):
28
28
 
29
29
    def test_aliases(self):
30
30
 
35
35
            return self.run_bzr(args, **kwargs)[1]
36
36
 
37
37
 
38
 
        conf = config.GlobalConfig.from_string('''[ALIASES]
39
 
c=cat
40
 
c1=cat -r 1
41
 
c2=cat -r 1 -r2
42
 
''', save=True)
 
38
        if os.path.isfile(config_filename()):
 
39
            # Something is wrong in environment,
 
40
            # we risk overwriting users config
 
41
            self.assert_(config_filename() + "exists, abort")
 
42
 
 
43
        ensure_config_dir_exists()
 
44
        CONFIG=("[ALIASES]\n"
 
45
                "c=cat\n"
 
46
                "c1=cat -r 1\n"
 
47
                "c2=cat -r 1 -r2\n")
 
48
 
 
49
        open(config_filename(),'wb').write(CONFIG)
43
50
 
44
51
        str1 = 'foo\n'
45
52
        str2 = 'bar\n'