← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

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

  • Committer: Vincent Ladeuil
  • Date: 2012-01-09 20:55:07 UTC
  • mto: This revision was merged to the branch mainline in revision 6468.
  • Revision ID: v.ladeuil+lp@free.fr-20120109205507-2i3en5r4w4ohdchj
Use idioms coherently and add comments to make their purpose clearer.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/blackbox/test_whoami.py
1
 
# Copyright (C) 2006 Canonical Ltd
 
1
# Copyright (C) 2006, 2007, 2009-2012 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
17
17
 
18
18
"""Black-box tests for bzr whoami."""
19
19
 
20
 
import os
21
 
 
22
20
import bzrlib
23
 
from bzrlib.branch import Branch
24
 
from bzrlib.tests.blackbox import ExternalBase
25
 
 
26
 
 
27
 
class TestWhoami(ExternalBase):
28
 
 
29
 
    def test_whoami(self):
 
21
from bzrlib import (
 
22
    branch,
 
23
    config,
 
24
    errors,
 
25
    tests,
 
26
    )
 
27
 
 
28
 
 
29
class TestWhoami(tests.TestCaseWithTransport):
 
30
 
 
31
    def assertWhoAmI(self, expected, *cmd_args, **kwargs):
 
32
        out, err = self.run_bzr(('whoami',) + cmd_args, **kwargs)
 
33
        self.assertEquals('', err)
 
34
        lines = out.splitlines()
 
35
        self.assertLength(1, lines)
 
36
        self.assertEquals(expected, lines[0].rstrip())
 
37
 
 
38
    def test_whoami_no_args_no_conf(self):
30
39
        # this should always identify something, if only "john@localhost"
31
40
        out = self.run_bzr("whoami")[0]
32
41
        self.assertTrue(len(out) > 0)
33
42
        self.assertEquals(1, out.count('@'))
34
43
 
 
44
    def test_whoami_email_no_args(self):
35
45
        out = self.run_bzr("whoami --email")[0]
36
46
        self.assertTrue(len(out) > 0)
37
47
        self.assertEquals(1, out.count('@'))
38
48
 
 
49
    def test_whoami_email_arg(self):
 
50
        # whoami --email is mutually exclusive with any arguments
 
51
        out = self.run_bzr("whoami --email 'foo <foo@example.com>'", 3)[0]
 
52
        self.assertEquals("", out)
 
53
 
 
54
    def set_branch_email(self, b, email):
 
55
        b.lock_write()
 
56
        try:
 
57
            b.get_config_stack().set('email', email)
 
58
        finally:
 
59
            b.unlock()
 
60
 
39
61
    def test_whoami_branch(self):
40
62
        """branch specific user identity works."""
41
63
        wt = self.make_branch_and_tree('.')
42
64
        b = bzrlib.branch.Branch.open('.')
43
 
        b.get_config().set_user_option('email',
44
 
                                       'Branch Identity <branch@identi.ty>')
45
 
        bzr_email = os.environ.get('BZR_EMAIL')
46
 
        if bzr_email is not None:
47
 
            del os.environ['BZR_EMAIL']
48
 
        try:
49
 
            whoami = self.run_bzr("whoami")[0]
50
 
            self.assertEquals('Branch Identity <branch@identi.ty>\n', whoami)
51
 
            whoami_email = self.run_bzr("whoami --email")[0]
52
 
            self.assertEquals('branch@identi.ty\n', whoami_email)
 
65
        self.set_branch_email(b, 'Branch Identity <branch@identi.ty>')
 
66
        self.assertWhoAmI('Branch Identity <branch@identi.ty>')
 
67
        self.assertWhoAmI('branch@identi.ty', '--email')
53
68
 
54
 
            # Verify that the environment variable overrides the value
55
 
            # in the file
56
 
            os.environ['BZR_EMAIL'] = 'Different ID <other@environ.ment>'
57
 
            whoami = self.run_bzr("whoami")[0]
58
 
            self.assertEquals('Different ID <other@environ.ment>\n', whoami)
59
 
            whoami_email = self.run_bzr("whoami --email")[0]
60
 
            self.assertEquals('other@environ.ment\n', whoami_email)
61
 
            del os.environ['BZR_EMAIL']
62
 
        finally:
63
 
            if bzr_email is not None:
64
 
                os.environ['BZR_EMAIL'] = bzr_email
 
69
        # Verify that the environment variable overrides the value
 
70
        # in the file
 
71
        self.overrideEnv('BZR_EMAIL', 'Different ID <other@environ.ment>')
 
72
        self.assertWhoAmI('Different ID <other@environ.ment>')
 
73
        self.assertWhoAmI('other@environ.ment', '--email')
65
74
 
66
75
    def test_whoami_utf8(self):
67
76
        """verify that an identity can be in utf-8."""
68
 
        wt = self.make_branch_and_tree('.')
69
77
        self.run_bzr(['whoami', u'Branch Identity \u20ac <branch@identi.ty>'],
70
78
                     encoding='utf-8')
71
 
        bzr_email = os.environ.get('BZR_EMAIL')
72
 
        if bzr_email is not None:
73
 
            del os.environ['BZR_EMAIL']
74
 
        try:
75
 
            whoami = self.run_bzr("whoami", encoding='utf-8')[0]
76
 
            self.assertEquals('Branch Identity \xe2\x82\xac ' +
77
 
                              '<branch@identi.ty>\n', whoami)
78
 
            whoami_email = self.run_bzr("whoami --email",
79
 
                                        encoding='utf-8')[0]
80
 
            self.assertEquals('branch@identi.ty\n', whoami_email)
81
 
        finally:
82
 
            if bzr_email is not None:
83
 
                os.environ['BZR_EMAIL'] = bzr_email
 
79
        self.assertWhoAmI('Branch Identity \xe2\x82\xac <branch@identi.ty>',
 
80
                          encoding='utf-8')
 
81
        self.assertWhoAmI('branch@identi.ty', '--email')
84
82
 
85
83
    def test_whoami_ascii(self):
86
84
        """
89
87
        """
90
88
        wt = self.make_branch_and_tree('.')
91
89
        b = bzrlib.branch.Branch.open('.')
92
 
        b.get_config().set_user_option('email', u'Branch Identity \u20ac ' +
93
 
                                       '<branch@identi.ty>')
94
 
        bzr_email = os.environ.get('BZR_EMAIL')
95
 
        if bzr_email is not None:
96
 
            del os.environ['BZR_EMAIL']
97
 
        try:
98
 
            whoami = self.run_bzr("whoami", encoding='ascii')[0]
99
 
            self.assertEquals('Branch Identity ? <branch@identi.ty>\n', whoami)
100
 
            whoami_email = self.run_bzr("whoami --email",
101
 
                                        encoding='ascii')[0]
102
 
            self.assertEquals('branch@identi.ty\n', whoami_email)
103
 
        finally:
104
 
            if bzr_email is not None:
105
 
                os.environ['BZR_EMAIL'] = bzr_email
 
90
        self.set_branch_email(b, u'Branch Identity \u20ac <branch@identi.ty>')
 
91
        self.assertWhoAmI('Branch Identity ? <branch@identi.ty>',
 
92
                          encoding='ascii')
 
93
        self.assertWhoAmI('branch@identi.ty', '--email',
 
94
                          encoding='ascii')
106
95
 
107
96
    def test_warning(self):
108
97
        """verify that a warning is displayed if no email is given."""
111
100
        self.assertEquals('"Branch Identity" does not seem to contain an '
112
101
                          'email address.  This is allowed, but not '
113
102
                          'recommended.\n', display)
 
103
 
 
104
    def test_whoami_not_set(self):
 
105
        """Ensure whoami error if username is not set and not inferred.
 
106
        """
 
107
        self.overrideEnv('EMAIL', None)
 
108
        self.overrideEnv('BZR_EMAIL', None)
 
109
        # Also, make sure that it's not inferred from mailname.
 
110
        self.overrideAttr(config, '_auto_user_id', lambda: (None, None))
 
111
        out, err = self.run_bzr(['whoami'], 3)
 
112
        self.assertContainsRe(err, 'Unable to determine your name')
 
113
 
 
114
    def test_whoami_directory(self):
 
115
        """Test --directory option."""
 
116
        wt = self.make_branch_and_tree('subdir')
 
117
        self.set_branch_email(wt.branch, 'Branch Identity <branch@identi.ty>')
 
118
        self.assertWhoAmI('Branch Identity <branch@identi.ty>',
 
119
                          '--directory', 'subdir')
 
120
        self.run_bzr(['whoami', '--directory', 'subdir', '--branch',
 
121
                      'Changed Identity <changed@identi.ty>'])
 
122
        # Refresh wt as 'whoami' modified it
 
123
        wt = wt.bzrdir.open_workingtree()
 
124
        c = wt.branch.get_config_stack()
 
125
        self.assertEquals('Changed Identity <changed@identi.ty>',
 
126
                          c.get('email'))
 
127
 
 
128
    def test_whoami_remote_directory(self):
 
129
        """Test --directory option with a remote directory."""
 
130
        wt = self.make_branch_and_tree('subdir')
 
131
        self.set_branch_email(wt.branch, 'Branch Identity <branch@identi.ty>')
 
132
        url = self.get_readonly_url() + '/subdir'
 
133
        self.assertWhoAmI('Branch Identity <branch@identi.ty>',
 
134
                          '--directory', url)
 
135
        url = self.get_url('subdir')
 
136
        self.run_bzr(['whoami', '--directory', url, '--branch',
 
137
                      'Changed Identity <changed@identi.ty>'])
 
138
        # The identity has been set in the branch config (but not the global
 
139
        # config)
 
140
        c = branch.Branch.open(url).get_config_stack()
 
141
        self.assertEquals('Changed Identity <changed@identi.ty>',
 
142
                          c.get('email'))
 
143
        # Ensuring that the value does not come from the bazaar.conf file
 
144
        # itself requires some isolation setup
 
145
        self.overrideEnv('BZR_EMAIL', None)
 
146
        self.overrideEnv('EMAIL', None)
 
147
        self.overrideAttr(config, '_auto_user_id', lambda: (None, None))
 
148
        global_conf = config.GlobalStack()
 
149
        self.assertRaises(errors.NoWhoami, global_conf.get, 'email')
 
150
 
 
151
    def test_whoami_nonbranch_directory(self):
 
152
        """Test --directory mentioning a non-branch directory."""
 
153
        wt = self.build_tree(['subdir/'])
 
154
        out, err = self.run_bzr("whoami --directory subdir", retcode=3)
 
155
        self.assertContainsRe(err, 'ERROR: Not a branch')