← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_msgeditor.py

  • Committer: Canonical.com Patch Queue Manager
  • Date: 2010-09-01 08:02:42 UTC
  • mfrom: (5390.3.3 faster-revert-593560)
  • Revision ID: pqm@pqm.ubuntu.com-20100901080242-esg62ody4frwmy66
(spiv) Avoid repeatedly calling self.target.all_file_ids() in
 InterTree.iter_changes. (Andrew Bennetts)

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_msgeditor.py
1
 
# Copyright (C) 2005-2011 Canonical Ltd
 
1
# Copyright (C) 2005-2010 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
26
26
    errors,
27
27
    msgeditor,
28
28
    osutils,
 
29
    tests,
29
30
    trace,
30
31
    )
 
32
from bzrlib.branch import Branch
 
33
from bzrlib.config import ensure_config_dir_exists, config_filename
31
34
from bzrlib.msgeditor import (
32
35
    make_commit_message_template_encoded,
33
36
    edit_commit_message_encoded
34
37
)
35
38
from bzrlib.tests import (
36
 
    features,
37
39
    TestCaseInTempDir,
38
40
    TestCaseWithTransport,
39
41
    TestNotApplicable,
80
82
""")
81
83
 
82
84
    def make_multiple_pending_tree(self):
83
 
        config.GlobalStack().set('email', 'Bilbo Baggins <bb@hobbit.net>')
 
85
        from bzrlib import config
 
86
        config.GlobalConfig().set_user_option('email',
 
87
                                              'Bilbo Baggins <bb@hobbit.net>')
84
88
        tree = self.make_branch_and_tree('a')
85
89
        tree.commit('Initial checkin.', timestamp=1230912900, timezone=0)
86
90
        tree2 = tree.bzrdir.clone('b').open_workingtree()
139
143
  hell\u00d8
140
144
""".encode('utf8') in template)
141
145
 
142
 
    def make_do_nothing_editor(self, basename='fed'):
 
146
    def make_do_nothing_editor(self):
143
147
        if sys.platform == "win32":
144
 
            name = basename + '.bat'
145
 
            f = file(name, 'w')
 
148
            f = file('fed.bat', 'w')
146
149
            f.write('@rem dummy fed')
147
150
            f.close()
148
 
            return name
 
151
            return 'fed.bat'
149
152
        else:
150
 
            name = basename + '.sh'
151
 
            f = file(name, 'wb')
 
153
            f = file('fed.sh', 'wb')
152
154
            f.write('#!/bin/sh\n')
153
155
            f.close()
154
 
            os.chmod(name, 0755)
155
 
            return './' + name
 
156
            os.chmod('fed.sh', 0755)
 
157
            return './fed.sh'
156
158
 
157
159
    def test_run_editor(self):
158
 
        self.overrideEnv('BZR_EDITOR', self.make_do_nothing_editor())
 
160
        os.environ['BZR_EDITOR'] = self.make_do_nothing_editor()
159
161
        self.assertEqual(True, msgeditor._run_editor(''),
160
162
                         'Unable to run dummy fake editor')
161
163
 
162
 
    def test_parse_editor_name(self):
163
 
        """Correctly interpret names with spaces.
164
 
 
165
 
        See <https://bugs.launchpad.net/bzr/+bug/220331>
166
 
        """
167
 
        self.overrideEnv('BZR_EDITOR',
168
 
            '"%s"' % self.make_do_nothing_editor('name with spaces'))
169
 
        self.assertEqual(True, msgeditor._run_editor('a_filename'))    
170
 
 
171
164
    def make_fake_editor(self, message='test message from fed\\n'):
172
165
        """Set up environment so that an editor will be a known script.
173
166
 
198
191
"%s" fed.py %%1
199
192
""" % sys.executable)
200
193
            f.close()
201
 
            self.overrideEnv('BZR_EDITOR', 'fed.bat')
 
194
            os.environ['BZR_EDITOR'] = 'fed.bat'
202
195
        else:
203
196
            # [non-win32] make python script executable and set BZR_EDITOR
204
197
            os.chmod('fed.py', 0755)
205
 
            self.overrideEnv('BZR_EDITOR', './fed.py')
 
198
            os.environ['BZR_EDITOR'] = './fed.py'
206
199
 
207
200
    def test_edit_commit_message(self):
208
201
        working_tree = self.make_uncommitted_tree()
237
230
        working_tree = self.make_uncommitted_tree()
238
231
 
239
232
        if sys.platform == 'win32':
240
 
            editor = 'cmd.exe /c del'
 
233
            os.environ['BZR_EDITOR'] = 'cmd.exe /c del'
241
234
        else:
242
 
            editor = 'rm'
243
 
        self.overrideEnv('BZR_EDITOR', editor)
 
235
            os.environ['BZR_EDITOR'] = 'rm'
244
236
 
245
237
        self.assertRaises((IOError, OSError), msgeditor.edit_commit_message, '')
246
238
 
247
239
    def test__get_editor(self):
248
 
        self.overrideEnv('BZR_EDITOR', 'bzr_editor')
249
 
        self.overrideEnv('VISUAL', 'visual')
250
 
        self.overrideEnv('EDITOR', 'editor')
251
 
 
252
 
        conf = config.GlobalStack()
253
 
        conf.store._load_from_string('[DEFAULT]\neditor = config_editor\n')
254
 
        conf.store.save()
 
240
        os.environ['BZR_EDITOR'] = 'bzr_editor'
 
241
        os.environ['VISUAL'] = 'visual'
 
242
        os.environ['EDITOR'] = 'editor'
 
243
 
 
244
        conf = config.GlobalConfig.from_string('editor = config_editor\n',
 
245
                                               save=True)
 
246
 
255
247
        editors = list(msgeditor._get_editor())
256
248
        editors = [editor for (editor, cfg_src) in editors]
257
249
 
267
259
 
268
260
    def test__run_editor_EACCES(self):
269
261
        """If running a configured editor raises EACESS, the user is warned."""
270
 
        self.overrideEnv('BZR_EDITOR', 'eacces.py')
 
262
        os.environ['BZR_EDITOR'] = 'eacces.py'
271
263
        f = file('eacces.py', 'wb')
272
264
        f.write('# Not a real editor')
273
265
        f.close()
275
267
        os.chmod('eacces.py', 0)
276
268
        # Set $EDITOR so that _run_editor will terminate before trying real
277
269
        # editors.
278
 
        self.overrideEnv('EDITOR', self.make_do_nothing_editor())
 
270
        os.environ['EDITOR'] = self.make_do_nothing_editor()
279
271
        # Call _run_editor, capturing mutter.warning calls.
280
272
        warnings = []
281
273
        def warning(*args):
307
299
        self.assertFileEqual(expected, msgfilename)
308
300
 
309
301
    def test__create_temp_file_with_commit_template_in_unicode_dir(self):
310
 
        self.requireFeature(features.UnicodeFilenameFeature)
 
302
        self.requireFeature(tests.UnicodeFilenameFeature)
311
303
        if hasattr(self, 'info'):
312
 
            tmpdir = self.info['directory']
313
 
            os.mkdir(tmpdir)
314
 
            # Force the creation of temp file in a directory whose name
315
 
            # requires some encoding support
316
 
            msgeditor._create_temp_file_with_commit_template('infotext',
317
 
                                                             tmpdir=tmpdir)
 
304
            os.mkdir(self.info['directory'])
 
305
            os.chdir(self.info['directory'])
 
306
            msgeditor._create_temp_file_with_commit_template('infotext')
318
307
        else:
319
308
            raise TestNotApplicable('Test run elsewhere with non-ascii data.')
320
309
 
327
316
        self.assertFileEqual('', msgfilename)
328
317
 
329
318
    def test_unsupported_encoding_commit_message(self):
330
 
        self.overrideEnv('LANG', 'C')
331
 
        # LANG env variable has no effect on Windows
332
 
        # but some characters anyway cannot be represented
333
 
        # in default user encoding
334
 
        char = probe_bad_non_ascii(osutils.get_user_encoding())
335
 
        if char is None:
336
 
            raise TestSkipped('Cannot find suitable non-ascii character '
337
 
                'for user_encoding (%s)' % osutils.get_user_encoding())
338
 
 
339
 
        self.make_fake_editor(message=char)
340
 
 
341
 
        working_tree = self.make_uncommitted_tree()
342
 
        self.assertRaises(errors.BadCommitMessageEncoding,
343
 
                          msgeditor.edit_commit_message, '')
344
 
 
345
 
    def test_set_commit_message_no_hooks(self):
346
 
        commit_obj = commit.Commit()
347
 
        self.assertIs(None,
348
 
            msgeditor.set_commit_message(commit_obj))
349
 
 
350
 
    def test_set_commit_message_hook(self):
351
 
        msgeditor.hooks.install_named_hook("set_commit_message",
352
 
                lambda commit_obj, existing_message: "save me some typing\n", None)
353
 
        commit_obj = commit.Commit()
354
 
        self.assertEquals("save me some typing\n",
355
 
            msgeditor.set_commit_message(commit_obj))
 
319
        old_env = osutils.set_or_unset_env('LANG', 'C')
 
320
        try:
 
321
            # LANG env variable has no effect on Windows
 
322
            # but some characters anyway cannot be represented
 
323
            # in default user encoding
 
324
            char = probe_bad_non_ascii(osutils.get_user_encoding())
 
325
            if char is None:
 
326
                raise TestSkipped('Cannot find suitable non-ascii character '
 
327
                    'for user_encoding (%s)' % osutils.get_user_encoding())
 
328
 
 
329
            self.make_fake_editor(message=char)
 
330
 
 
331
            working_tree = self.make_uncommitted_tree()
 
332
            self.assertRaises(errors.BadCommitMessageEncoding,
 
333
                              msgeditor.edit_commit_message, '')
 
334
        finally:
 
335
            osutils.set_or_unset_env('LANG', old_env)
356
336
 
357
337
    def test_generate_commit_message_template_no_hooks(self):
358
338
        commit_obj = commit.Commit()
378
358
        ERROR_BAD_EXE_FORMAT = 193
379
359
        file("textfile.txt", "w").close()
380
360
        e = self.assertRaises(WindowsError, subprocess.call, "textfile.txt")
381
 
        self.assertEqual(e.errno, errno.ENOEXEC)
382
 
        self.assertEqual(e.winerror, ERROR_BAD_EXE_FORMAT)
 
361
        # Python2.4 used the 'winerror' as the errno, which confuses a lot of
 
362
        # our error trapping code. Make sure that we understand the mapping
 
363
        # correctly.
 
364
        if sys.version_info >= (2, 5):
 
365
            self.assertEqual(e.errno, errno.ENOEXEC)
 
366
            self.assertEqual(e.winerror, ERROR_BAD_EXE_FORMAT)
 
367
        else:
 
368
            self.assertEqual(e.errno, ERROR_BAD_EXE_FORMAT)