143
142
""".encode('utf8') in template)
 
145
 
    def make_do_nothing_editor(self):
 
 
144
    def make_do_nothing_editor(self, basename='fed'):
 
146
145
        if sys.platform == "win32":
 
147
 
            f = file('fed.bat', 'w')
 
 
146
            name = basename + '.bat'
 
148
148
            f.write('@rem dummy fed')
 
152
 
            f = file('fed.sh', 'wb')
 
 
152
            name = basename + '.sh'
 
153
154
            f.write('#!/bin/sh\n')
 
155
 
            os.chmod('fed.sh', 0755)
 
158
159
    def test_run_editor(self):
 
159
 
        os.environ['BZR_EDITOR'] = self.make_do_nothing_editor()
 
 
160
        self.overrideEnv('BZR_EDITOR', self.make_do_nothing_editor())
 
160
161
        self.assertEqual(True, msgeditor._run_editor(''),
 
161
162
                         'Unable to run dummy fake editor')
 
 
164
    def test_parse_editor_name(self):
 
 
165
        """Correctly interpret names with spaces.
 
 
167
        See <https://bugs.launchpad.net/bzr/+bug/220331>
 
 
169
        self.overrideEnv('BZR_EDITOR',
 
 
170
            '"%s"' % self.make_do_nothing_editor('name with spaces'))
 
 
171
        self.assertEqual(True, msgeditor._run_editor('a_filename'))    
 
163
173
    def make_fake_editor(self, message='test message from fed\\n'):
 
164
174
        """Set up environment so that an editor will be a known script.
 
 
229
239
        working_tree = self.make_uncommitted_tree()
 
231
241
        if sys.platform == 'win32':
 
232
 
            os.environ['BZR_EDITOR'] = 'cmd.exe /c del'
 
 
242
            editor = 'cmd.exe /c del'
 
234
 
            os.environ['BZR_EDITOR'] = 'rm'
 
 
245
        self.overrideEnv('BZR_EDITOR', editor)
 
236
247
        self.assertRaises((IOError, OSError), msgeditor.edit_commit_message, '')
 
238
249
    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
 
 
250
        self.overrideEnv('BZR_EDITOR', 'bzr_editor')
 
 
251
        self.overrideEnv('VISUAL', 'visual')
 
 
252
        self.overrideEnv('EDITOR', 'editor')
 
 
254
        conf = config.GlobalConfig.from_string('editor = config_editor\n',
 
 
257
        editors = list(msgeditor._get_editor())
 
 
258
        editors = [editor for (editor, cfg_src) in editors]
 
 
260
        self.assertEqual(['bzr_editor', 'config_editor', 'visual', 'editor'],
 
 
263
        if sys.platform == 'win32':
 
 
264
            self.assertEqual(['wordpad.exe', 'notepad.exe'], editors[4:])
 
 
266
            self.assertEqual(['/usr/bin/editor', 'vi', 'pico', 'nano', 'joe'],
 
280
270
    def test__run_editor_EACCES(self):
 
281
271
        """If running a configured editor raises EACESS, the user is warned."""
 
282
 
        os.environ['BZR_EDITOR'] = 'eacces.py'
 
 
272
        self.overrideEnv('BZR_EDITOR', 'eacces.py')
 
283
273
        f = file('eacces.py', 'wb')
 
284
274
        f.write('# Not a real editor')
 
 
321
311
    def test__create_temp_file_with_commit_template_in_unicode_dir(self):
 
322
312
        self.requireFeature(tests.UnicodeFilenameFeature)
 
323
313
        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')
 
 
314
            tmpdir = self.info['directory']
 
 
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',
 
328
321
            raise TestNotApplicable('Test run elsewhere with non-ascii data.')
 
 
336
329
        self.assertFileEqual('', msgfilename)
 
338
331
    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)
 
 
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())
 
 
338
            raise TestSkipped('Cannot find suitable non-ascii character '
 
 
339
                'for user_encoding (%s)' % osutils.get_user_encoding())
 
 
341
        self.make_fake_editor(message=char)
 
 
343
        working_tree = self.make_uncommitted_tree()
 
 
344
        self.assertRaises(errors.BadCommitMessageEncoding,
 
 
345
                          msgeditor.edit_commit_message, '')
 
357
347
    def test_generate_commit_message_template_no_hooks(self):
 
358
348
        commit_obj = commit.Commit()