← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_msgeditor.py

More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_msgeditor.py
157
157
            return './fed.sh'
158
158
 
159
159
    def test_run_editor(self):
160
 
        os.environ['BZR_EDITOR'] = self.make_do_nothing_editor()
 
160
        self.overrideEnv('BZR_EDITOR', self.make_do_nothing_editor())
161
161
        self.assertEqual(True, msgeditor._run_editor(''),
162
162
                         'Unable to run dummy fake editor')
163
163
 
191
191
"%s" fed.py %%1
192
192
""" % sys.executable)
193
193
            f.close()
194
 
            os.environ['BZR_EDITOR'] = 'fed.bat'
 
194
            self.overrideEnv('BZR_EDITOR', 'fed.bat')
195
195
        else:
196
196
            # [non-win32] make python script executable and set BZR_EDITOR
197
197
            os.chmod('fed.py', 0755)
198
 
            os.environ['BZR_EDITOR'] = './fed.py'
 
198
            self.overrideEnv('BZR_EDITOR', './fed.py')
199
199
 
200
200
    def test_edit_commit_message(self):
201
201
        working_tree = self.make_uncommitted_tree()
230
230
        working_tree = self.make_uncommitted_tree()
231
231
 
232
232
        if sys.platform == 'win32':
233
 
            os.environ['BZR_EDITOR'] = 'cmd.exe /c del'
 
233
            editor = 'cmd.exe /c del'
234
234
        else:
235
 
            os.environ['BZR_EDITOR'] = 'rm'
 
235
            editor = 'rm'
 
236
        self.overrideEnv('BZR_EDITOR', editor)
236
237
 
237
238
        self.assertRaises((IOError, OSError), msgeditor.edit_commit_message, '')
238
239
 
239
240
    def test__get_editor(self):
240
 
        os.environ['BZR_EDITOR'] = 'bzr_editor'
241
 
        os.environ['VISUAL'] = 'visual'
242
 
        os.environ['EDITOR'] = 'editor'
 
241
        self.overrideEnv('BZR_EDITOR', 'bzr_editor')
 
242
        self.overrideEnv('VISUAL', 'visual')
 
243
        self.overrideEnv('EDITOR', 'editor')
243
244
 
244
245
        conf = config.GlobalConfig.from_string('editor = config_editor\n',
245
246
                                               save=True)
259
260
 
260
261
    def test__run_editor_EACCES(self):
261
262
        """If running a configured editor raises EACESS, the user is warned."""
262
 
        os.environ['BZR_EDITOR'] = 'eacces.py'
 
263
        self.overrideEnv('BZR_EDITOR', 'eacces.py')
263
264
        f = file('eacces.py', 'wb')
264
265
        f.write('# Not a real editor')
265
266
        f.close()
267
268
        os.chmod('eacces.py', 0)
268
269
        # Set $EDITOR so that _run_editor will terminate before trying real
269
270
        # editors.
270
 
        os.environ['EDITOR'] = self.make_do_nothing_editor()
 
271
        self.overrideEnv('EDITOR', self.make_do_nothing_editor())
271
272
        # Call _run_editor, capturing mutter.warning calls.
272
273
        warnings = []
273
274
        def warning(*args):