/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to breezy/tests/test_msgeditor.py

  • Committer: Martin
  • Date: 2018-07-03 17:23:08 UTC
  • mto: (7027.4.10 python3-blackbox)
  • mto: This revision was merged to the branch mainline in revision 7028.
  • Revision ID: gzlist@googlemail.com-20180703172308-lre82d15bfkdv48e
Fixup low level knit tests for Python 3

Show diffs side-by-side

added added

removed removed

Lines of Context:
breezy/tests/test_msgeditor.py
 
1
# Copyright (C) 2005-2012, 2016 Canonical Ltd
 
2
#
 
3
# This program is free software; you can redistribute it and/or modify
 
4
# it under the terms of the GNU General Public License as published by
 
5
# the Free Software Foundation; either version 2 of the License, or
 
6
# (at your option) any later version.
 
7
#
 
8
# This program is distributed in the hope that it will be useful,
 
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
 
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
11
# GNU General Public License for more details.
 
12
#
 
13
# You should have received a copy of the GNU General Public License
 
14
# along with this program; if not, write to the Free Software
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 
16
 
 
17
"""Test commit message editor.
 
18
"""
 
19
 
 
20
import os
 
21
import sys
 
22
 
 
23
from .. import (
 
24
    commit,
 
25
    config,
 
26
    errors,
 
27
    msgeditor,
 
28
    osutils,
 
29
    trace,
 
30
    )
 
31
from ..msgeditor import (
 
32
    make_commit_message_template_encoded,
 
33
    edit_commit_message_encoded
 
34
)
 
35
from . import (
 
36
    features,
 
37
    TestCaseInTempDir,
 
38
    TestCaseWithTransport,
 
39
    TestNotApplicable,
 
40
    multiply_tests,
 
41
    probe_bad_non_ascii,
 
42
    probe_unicode_in_user_encoding,
 
43
    split_suite_by_re,
 
44
    )
 
45
from .EncodingAdapter import encoding_scenarios
 
46
from ..trace import mutter
 
47
 
 
48
 
 
49
def load_tests(loader, standard_tests, pattern):
 
50
    """Parameterize the test for tempfile creation with different encodings."""
 
51
    to_adapt, result = split_suite_by_re(standard_tests,
 
52
        "test__create_temp_file_with_commit_template_in_unicode_dir")
 
53
    return multiply_tests(to_adapt, encoding_scenarios, result)
 
54
 
 
55
 
 
56
class MsgEditorTest(TestCaseWithTransport):
 
57
 
 
58
    def make_uncommitted_tree(self):
 
59
        """Build a branch with uncommitted unicode named changes in the cwd."""
 
60
        working_tree = self.make_branch_and_tree('.')
 
61
        b = working_tree.branch
 
62
        filename = u'hell\u00d8'
 
63
        try:
 
64
            self.build_tree_contents([(filename, b'contents of hello')])
 
65
        except UnicodeEncodeError:
 
66
            self.skipTest("can't build unicode working tree in "
 
67
                "filesystem encoding %s" % sys.getfilesystemencoding())
 
68
        working_tree.add(filename)
 
69
        return working_tree
 
70
 
 
71
    def test_commit_template(self):
 
72
        """Test building a commit message template"""
 
73
        working_tree = self.make_uncommitted_tree()
 
74
        template = msgeditor.make_commit_message_template(working_tree,
 
75
                                                                 None)
 
76
        self.assertEqualDiff(template,
 
77
u"""\
 
78
added:
 
79
  hell\u00d8
 
80
""")
 
81
 
 
82
    def make_multiple_pending_tree(self):
 
83
        config.GlobalStack().set('email', 'Bilbo Baggins <bb@hobbit.net>')
 
84
        tree = self.make_branch_and_tree('a')
 
85
        tree.commit('Initial checkin.', timestamp=1230912900, timezone=0)
 
86
        tree2 = tree.controldir.clone('b').open_workingtree()
 
87
        tree.commit('Minor tweak.', timestamp=1231977840, timezone=0)
 
88
        tree2.commit('Feature X work.', timestamp=1233186240, timezone=0)
 
89
        tree3 = tree2.controldir.clone('c').open_workingtree()
 
90
        tree2.commit('Feature X finished.', timestamp=1233187680, timezone=0)
 
91
        tree3.commit('Feature Y, based on initial X work.',
 
92
                     timestamp=1233285960, timezone=0)
 
93
        tree.merge_from_branch(tree2.branch)
 
94
        tree.merge_from_branch(tree3.branch, force=True)
 
95
        return tree
 
96
 
 
97
    def test_commit_template_pending_merges(self):
 
98
        """Test building a commit message template when there are pending
 
99
        merges.  The commit message should show all pending merge revisions,
 
100
        as does 'status -v', not only the merge tips.
 
101
        """
 
102
        working_tree = self.make_multiple_pending_tree()
 
103
        template = msgeditor.make_commit_message_template(working_tree, None)
 
104
        self.assertEqualDiff(template,
 
105
u"""\
 
106
pending merges:
 
107
  Bilbo Baggins 2009-01-29 Feature X finished.
 
108
    Bilbo Baggins 2009-01-28 Feature X work.
 
109
  Bilbo Baggins 2009-01-30 Feature Y, based on initial X work.
 
110
""")
 
111
 
 
112
    def test_commit_template_encoded(self):
 
113
        """Test building a commit message template"""
 
114
        working_tree = self.make_uncommitted_tree()
 
115
        template = make_commit_message_template_encoded(working_tree,
 
116
                                                        None,
 
117
                                                        output_encoding='utf8')
 
118
        self.assertEqualDiff(template,
 
119
u"""\
 
120
added:
 
121
  hell\u00d8
 
122
""".encode("utf8"))
 
123
 
 
124
 
 
125
    def test_commit_template_and_diff(self):
 
126
        """Test building a commit message template"""
 
127
        working_tree = self.make_uncommitted_tree()
 
128
        template = make_commit_message_template_encoded(working_tree,
 
129
                                                        None,
 
130
                                                        diff=True,
 
131
                                                        output_encoding='utf8')
 
132
 
 
133
        self.assertTrue("""\
 
134
@@ -0,0 +1,1 @@
 
135
+contents of hello
 
136
""" in template)
 
137
        self.assertTrue(u"""\
 
138
added:
 
139
  hell\u00d8
 
140
""".encode('utf8') in template)
 
141
 
 
142
    def make_do_nothing_editor(self, basename='fed'):
 
143
        if sys.platform == "win32":
 
144
            name = basename + '.bat'
 
145
            with open(name, 'w') as f:
 
146
                f.write('@rem dummy fed')
 
147
            return name
 
148
        else:
 
149
            name = basename + '.sh'
 
150
            with open(name, 'wb') as f:
 
151
                f.write(b'#!/bin/sh\n')
 
152
            os.chmod(name, 0o755)
 
153
            return './' + name
 
154
 
 
155
    def test_run_editor(self):
 
156
        self.overrideEnv('BRZ_EDITOR', self.make_do_nothing_editor())
 
157
        self.assertEqual(True, msgeditor._run_editor(''),
 
158
                         'Unable to run dummy fake editor')
 
159
 
 
160
    def test_parse_editor_name(self):
 
161
        """Correctly interpret names with spaces.
 
162
 
 
163
        See <https://bugs.launchpad.net/bzr/+bug/220331>
 
164
        """
 
165
        self.overrideEnv('BRZ_EDITOR',
 
166
            '"%s"' % self.make_do_nothing_editor('name with spaces'))
 
167
        self.assertEqual(True, msgeditor._run_editor('a_filename'))    
 
168
 
 
169
    def make_fake_editor(self, message='test message from fed\\n'):
 
170
        """Set up environment so that an editor will be a known script.
 
171
 
 
172
        Sets up BRZ_EDITOR so that if an editor is spawned it will run a
 
173
        script that just adds a known message to the start of the file.
 
174
        """
 
175
        with open('fed.py', 'wb') as f:
 
176
            f.write(b'#!%s\n' % sys.executable)
 
177
            f.write(b"""\
 
178
# coding=utf-8
 
179
import sys
 
180
if len(sys.argv) == 2:
 
181
    fn = sys.argv[1]
 
182
    f = open(fn, 'rb')
 
183
    s = f.read()
 
184
    f.close()
 
185
    f = open(fn, 'wb')
 
186
    f.write('%s')
 
187
    f.write(s)
 
188
    f.close()
 
189
""" % (message, ))
 
190
        if sys.platform == "win32":
 
191
            # [win32] make batch file and set BRZ_EDITOR
 
192
            with open('fed.bat', 'w') as f:
 
193
                f.write("""\
 
194
@echo off
 
195
"%s" fed.py %%1
 
196
""" % sys.executable)
 
197
            self.overrideEnv('BRZ_EDITOR', 'fed.bat')
 
198
        else:
 
199
            # [non-win32] make python script executable and set BRZ_EDITOR
 
200
            os.chmod('fed.py', 0o755)
 
201
            self.overrideEnv('BRZ_EDITOR', './fed.py')
 
202
 
 
203
    def test_edit_commit_message_without_infotext(self):
 
204
        working_tree = self.make_uncommitted_tree()
 
205
        self.make_fake_editor()
 
206
 
 
207
        mutter('edit_commit_message without infotext')
 
208
        self.assertEqual('test message from fed\n',
 
209
                         msgeditor.edit_commit_message(''))
 
210
 
 
211
    def test_edit_commit_message_with_ascii_infotext(self):
 
212
        working_tree = self.make_uncommitted_tree()
 
213
        self.make_fake_editor()
 
214
 
 
215
        mutter('edit_commit_message with ascii string infotext')
 
216
        self.assertEqual('test message from fed\n',
 
217
                         msgeditor.edit_commit_message('spam'))
 
218
 
 
219
    def test_edit_commit_message_with_unicode_infotext(self):
 
220
        working_tree = self.make_uncommitted_tree()
 
221
        self.make_fake_editor()
 
222
 
 
223
        mutter('edit_commit_message with unicode infotext')
 
224
        uni_val, ue_val = probe_unicode_in_user_encoding()
 
225
        if ue_val is None:
 
226
            self.skipTest(
 
227
                'Cannot find a unicode character that works in encoding %s'
 
228
                % (osutils.get_user_encoding(),))
 
229
 
 
230
        self.assertEqual('test message from fed\n',
 
231
                         msgeditor.edit_commit_message(uni_val))
 
232
 
 
233
        tmpl = edit_commit_message_encoded(u'\u1234'.encode("utf8"))
 
234
        self.assertEqual('test message from fed\n', tmpl)
 
235
 
 
236
    def test_start_message(self):
 
237
        self.make_uncommitted_tree()
 
238
        self.make_fake_editor()
 
239
        self.assertEqual('test message from fed\nstart message\n',
 
240
                         msgeditor.edit_commit_message('',
 
241
                                              start_message='start message\n'))
 
242
        self.assertEqual('test message from fed\n',
 
243
                         msgeditor.edit_commit_message('',
 
244
                                              start_message=''))
 
245
 
 
246
    def test_deleted_commit_message(self):
 
247
        working_tree = self.make_uncommitted_tree()
 
248
 
 
249
        if sys.platform == 'win32':
 
250
            editor = 'cmd.exe /c del'
 
251
        else:
 
252
            editor = 'rm'
 
253
        self.overrideEnv('BRZ_EDITOR', editor)
 
254
 
 
255
        self.assertRaises((IOError, OSError), msgeditor.edit_commit_message, '')
 
256
 
 
257
    def test__get_editor(self):
 
258
        self.overrideEnv('BRZ_EDITOR', 'bzr_editor')
 
259
        self.overrideEnv('VISUAL', 'visual')
 
260
        self.overrideEnv('EDITOR', 'editor')
 
261
 
 
262
        conf = config.GlobalStack()
 
263
        conf.store._load_from_string(b'[DEFAULT]\neditor = config_editor\n')
 
264
        conf.store.save()
 
265
        editors = list(msgeditor._get_editor())
 
266
        editors = [editor for (editor, cfg_src) in editors]
 
267
 
 
268
        self.assertEqual(['bzr_editor', 'config_editor', 'visual', 'editor'],
 
269
                         editors[:4])
 
270
 
 
271
        if sys.platform == 'win32':
 
272
            self.assertEqual(['wordpad.exe', 'notepad.exe'], editors[4:])
 
273
        else:
 
274
            self.assertEqual(['/usr/bin/editor', 'vi', 'pico', 'nano', 'joe'],
 
275
                             editors[4:])
 
276
 
 
277
 
 
278
    def test__run_editor_EACCES(self):
 
279
        """If running a configured editor raises EACESS, the user is warned."""
 
280
        self.overrideEnv('BRZ_EDITOR', 'eacces.py')
 
281
        with open('eacces.py', 'wb') as f:
 
282
            f.write(b'# Not a real editor')
 
283
        # Make the fake editor unreadable (and unexecutable)
 
284
        os.chmod('eacces.py', 0)
 
285
        # Set $EDITOR so that _run_editor will terminate before trying real
 
286
        # editors.
 
287
        self.overrideEnv('EDITOR', self.make_do_nothing_editor())
 
288
        # Call _run_editor, capturing mutter.warning calls.
 
289
        warnings = []
 
290
        def warning(*args):
 
291
            if len(args) > 1:
 
292
                warnings.append(args[0] % args[1:])
 
293
            else:
 
294
                warnings.append(args[0])
 
295
        _warning = trace.warning
 
296
        trace.warning = warning
 
297
        try:
 
298
            msgeditor._run_editor('')
 
299
        finally:
 
300
            trace.warning = _warning
 
301
        self.assertStartsWith(warnings[0], 'Could not start editor "eacces.py"')
 
302
 
 
303
    def test__create_temp_file_with_commit_template(self):
 
304
        # check that commit template written properly
 
305
        # and has platform native line-endings (CRLF on win32)
 
306
        create_file = msgeditor._create_temp_file_with_commit_template
 
307
        msgfilename, hasinfo = create_file('infotext', '----', 'start message')
 
308
        self.assertNotEqual(None, msgfilename)
 
309
        self.assertTrue(hasinfo)
 
310
        expected = os.linesep.join(['start message',
 
311
                                    '',
 
312
                                    '',
 
313
                                    '----',
 
314
                                    '',
 
315
                                    'infotext'])
 
316
        self.assertFileEqual(expected, msgfilename)
 
317
 
 
318
    def test__create_temp_file_with_commit_template_in_unicode_dir(self):
 
319
        self.requireFeature(features.UnicodeFilenameFeature)
 
320
        if hasattr(self, 'info'):
 
321
            tmpdir = self.info['directory']
 
322
            os.mkdir(tmpdir)
 
323
            # Force the creation of temp file in a directory whose name
 
324
            # requires some encoding support
 
325
            msgeditor._create_temp_file_with_commit_template('infotext',
 
326
                                                             tmpdir=tmpdir)
 
327
        else:
 
328
            raise TestNotApplicable('Test run elsewhere with non-ascii data.')
 
329
 
 
330
    def test__create_temp_file_with_empty_commit_template(self):
 
331
        # empty file
 
332
        create_file = msgeditor._create_temp_file_with_commit_template
 
333
        msgfilename, hasinfo = create_file('')
 
334
        self.assertNotEqual(None, msgfilename)
 
335
        self.assertFalse(hasinfo)
 
336
        self.assertFileEqual('', msgfilename)
 
337
 
 
338
    def test_unsupported_encoding_commit_message(self):
 
339
        self.overrideEnv('LANG', 'C')
 
340
        # LANG env variable has no effect on Windows
 
341
        # but some characters anyway cannot be represented
 
342
        # in default user encoding
 
343
        char = probe_bad_non_ascii(osutils.get_user_encoding())
 
344
        if char is None:
 
345
            self.skipTest('Cannot find suitable non-ascii character '
 
346
                'for user_encoding (%s)' % osutils.get_user_encoding())
 
347
 
 
348
        self.make_fake_editor(message=char)
 
349
 
 
350
        working_tree = self.make_uncommitted_tree()
 
351
        self.assertRaises(msgeditor.BadCommitMessageEncoding,
 
352
                          msgeditor.edit_commit_message, '')
 
353
 
 
354
    def test_set_commit_message_no_hooks(self):
 
355
        commit_obj = commit.Commit()
 
356
        self.assertIs(None,
 
357
            msgeditor.set_commit_message(commit_obj))
 
358
 
 
359
    def test_set_commit_message_hook(self):
 
360
        msgeditor.hooks.install_named_hook("set_commit_message",
 
361
                lambda commit_obj, existing_message: "save me some typing\n", None)
 
362
        commit_obj = commit.Commit()
 
363
        self.assertEqual("save me some typing\n",
 
364
            msgeditor.set_commit_message(commit_obj))
 
365
 
 
366
    def test_generate_commit_message_template_no_hooks(self):
 
367
        commit_obj = commit.Commit()
 
368
        self.assertIs(None,
 
369
            msgeditor.generate_commit_message_template(commit_obj))
 
370
 
 
371
    def test_generate_commit_message_template_hook(self):
 
372
        msgeditor.hooks.install_named_hook("commit_message_template",
 
373
                lambda commit_obj, msg: "save me some typing\n", None)
 
374
        commit_obj = commit.Commit()
 
375
        self.assertEqual("save me some typing\n",
 
376
            msgeditor.generate_commit_message_template(commit_obj))
 
377
 
 
378
 
 
379
# GZ 2009-11-17: This wants moving to osutils when the errno checking code is
 
380
class TestPlatformErrnoWorkarounds(TestCaseInTempDir):
 
381
    """Ensuring workarounds enshrined in code actually serve a purpose"""
 
382
 
 
383
    def test_subprocess_call_bad_file(self):
 
384
        if sys.platform != "win32":
 
385
            raise TestNotApplicable("Workarounds for windows only")
 
386
        import subprocess, errno
 
387
        ERROR_BAD_EXE_FORMAT = 193
 
388
        open("textfile.txt", "w").close()
 
389
        e = self.assertRaises(WindowsError, subprocess.call, "textfile.txt")
 
390
        self.assertEqual(e.errno, errno.ENOEXEC)
 
391
        self.assertEqual(e.winerror, ERROR_BAD_EXE_FORMAT)