~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

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

  • Committer: Jelmer Vernooij
  • Date: 2011-12-16 19:18:39 UTC
  • mto: This revision was merged to the branch mainline in revision 6391.
  • Revision ID: jelmer@samba.org-20111216191839-eg681lxqibi1qxu1
Fix remaining tests.

Show diffs side-by-side

added added

removed removed

Lines of Context:
17
17
 
18
18
"""Black-box tests for bzr config."""
19
19
 
20
 
import os
21
 
 
22
20
from bzrlib import (
23
21
    config,
24
 
    errors,
25
22
    tests,
26
23
    )
27
24
from bzrlib.tests import (
28
25
    script,
29
26
    test_config as _t_config,
30
27
    )
 
28
from bzrlib.tests.matchers import ContainsNoVfsCalls
 
29
 
31
30
 
32
31
class TestWithoutConfig(tests.TestCaseWithTransport):
33
32
 
34
 
    def test_no_config(self):
 
33
    def test_config_all(self):
35
34
        out, err = self.run_bzr(['config'])
36
35
        self.assertEquals('', out)
37
36
        self.assertEquals('', err)
38
37
 
39
 
    def test_all_variables_no_config(self):
40
 
        out, err = self.run_bzr(['config', '*'])
41
 
        self.assertEquals('', out)
42
 
        self.assertEquals('', err)
 
38
    def test_remove_unknown_option(self):
 
39
        self.run_bzr_error(['The "file" configuration option does not exist',],
 
40
                           ['config', '--remove', 'file'])
 
41
 
 
42
    def test_all_remove_exclusive(self):
 
43
        self.run_bzr_error(['--all and --remove are mutually exclusive.',],
 
44
                           ['config', '--remove', '--all'])
 
45
 
 
46
    def test_all_set_exclusive(self):
 
47
        self.run_bzr_error(['Only one option can be set.',],
 
48
                           ['config', '--all', 'hello=world'])
 
49
 
 
50
    def test_remove_no_option(self):
 
51
        self.run_bzr_error(['--remove expects an option to remove.',],
 
52
                           ['config', '--remove'])
43
53
 
44
54
    def test_unknown_option(self):
45
55
        self.run_bzr_error(['The "file" configuration option does not exist',],
46
 
                           ['config', '--remove', 'file'])
 
56
                           ['config', 'file'])
 
57
 
 
58
    def test_unexpected_regexp(self):
 
59
        self.run_bzr_error(
 
60
            ['The "\*file" configuration option does not exist',],
 
61
            ['config', '*file'])
 
62
 
 
63
    def test_wrong_regexp(self):
 
64
        self.run_bzr_error(
 
65
            ['Invalid pattern\(s\) found. "\*file" nothing to repeat',],
 
66
            ['config', '--all', '*file'])
 
67
 
 
68
 
47
69
 
48
70
class TestConfigDisplay(tests.TestCaseWithTransport):
49
71
 
51
73
        super(TestConfigDisplay, self).setUp()
52
74
        _t_config.create_configs(self)
53
75
 
 
76
    def test_multiline_all_values(self):
 
77
        self.bazaar_config.set_user_option('multiline', '1\n2\n')
 
78
        # Fallout from bug 710410, the triple quotes have been toggled
 
79
        script.run_script(self, '''\
 
80
            $ bzr config -d tree
 
81
            bazaar:
 
82
              multiline = """1
 
83
            2
 
84
            """
 
85
            ''')
 
86
 
 
87
    def test_multiline_value_only(self):
 
88
        self.bazaar_config.set_user_option('multiline', '1\n2\n')
 
89
        # Fallout from bug 710410, the triple quotes have been toggled
 
90
        script.run_script(self, '''\
 
91
            $ bzr config -d tree multiline
 
92
            """1
 
93
            2
 
94
            """
 
95
            ''')
 
96
 
 
97
    def test_list_all_values(self):
 
98
        # FIXME: we should register the option as a list or it's displayed as
 
99
        # astring and as such, quoted.
 
100
        self.bazaar_config.set_user_option('list', [1, 'a', 'with, a comma'])
 
101
        script.run_script(self, '''\
 
102
            $ bzr config -d tree
 
103
            bazaar:
 
104
              list = '1, a, "with, a comma"'
 
105
            ''')
 
106
 
 
107
    def test_list_value_only(self):
 
108
        # FIXME: we should register the option as a list or it's displayed as
 
109
        # astring and as such, quoted.
 
110
        self.bazaar_config.set_user_option('list', [1, 'a', 'with, a comma'])
 
111
        script.run_script(self, '''\
 
112
            $ bzr config -d tree list
 
113
            '1, a, "with, a comma"'
 
114
            ''')
 
115
 
54
116
    def test_bazaar_config(self):
55
117
        self.bazaar_config.set_user_option('hello', 'world')
56
118
        script.run_script(self, '''\
65
127
        script.run_script(self, '''\
66
128
            $ bzr config -d tree
67
129
            locations:
 
130
              [.../tree]
68
131
              hello = world
69
132
            branch:
70
133
              hello = you
79
142
              hello = world
80
143
            ''')
81
144
 
 
145
class TestConfigDisplayWithPolicy(tests.TestCaseWithTransport):
 
146
 
 
147
    def test_location_with_policy(self):
 
148
        # LocationConfig is the only one dealing with policies so far.
 
149
        self.make_branch_and_tree('tree')
 
150
        config_text = """\
 
151
[%(dir)s]
 
152
url = dir
 
153
url:policy = appendpath
 
154
[%(dir)s/tree]
 
155
url = tree
 
156
""" % {'dir': self.test_dir}
 
157
        # We don't use the config directly so we save it to disk
 
158
        config.LocationConfig.from_string(config_text, 'tree', save=True)
 
159
        # policies are displayed with their options since they are part of
 
160
        # their definition, likewise the path is not appended, we are just
 
161
        # presenting the relevant portions of the config files
 
162
        script.run_script(self, '''\
 
163
            $ bzr config -d tree --all url
 
164
            locations:
 
165
              [.../work/tree]
 
166
              url = tree
 
167
              [.../work]
 
168
              url = dir
 
169
              url:policy = appendpath
 
170
            ''')
 
171
 
 
172
 
 
173
class TestConfigActive(tests.TestCaseWithTransport):
 
174
 
 
175
    def setUp(self):
 
176
        super(TestConfigActive, self).setUp()
 
177
        _t_config.create_configs_with_file_option(self)
 
178
 
 
179
    def test_active_in_locations(self):
 
180
        script.run_script(self, '''\
 
181
            $ bzr config -d tree file
 
182
            locations
 
183
            ''')
 
184
 
 
185
    def test_active_in_bazaar(self):
 
186
        script.run_script(self, '''\
 
187
            $ bzr config -d tree --scope bazaar file
 
188
            bazaar
 
189
            ''')
 
190
 
 
191
    def test_active_in_branch(self):
 
192
        # We need to delete the locations definition that overrides the branch
 
193
        # one
 
194
        script.run_script(self, '''\
 
195
            $ bzr config -d tree --scope locations --remove file
 
196
            $ bzr config -d tree file
 
197
            branch
 
198
            ''')
 
199
 
82
200
 
83
201
class TestConfigSetOption(tests.TestCaseWithTransport):
84
202
 
93
211
    def test_bazaar_config_outside_branch(self):
94
212
        script.run_script(self, '''\
95
213
            $ bzr config --scope bazaar hello=world
96
 
            $ bzr config -d tree hello
 
214
            $ bzr config -d tree --all hello
97
215
            bazaar:
98
216
              hello = world
99
217
            ''')
101
219
    def test_bazaar_config_inside_branch(self):
102
220
        script.run_script(self, '''\
103
221
            $ bzr config -d tree --scope bazaar hello=world
104
 
            $ bzr config -d tree hello
 
222
            $ bzr config -d tree --all hello
105
223
            bazaar:
106
224
              hello = world
107
225
            ''')
109
227
    def test_locations_config_inside_branch(self):
110
228
        script.run_script(self, '''\
111
229
            $ bzr config -d tree --scope locations hello=world
112
 
            $ bzr config -d tree hello
 
230
            $ bzr config -d tree --all hello
113
231
            locations:
 
232
              [.../work/tree]
114
233
              hello = world
115
234
            ''')
116
235
 
117
236
    def test_branch_config_default(self):
118
237
        script.run_script(self, '''\
119
238
            $ bzr config -d tree hello=world
120
 
            $ bzr config -d tree hello
 
239
            $ bzr config -d tree --all hello
121
240
            branch:
122
241
              hello = world
123
242
            ''')
125
244
    def test_branch_config_forcing_branch(self):
126
245
        script.run_script(self, '''\
127
246
            $ bzr config -d tree --scope branch hello=world
128
 
            $ bzr config -d tree hello
 
247
            $ bzr config -d tree --all hello
129
248
            branch:
130
249
              hello = world
131
250
            ''')
144
263
    def test_bazaar_config_outside_branch(self):
145
264
        script.run_script(self, '''\
146
265
            $ bzr config --scope bazaar --remove file
147
 
            $ bzr config -d tree file
 
266
            $ bzr config -d tree --all file
148
267
            locations:
 
268
              [.../work/tree]
149
269
              file = locations
150
270
            branch:
151
271
              file = branch
154
274
    def test_bazaar_config_inside_branch(self):
155
275
        script.run_script(self, '''\
156
276
            $ bzr config -d tree --scope bazaar --remove file
157
 
            $ bzr config -d tree file
 
277
            $ bzr config -d tree --all file
158
278
            locations:
 
279
              [.../work/tree]
159
280
              file = locations
160
281
            branch:
161
282
              file = branch
164
285
    def test_locations_config_inside_branch(self):
165
286
        script.run_script(self, '''\
166
287
            $ bzr config -d tree --scope locations --remove file
167
 
            $ bzr config -d tree file
 
288
            $ bzr config -d tree --all file
168
289
            branch:
169
290
              file = branch
170
291
            bazaar:
173
294
 
174
295
    def test_branch_config_default(self):
175
296
        script.run_script(self, '''\
176
 
            $ bzr config -d tree --remove file
177
 
            $ bzr config -d tree file
 
297
            $ bzr config -d tree --scope locations --remove file
 
298
            $ bzr config -d tree --all file
178
299
            branch:
179
300
              file = branch
180
301
            bazaar:
182
303
            ''')
183
304
        script.run_script(self, '''\
184
305
            $ bzr config -d tree --remove file
185
 
            $ bzr config -d tree file
 
306
            $ bzr config -d tree --all file
186
307
            bazaar:
187
308
              file = bazaar
188
309
            ''')
190
311
    def test_branch_config_forcing_branch(self):
191
312
        script.run_script(self, '''\
192
313
            $ bzr config -d tree --scope branch --remove file
193
 
            $ bzr config -d tree file
 
314
            $ bzr config -d tree --all file
194
315
            locations:
 
316
              [.../work/tree]
195
317
              file = locations
196
318
            bazaar:
197
319
              file = bazaar
198
320
            ''')
199
321
        script.run_script(self, '''\
200
 
            $ bzr config -d tree --remove file
201
 
            $ bzr config -d tree file
 
322
            $ bzr config -d tree --scope locations --remove file
 
323
            $ bzr config -d tree --all file
202
324
            bazaar:
203
325
              file = bazaar
204
326
            ''')
 
327
 
 
328
 
 
329
class TestSmartServerConfig(tests.TestCaseWithTransport):
 
330
 
 
331
    def test_simple_branch_config(self):
 
332
        self.setup_smart_server_with_call_log()
 
333
        t = self.make_branch_and_tree('branch')
 
334
        self.reset_smart_call_log()
 
335
        out, err = self.run_bzr(['config', '-d', self.get_url('branch')])
 
336
        # This figure represent the amount of work to perform this use case. It
 
337
        # is entirely ok to reduce this number if a test fails due to rpc_count
 
338
        # being too low. If rpc_count increases, more network roundtrips have
 
339
        # become necessary for this use case. Please do not adjust this number
 
340
        # upwards without agreement from bzr's network support maintainers.
 
341
        self.assertLength(5, self.hpss_calls)
 
342
        self.assertThat(self.hpss_calls, ContainsNoVfsCalls)