← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_msgeditor.py

Merge bzr.dev into cleanup

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
22
22
 
23
23
from bzrlib import (
24
24
    commit,
25
 
    config,
26
25
    errors,
27
26
    msgeditor,
28
27
    osutils,
29
28
    tests,
30
29
    trace,
31
30
    )
 
31
from bzrlib.branch import Branch
 
32
from bzrlib.config import ensure_config_dir_exists, config_filename
32
33
from bzrlib.msgeditor import (
33
34
    make_commit_message_template_encoded,
34
35
    edit_commit_message_encoded
141
142
  hell\u00d8
142
143
""".encode('utf8') in template)
143
144
 
144
 
    def make_do_nothing_editor(self, basename='fed'):
 
145
    def make_do_nothing_editor(self):
145
146
        if sys.platform == "win32":
146
 
            name = basename + '.bat'
147
 
            f = file(name, 'w')
 
147
            f = file('fed.bat', 'w')
148
148
            f.write('@rem dummy fed')
149
149
            f.close()
150
 
            return name
 
150
            return 'fed.bat'
151
151
        else:
152
 
            name = basename + '.sh'
153
 
            f = file(name, 'wb')
 
152
            f = file('fed.sh', 'wb')
154
153
            f.write('#!/bin/sh\n')
155
154
            f.close()
156
 
            os.chmod(name, 0755)
157
 
            return './' + name
 
155
            os.chmod('fed.sh', 0755)
 
156
            return './fed.sh'
158
157
 
159
158
    def test_run_editor(self):
160
 
        self.overrideEnv('BZR_EDITOR', self.make_do_nothing_editor())
 
159
        os.environ['BZR_EDITOR'] = self.make_do_nothing_editor()
161
160
        self.assertEqual(True, msgeditor._run_editor(''),
162
161
                         'Unable to run dummy fake editor')
163
162
 
164
 
    def test_parse_editor_name(self):
165
 
        """Correctly interpret names with spaces.
166
 
 
167
 
        See <https://bugs.launchpad.net/bzr/+bug/220331>
168
 
        """
169
 
        self.overrideEnv('BZR_EDITOR',
170
 
            '"%s"' % self.make_do_nothing_editor('name with spaces'))
171
 
        self.assertEqual(True, msgeditor._run_editor('a_filename'))    
172
 
 
173
163
    def make_fake_editor(self, message='test message from fed\\n'):
174
164
        """Set up environment so that an editor will be a known script.
175
165
 
200
190
"%s" fed.py %%1
201
191
""" % sys.executable)
202
192
            f.close()
203
 
            self.overrideEnv('BZR_EDITOR', 'fed.bat')
 
193
            os.environ['BZR_EDITOR'] = 'fed.bat'
204
194
        else:
205
195
            # [non-win32] make python script executable and set BZR_EDITOR
206
196
            os.chmod('fed.py', 0755)
207
 
            self.overrideEnv('BZR_EDITOR', './fed.py')
 
197
            os.environ['BZR_EDITOR'] = './fed.py'
208
198
 
209
199
    def test_edit_commit_message(self):
210
200
        working_tree = self.make_uncommitted_tree()
239
229
        working_tree = self.make_uncommitted_tree()
240
230
 
241
231
        if sys.platform == 'win32':
242
 
            editor = 'cmd.exe /c del'
 
232
            os.environ['BZR_EDITOR'] = 'cmd.exe /c del'
243
233
        else:
244
 
            editor = 'rm'
245
 
        self.overrideEnv('BZR_EDITOR', editor)
 
234
            os.environ['BZR_EDITOR'] = 'rm'
246
235
 
247
236
        self.assertRaises((IOError, OSError), msgeditor.edit_commit_message, '')
248
237
 
249
238
    def test__get_editor(self):
250
 
        self.overrideEnv('BZR_EDITOR', 'bzr_editor')
251
 
        self.overrideEnv('VISUAL', 'visual')
252
 
        self.overrideEnv('EDITOR', 'editor')
253
 
 
254
 
        conf = config.GlobalConfig.from_string('editor = config_editor\n',
255
 
                                               save=True)
256
 
 
257
 
        editors = list(msgeditor._get_editor())
258
 
        editors = [editor for (editor, cfg_src) in editors]
259
 
 
260
 
        self.assertEqual(['bzr_editor', 'config_editor', 'visual', 'editor'],
261
 
                         editors[:4])
262
 
 
263
 
        if sys.platform == 'win32':
264
 
            self.assertEqual(['wordpad.exe', 'notepad.exe'], editors[4:])
265
 
        else:
266
 
            self.assertEqual(['/usr/bin/editor', 'vi', 'pico', 'nano', 'joe'],
267
 
                             editors[4:])
268
 
 
 
239
        # Test that _get_editor can return a decent list of items
 
240
        bzr_editor = os.environ.get('BZR_EDITOR')
 
241
        visual = os.environ.get('VISUAL')
 
242
        editor = os.environ.get('EDITOR')
 
243
        try:
 
244
            os.environ['BZR_EDITOR'] = 'bzr_editor'
 
245
            os.environ['VISUAL'] = 'visual'
 
246
            os.environ['EDITOR'] = 'editor'
 
247
 
 
248
            ensure_config_dir_exists()
 
249
            f = open(config_filename(), 'wb')
 
250
            f.write('editor = config_editor\n')
 
251
            f.close()
 
252
 
 
253
            editors = list(msgeditor._get_editor())
 
254
            editors = [editor for (editor, cfg_src) in editors]
 
255
 
 
256
            self.assertEqual(['bzr_editor', 'config_editor', 'visual',
 
257
                              'editor'], editors[:4])
 
258
 
 
259
            if sys.platform == 'win32':
 
260
                self.assertEqual(['wordpad.exe', 'notepad.exe'], editors[4:])
 
261
            else:
 
262
                self.assertEqual(['/usr/bin/editor', 'vi', 'pico', 'nano',
 
263
                                  'joe'], editors[4:])
 
264
 
 
265
        finally:
 
266
            # Restore the environment
 
267
            if bzr_editor is None:
 
268
                del os.environ['BZR_EDITOR']
 
269
            else:
 
270
                os.environ['BZR_EDITOR'] = bzr_editor
 
271
            if visual is None:
 
272
                del os.environ['VISUAL']
 
273
            else:
 
274
                os.environ['VISUAL'] = visual
 
275
            if editor is None:
 
276
                del os.environ['EDITOR']
 
277
            else:
 
278
                os.environ['EDITOR'] = editor
269
279
 
270
280
    def test__run_editor_EACCES(self):
271
281
        """If running a configured editor raises EACESS, the user is warned."""
272
 
        self.overrideEnv('BZR_EDITOR', 'eacces.py')
 
282
        os.environ['BZR_EDITOR'] = 'eacces.py'
273
283
        f = file('eacces.py', 'wb')
274
284
        f.write('# Not a real editor')
275
285
        f.close()
277
287
        os.chmod('eacces.py', 0)
278
288
        # Set $EDITOR so that _run_editor will terminate before trying real
279
289
        # editors.
280
 
        self.overrideEnv('EDITOR', self.make_do_nothing_editor())
 
290
        os.environ['EDITOR'] = self.make_do_nothing_editor()
281
291
        # Call _run_editor, capturing mutter.warning calls.
282
292
        warnings = []
283
293
        def warning(*args):
311
321
    def test__create_temp_file_with_commit_template_in_unicode_dir(self):
312
322
        self.requireFeature(tests.UnicodeFilenameFeature)
313
323
        if hasattr(self, 'info'):
314
 
            tmpdir = self.info['directory']
315
 
            os.mkdir(tmpdir)
316
 
            # Force the creation of temp file in a directory whose name
317
 
            # requires some encoding support
318
 
            msgeditor._create_temp_file_with_commit_template('infotext',
319
 
                                                             tmpdir=tmpdir)
 
324
            os.mkdir(self.info['directory'])
 
325
            os.chdir(self.info['directory'])
 
326
            msgeditor._create_temp_file_with_commit_template('infotext')
320
327
        else:
321
328
            raise TestNotApplicable('Test run elsewhere with non-ascii data.')
322
329
 
329
336
        self.assertFileEqual('', msgfilename)
330
337
 
331
338
    def test_unsupported_encoding_commit_message(self):
332
 
        self.overrideEnv('LANG', 'C')
333
 
        # LANG env variable has no effect on Windows
334
 
        # but some characters anyway cannot be represented
335
 
        # in default user encoding
336
 
        char = probe_bad_non_ascii(osutils.get_user_encoding())
337
 
        if char is None:
338
 
            raise TestSkipped('Cannot find suitable non-ascii character '
339
 
                'for user_encoding (%s)' % osutils.get_user_encoding())
340
 
 
341
 
        self.make_fake_editor(message=char)
342
 
 
343
 
        working_tree = self.make_uncommitted_tree()
344
 
        self.assertRaises(errors.BadCommitMessageEncoding,
345
 
                          msgeditor.edit_commit_message, '')
346
 
 
347
 
    def test_set_commit_message_no_hooks(self):
348
 
        commit_obj = commit.Commit()
349
 
        self.assertIs(None,
350
 
            msgeditor.set_commit_message(commit_obj))
351
 
 
352
 
    def test_set_commit_message_hook(self):
353
 
        msgeditor.hooks.install_named_hook("set_commit_message",
354
 
                lambda commit_obj, existing_message: "save me some typing\n", None)
355
 
        commit_obj = commit.Commit()
356
 
        self.assertEquals("save me some typing\n",
357
 
            msgeditor.set_commit_message(commit_obj))
 
339
        old_env = osutils.set_or_unset_env('LANG', 'C')
 
340
        try:
 
341
            # LANG env variable has no effect on Windows
 
342
            # but some characters anyway cannot be represented
 
343
            # in default user encoding
 
344
            char = probe_bad_non_ascii(osutils.get_user_encoding())
 
345
            if char is None:
 
346
                raise TestSkipped('Cannot find suitable non-ascii character '
 
347
                    'for user_encoding (%s)' % osutils.get_user_encoding())
 
348
 
 
349
            self.make_fake_editor(message=char)
 
350
 
 
351
            working_tree = self.make_uncommitted_tree()
 
352
            self.assertRaises(errors.BadCommitMessageEncoding,
 
353
                              msgeditor.edit_commit_message, '')
 
354
        finally:
 
355
            osutils.set_or_unset_env('LANG', old_env)
358
356
 
359
357
    def test_generate_commit_message_template_no_hooks(self):
360
358
        commit_obj = commit.Commit()
380
378
        ERROR_BAD_EXE_FORMAT = 193
381
379
        file("textfile.txt", "w").close()
382
380
        e = self.assertRaises(WindowsError, subprocess.call, "textfile.txt")
383
 
        self.assertEqual(e.errno, errno.ENOEXEC)
384
 
        self.assertEqual(e.winerror, ERROR_BAD_EXE_FORMAT)
 
381
        # Python2.4 used the 'winerror' as the errno, which confuses a lot of
 
382
        # our error trapping code. Make sure that we understand the mapping
 
383
        # correctly.
 
384
        if sys.version_info >= (2, 5):
 
385
            self.assertEqual(e.errno, errno.ENOEXEC)
 
386
            self.assertEqual(e.winerror, ERROR_BAD_EXE_FORMAT)
 
387
        else:
 
388
            self.assertEqual(e.errno, ERROR_BAD_EXE_FORMAT)