229
228
working_tree = self.make_uncommitted_tree()
231
230
if sys.platform == 'win32':
232
os.environ['BZR_EDITOR'] = 'cmd.exe /c del'
231
editor = 'cmd.exe /c del'
234
os.environ['BZR_EDITOR'] = 'rm'
234
self.overrideEnv('BZR_EDITOR', editor)
236
236
self.assertRaises((IOError, OSError), msgeditor.edit_commit_message, '')
238
238
def test__get_editor(self):
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')
244
os.environ['BZR_EDITOR'] = 'bzr_editor'
245
os.environ['VISUAL'] = 'visual'
246
os.environ['EDITOR'] = 'editor'
248
ensure_config_dir_exists()
249
f = open(config_filename(), 'wb')
250
f.write('editor = config_editor\n')
253
editors = list(msgeditor._get_editor())
254
editors = [editor for (editor, cfg_src) in editors]
256
self.assertEqual(['bzr_editor', 'config_editor', 'visual',
257
'editor'], editors[:4])
259
if sys.platform == 'win32':
260
self.assertEqual(['wordpad.exe', 'notepad.exe'], editors[4:])
262
self.assertEqual(['/usr/bin/editor', 'vi', 'pico', 'nano',
266
# Restore the environment
267
if bzr_editor is None:
268
del os.environ['BZR_EDITOR']
270
os.environ['BZR_EDITOR'] = bzr_editor
272
del os.environ['VISUAL']
274
os.environ['VISUAL'] = visual
276
del os.environ['EDITOR']
278
os.environ['EDITOR'] = editor
239
self.overrideEnv('BZR_EDITOR', 'bzr_editor')
240
self.overrideEnv('VISUAL', 'visual')
241
self.overrideEnv('EDITOR', 'editor')
243
conf = config.GlobalConfig.from_string('editor = config_editor\n',
246
editors = list(msgeditor._get_editor())
247
editors = [editor for (editor, cfg_src) in editors]
249
self.assertEqual(['bzr_editor', 'config_editor', 'visual', 'editor'],
252
if sys.platform == 'win32':
253
self.assertEqual(['wordpad.exe', 'notepad.exe'], editors[4:])
255
self.assertEqual(['/usr/bin/editor', 'vi', 'pico', 'nano', 'joe'],
280
259
def test__run_editor_EACCES(self):
281
260
"""If running a configured editor raises EACESS, the user is warned."""
282
os.environ['BZR_EDITOR'] = 'eacces.py'
261
self.overrideEnv('BZR_EDITOR', 'eacces.py')
283
262
f = file('eacces.py', 'wb')
284
263
f.write('# Not a real editor')
321
300
def test__create_temp_file_with_commit_template_in_unicode_dir(self):
322
301
self.requireFeature(tests.UnicodeFilenameFeature)
323
302
if hasattr(self, 'info'):
324
os.mkdir(self.info['directory'])
325
os.chdir(self.info['directory'])
326
msgeditor._create_temp_file_with_commit_template('infotext')
303
tmpdir = self.info['directory']
305
# Force the creation of temp file in a directory whose name
306
# requires some encoding support
307
msgeditor._create_temp_file_with_commit_template('infotext',
328
310
raise TestNotApplicable('Test run elsewhere with non-ascii data.')
336
318
self.assertFileEqual('', msgfilename)
338
320
def test_unsupported_encoding_commit_message(self):
339
old_env = osutils.set_or_unset_env('LANG', 'C')
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())
346
raise TestSkipped('Cannot find suitable non-ascii character '
347
'for user_encoding (%s)' % osutils.get_user_encoding())
349
self.make_fake_editor(message=char)
351
working_tree = self.make_uncommitted_tree()
352
self.assertRaises(errors.BadCommitMessageEncoding,
353
msgeditor.edit_commit_message, '')
355
osutils.set_or_unset_env('LANG', old_env)
321
self.overrideEnv('LANG', 'C')
322
# LANG env variable has no effect on Windows
323
# but some characters anyway cannot be represented
324
# in default user encoding
325
char = probe_bad_non_ascii(osutils.get_user_encoding())
327
raise TestSkipped('Cannot find suitable non-ascii character '
328
'for user_encoding (%s)' % osutils.get_user_encoding())
330
self.make_fake_editor(message=char)
332
working_tree = self.make_uncommitted_tree()
333
self.assertRaises(errors.BadCommitMessageEncoding,
334
msgeditor.edit_commit_message, '')
357
336
def test_generate_commit_message_template_no_hooks(self):
358
337
commit_obj = commit.Commit()