6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
1 |
# Copyright (C) 2005-2012, 2016 Canonical Ltd
|
1185.33.72
by Martin Pool
Fix commit message template for non-ascii files, and add test for handling of |
2 |
#
|
3 |
# This program is free software; you can redistribute it and/or modify
|
|
2052.3.1
by John Arbash Meinel
Add tests to cleanup the copyright of all source files |
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.
|
|
1185.33.72
by Martin Pool
Fix commit message template for non-ascii files, and add test for handling of |
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
|
|
4183.7.1
by Sabin Iacob
update FSF mailing address |
15 |
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
1185.33.72
by Martin Pool
Fix commit message template for non-ascii files, and add test for handling of |
16 |
|
17 |
"""Test commit message editor.
|
|
18 |
"""
|
|
19 |
||
20 |
import os |
|
21 |
import sys |
|
22 |
||
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
23 |
from bzrlib import ( |
3642.2.2
by Jelmer Vernooij
Add tests for commit_message_template hooks. |
24 |
commit, |
5345.1.13
by Vincent Ladeuil
Cleanup test_msgeditor. |
25 |
config, |
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
26 |
errors, |
27 |
msgeditor, |
|
28 |
osutils, |
|
3910.1.1
by Andrew Bennetts
Improve error handling in msgeditor._run_editor. |
29 |
trace, |
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
30 |
)
|
2598.6.24
by ghigo
update on the basis of Aaron suggestions |
31 |
from bzrlib.msgeditor import ( |
32 |
make_commit_message_template_encoded, |
|
2598.6.27
by ghigo
small cleanup (line lenght) |
33 |
edit_commit_message_encoded
|
2598.6.24
by ghigo
update on the basis of Aaron suggestions |
34 |
)
|
2804.4.1
by Alexander Belchenko
some win32-specific fixes for selftest |
35 |
from bzrlib.tests import ( |
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
36 |
features, |
4789.23.3
by Martin
Test that msgeditor errno workarounds for windows are actually required |
37 |
TestCaseInTempDir, |
2804.4.1
by Alexander Belchenko
some win32-specific fixes for selftest |
38 |
TestCaseWithTransport, |
3004.1.8
by Daniel Watkins
Changed TestSkipped to TestNotApplicable as suggested by Aaron. |
39 |
TestNotApplicable, |
2804.4.1
by Alexander Belchenko
some win32-specific fixes for selftest |
40 |
TestSkipped, |
4084.5.1
by Robert Collins
Bulk update all test adaptation into a single approach, using multiply_tests rather than test adapters. |
41 |
multiply_tests, |
42 |
probe_bad_non_ascii, |
|
43 |
split_suite_by_re, |
|
2804.4.1
by Alexander Belchenko
some win32-specific fixes for selftest |
44 |
)
|
4084.5.1
by Robert Collins
Bulk update all test adaptation into a single approach, using multiply_tests rather than test adapters. |
45 |
from bzrlib.tests.EncodingAdapter import encoding_scenarios |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
46 |
from bzrlib.trace import mutter |
1534.4.26
by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create. |
47 |
|
2592.3.164
by Robert Collins
Reduce spurious differences to bzr.dev from merge mishaps. |
48 |
|
2921.6.14
by Robert Collins
Push the message editor test parameterisation down into the message editor |
49 |
def load_tests(standard_tests, module, loader): |
3128.1.3
by Vincent Ladeuil
Since we are there s/parameteris.*/parameteriz&/. |
50 |
"""Parameterize the test for tempfile creation with different encodings."""
|
2921.6.14
by Robert Collins
Push the message editor test parameterisation down into the message editor |
51 |
to_adapt, result = split_suite_by_re(standard_tests, |
52 |
"test__create_temp_file_with_commit_template_in_unicode_dir") |
|
4084.5.1
by Robert Collins
Bulk update all test adaptation into a single approach, using multiply_tests rather than test adapters. |
53 |
return multiply_tests(to_adapt, encoding_scenarios, result) |
2921.6.14
by Robert Collins
Push the message editor test parameterisation down into the message editor |
54 |
|
55 |
||
1534.4.26
by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create. |
56 |
class MsgEditorTest(TestCaseWithTransport): |
1185.33.72
by Martin Pool
Fix commit message template for non-ascii files, and add test for handling of |
57 |
|
1526.1.4
by Robert Collins
forgot my self. |
58 |
def make_uncommitted_tree(self): |
1526.1.1
by Robert Collins
Run the test suite with no locale as well as the default locale. Also add a test for build_tree_shape to selftest. |
59 |
"""Build a branch with uncommitted unicode named changes in the cwd."""
|
1534.4.26
by Robert Collins
Move working tree initialisation out from Branch.initialize, deprecated Branch.initialize to Branch.create. |
60 |
working_tree = self.make_branch_and_tree('.') |
61 |
b = working_tree.branch |
|
1526.1.1
by Robert Collins
Run the test suite with no locale as well as the default locale. Also add a test for build_tree_shape to selftest. |
62 |
filename = u'hell\u00d8' |
1526.1.3
by Robert Collins
Merge from upstream. |
63 |
try: |
64 |
self.build_tree_contents([(filename, 'contents of hello')]) |
|
65 |
except UnicodeEncodeError: |
|
66 |
raise TestSkipped("can't build unicode working tree in " |
|
1185.33.97
by Martin Pool
MsgEditor tests should be skipped on platforms without unicode fs. |
67 |
"filesystem encoding %s" % sys.getfilesystemencoding()) |
1526.1.1
by Robert Collins
Run the test suite with no locale as well as the default locale. Also add a test for build_tree_shape to selftest. |
68 |
working_tree.add(filename) |
69 |
return working_tree |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
70 |
|
1185.33.72
by Martin Pool
Fix commit message template for non-ascii files, and add test for handling of |
71 |
def test_commit_template(self): |
72 |
"""Test building a commit message template"""
|
|
1526.1.2
by Robert Collins
Fix typo in my msgeditor test changes. |
73 |
working_tree = self.make_uncommitted_tree() |
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
74 |
template = msgeditor.make_commit_message_template(working_tree, |
2598.6.24
by ghigo
update on the basis of Aaron suggestions |
75 |
None) |
2598.6.18
by ghigo
Update the tests to the new *_encoded() functions |
76 |
self.assertEqualDiff(template, |
77 |
u"""\ |
|
78 |
added:
|
|
79 |
hell\u00d8
|
|
80 |
""") |
|
81 |
||
3968.1.1
by Colin D Bennett
Show all pending merge revisions in the commit message template. |
82 |
def make_multiple_pending_tree(self): |
6449.6.7
by Jelmer Vernooij
Fix tests. |
83 |
config.GlobalStack().set('email', 'Bilbo Baggins <bb@hobbit.net>') |
3968.1.1
by Colin D Bennett
Show all pending merge revisions in the commit message template. |
84 |
tree = self.make_branch_and_tree('a') |
85 |
tree.commit('Initial checkin.', timestamp=1230912900, timezone=0) |
|
86 |
tree2 = tree.bzrdir.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.bzrdir.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) |
|
4721.3.2
by Vincent Ladeuil
Simplify mutable_tree.has_changes() and update call sites. |
94 |
tree.merge_from_branch(tree3.branch, force=True) |
3968.1.1
by Colin D Bennett
Show all pending merge revisions in the commit message template. |
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 |
||
2598.6.18
by ghigo
Update the tests to the new *_encoded() functions |
112 |
def test_commit_template_encoded(self): |
113 |
"""Test building a commit message template"""
|
|
114 |
working_tree = self.make_uncommitted_tree() |
|
2598.6.24
by ghigo
update on the basis of Aaron suggestions |
115 |
template = make_commit_message_template_encoded(working_tree, |
116 |
None, |
|
117 |
output_encoding='utf8') |
|
1185.33.72
by Martin Pool
Fix commit message template for non-ascii files, and add test for handling of |
118 |
self.assertEqualDiff(template, |
119 |
u"""\ |
|
120 |
added:
|
|
121 |
hell\u00d8
|
|
2598.6.14
by ghigo
Update the test |
122 |
""".encode("utf8")) |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
123 |
|
2598.6.18
by ghigo
Update the tests to the new *_encoded() functions |
124 |
|
2598.6.2
by ghigo
Add testcase |
125 |
def test_commit_template_and_diff(self): |
126 |
"""Test building a commit message template"""
|
|
127 |
working_tree = self.make_uncommitted_tree() |
|
2598.6.24
by ghigo
update on the basis of Aaron suggestions |
128 |
template = make_commit_message_template_encoded(working_tree, |
129 |
None, |
|
130 |
diff=True, |
|
131 |
output_encoding='utf8') |
|
2598.6.2
by ghigo
Add testcase |
132 |
|
2598.6.10
by ghigo
In the commit dialog, the diff is stored as 8-bit raw data |
133 |
self.assertTrue("""\ |
2598.6.2
by ghigo
Add testcase |
134 |
@@ -0,0 +1,1 @@
|
135 |
+contents of hello
|
|
2598.6.14
by ghigo
Update the test |
136 |
""" in template) |
2598.6.3
by ghigo
Add test case |
137 |
self.assertTrue(u"""\ |
138 |
added:
|
|
139 |
hell\u00d8 |
|
2598.6.14
by ghigo
Update the test |
140 |
""".encode('utf8') in template) |
2598.6.2
by ghigo
Add testcase |
141 |
|
5646.1.1
by Martin Pool
Use cmdline.split on BZR_EDITOR so that you can pass names including spaces. |
142 |
def make_do_nothing_editor(self, basename='fed'): |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
143 |
if sys.platform == "win32": |
5646.1.1
by Martin Pool
Use cmdline.split on BZR_EDITOR so that you can pass names including spaces. |
144 |
name = basename + '.bat' |
145 |
f = file(name, 'w') |
|
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
146 |
f.write('@rem dummy fed') |
147 |
f.close() |
|
5646.1.1
by Martin Pool
Use cmdline.split on BZR_EDITOR so that you can pass names including spaces. |
148 |
return name |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
149 |
else: |
5646.1.1
by Martin Pool
Use cmdline.split on BZR_EDITOR so that you can pass names including spaces. |
150 |
name = basename + '.sh' |
151 |
f = file(name, 'wb') |
|
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
152 |
f.write('#!/bin/sh\n') |
153 |
f.close() |
|
5646.1.1
by Martin Pool
Use cmdline.split on BZR_EDITOR so that you can pass names including spaces. |
154 |
os.chmod(name, 0755) |
155 |
return './' + name |
|
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
156 |
|
3910.1.1
by Andrew Bennetts
Improve error handling in msgeditor._run_editor. |
157 |
def test_run_editor(self): |
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
158 |
self.overrideEnv('BZR_EDITOR', self.make_do_nothing_editor()) |
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
159 |
self.assertEqual(True, msgeditor._run_editor(''), |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
160 |
'Unable to run dummy fake editor') |
161 |
||
5646.1.1
by Martin Pool
Use cmdline.split on BZR_EDITOR so that you can pass names including spaces. |
162 |
def test_parse_editor_name(self): |
163 |
"""Correctly interpret names with spaces.
|
|
164 |
||
165 |
See <https://bugs.launchpad.net/bzr/+bug/220331>
|
|
166 |
"""
|
|
167 |
self.overrideEnv('BZR_EDITOR', |
|
168 |
'"%s"' % self.make_do_nothing_editor('name with spaces')) |
|
169 |
self.assertEqual(True, msgeditor._run_editor('a_filename')) |
|
170 |
||
2625.9.5
by Daniel Watkins
Modified test_msgeditor.make_fake_editor to allow a custom message to be inserted. |
171 |
def make_fake_editor(self, message='test message from fed\\n'): |
2258.3.2
by James Westby
Allow an empty start message. |
172 |
"""Set up environment so that an editor will be a known script.
|
173 |
||
174 |
Sets up BZR_EDITOR so that if an editor is spawned it will run a
|
|
175 |
script that just adds a known message to the start of the file.
|
|
176 |
"""
|
|
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
177 |
f = file('fed.py', 'wb') |
178 |
f.write('#!%s\n' % sys.executable) |
|
179 |
f.write("""\ |
|
2625.9.5
by Daniel Watkins
Modified test_msgeditor.make_fake_editor to allow a custom message to be inserted. |
180 |
# coding=utf-8
|
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
181 |
import sys
|
182 |
if len(sys.argv) == 2:
|
|
183 |
fn = sys.argv[1]
|
|
184 |
f = file(fn, 'rb')
|
|
185 |
s = f.read()
|
|
186 |
f.close()
|
|
187 |
f = file(fn, 'wb')
|
|
2625.9.9
by Daniel Watkins
Used format string as suggested by abentley. |
188 |
f.write('%s') |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
189 |
f.write(s)
|
190 |
f.close()
|
|
2625.9.9
by Daniel Watkins
Used format string as suggested by abentley. |
191 |
""" % (message, )) |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
192 |
f.close() |
193 |
if sys.platform == "win32": |
|
194 |
# [win32] make batch file and set BZR_EDITOR
|
|
195 |
f = file('fed.bat', 'w') |
|
196 |
f.write("""\ |
|
197 |
@echo off
|
|
1711.4.2
by jfmeinel
current python may be running in a path that has a space, so properly quote the python exe name. for test_msgeditor |
198 |
"%s" fed.py %%1 |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
199 |
""" % sys.executable) |
200 |
f.close() |
|
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
201 |
self.overrideEnv('BZR_EDITOR', 'fed.bat') |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
202 |
else: |
203 |
# [non-win32] make python script executable and set BZR_EDITOR
|
|
204 |
os.chmod('fed.py', 0755) |
|
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
205 |
self.overrideEnv('BZR_EDITOR', './fed.py') |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
206 |
|
2258.3.2
by James Westby
Allow an empty start message. |
207 |
def test_edit_commit_message(self): |
208 |
working_tree = self.make_uncommitted_tree() |
|
209 |
self.make_fake_editor() |
|
210 |
||
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
211 |
mutter('edit_commit_message without infotext') |
212 |
self.assertEqual('test message from fed\n', |
|
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
213 |
msgeditor.edit_commit_message('')) |
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
214 |
|
2804.4.1
by Alexander Belchenko
some win32-specific fixes for selftest |
215 |
mutter('edit_commit_message with ascii string infotext') |
216 |
self.assertEqual('test message from fed\n', |
|
217 |
msgeditor.edit_commit_message('spam')) |
|
218 |
||
1185.85.9
by John Arbash Meinel
[patch] Alexander Belchenko: test spawning a msg editor |
219 |
mutter('edit_commit_message with unicode infotext') |
220 |
self.assertEqual('test message from fed\n', |
|
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
221 |
msgeditor.edit_commit_message(u'\u1234')) |
2598.6.18
by ghigo
Update the tests to the new *_encoded() functions |
222 |
|
2598.6.27
by ghigo
small cleanup (line lenght) |
223 |
tmpl = edit_commit_message_encoded(u'\u1234'.encode("utf8")) |
224 |
self.assertEqual('test message from fed\n', tmpl) |
|
1185.85.14
by John Arbash Meinel
Change exception handling for msgeditor.py to only catch specific exceptions. |
225 |
|
2258.3.2
by James Westby
Allow an empty start message. |
226 |
def test_start_message(self): |
227 |
self.make_uncommitted_tree() |
|
228 |
self.make_fake_editor() |
|
2258.3.1
by James Westby
Add a way to specify a template commit message. |
229 |
self.assertEqual('test message from fed\nstart message\n', |
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
230 |
msgeditor.edit_commit_message('', |
2258.3.1
by James Westby
Add a way to specify a template commit message. |
231 |
start_message='start message\n')) |
2258.3.2
by James Westby
Allow an empty start message. |
232 |
self.assertEqual('test message from fed\n', |
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
233 |
msgeditor.edit_commit_message('', |
2258.3.2
by James Westby
Allow an empty start message. |
234 |
start_message='')) |
2258.3.1
by James Westby
Add a way to specify a template commit message. |
235 |
|
1185.85.14
by John Arbash Meinel
Change exception handling for msgeditor.py to only catch specific exceptions. |
236 |
def test_deleted_commit_message(self): |
237 |
working_tree = self.make_uncommitted_tree() |
|
238 |
||
239 |
if sys.platform == 'win32': |
|
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
240 |
editor = 'cmd.exe /c del' |
1185.85.14
by John Arbash Meinel
Change exception handling for msgeditor.py to only catch specific exceptions. |
241 |
else: |
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
242 |
editor = 'rm' |
243 |
self.overrideEnv('BZR_EDITOR', editor) |
|
1185.85.14
by John Arbash Meinel
Change exception handling for msgeditor.py to only catch specific exceptions. |
244 |
|
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
245 |
self.assertRaises((IOError, OSError), msgeditor.edit_commit_message, '') |
1185.85.14
by John Arbash Meinel
Change exception handling for msgeditor.py to only catch specific exceptions. |
246 |
|
1185.50.93
by John Arbash Meinel
Added a test for the new list of editors. |
247 |
def test__get_editor(self): |
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
248 |
self.overrideEnv('BZR_EDITOR', 'bzr_editor') |
249 |
self.overrideEnv('VISUAL', 'visual') |
|
250 |
self.overrideEnv('EDITOR', 'editor') |
|
5345.1.13
by Vincent Ladeuil
Cleanup test_msgeditor. |
251 |
|
6260.3.2
by Vincent Ladeuil
Only the DEFAULT section is searched for the normal uses of bazaar.conf |
252 |
conf = config.GlobalStack() |
253 |
conf.store._load_from_string('[DEFAULT]\neditor = config_editor\n') |
|
254 |
conf.store.save() |
|
5345.1.13
by Vincent Ladeuil
Cleanup test_msgeditor. |
255 |
editors = list(msgeditor._get_editor()) |
256 |
editors = [editor for (editor, cfg_src) in editors] |
|
257 |
||
258 |
self.assertEqual(['bzr_editor', 'config_editor', 'visual', 'editor'], |
|
259 |
editors[:4]) |
|
260 |
||
261 |
if sys.platform == 'win32': |
|
262 |
self.assertEqual(['wordpad.exe', 'notepad.exe'], editors[4:]) |
|
263 |
else: |
|
264 |
self.assertEqual(['/usr/bin/editor', 'vi', 'pico', 'nano', 'joe'], |
|
265 |
editors[4:]) |
|
266 |
||
2472.4.1
by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added |
267 |
|
3910.1.1
by Andrew Bennetts
Improve error handling in msgeditor._run_editor. |
268 |
def test__run_editor_EACCES(self): |
269 |
"""If running a configured editor raises EACESS, the user is warned."""
|
|
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
270 |
self.overrideEnv('BZR_EDITOR', 'eacces.py') |
3910.1.1
by Andrew Bennetts
Improve error handling in msgeditor._run_editor. |
271 |
f = file('eacces.py', 'wb') |
272 |
f.write('# Not a real editor') |
|
273 |
f.close() |
|
274 |
# Make the fake editor unreadable (and unexecutable)
|
|
275 |
os.chmod('eacces.py', 0) |
|
276 |
# Set $EDITOR so that _run_editor will terminate before trying real
|
|
277 |
# editors.
|
|
5570.3.9
by Vincent Ladeuil
More use cases for overrideEnv, _cleanEnvironment *may* contain too much variables now. |
278 |
self.overrideEnv('EDITOR', self.make_do_nothing_editor()) |
3910.1.1
by Andrew Bennetts
Improve error handling in msgeditor._run_editor. |
279 |
# Call _run_editor, capturing mutter.warning calls.
|
280 |
warnings = [] |
|
281 |
def warning(*args): |
|
4789.23.1
by John Arbash Meinel
Spawning a non-executable file raises ENOEXEC on windows, not EACCESS. |
282 |
if len(args) > 1: |
283 |
warnings.append(args[0] % args[1:]) |
|
284 |
else: |
|
285 |
warnings.append(args[0]) |
|
3910.1.1
by Andrew Bennetts
Improve error handling in msgeditor._run_editor. |
286 |
_warning = trace.warning |
287 |
trace.warning = warning |
|
288 |
try: |
|
289 |
msgeditor._run_editor('') |
|
290 |
finally: |
|
291 |
trace.warning = _warning |
|
3910.1.3
by Andrew Bennetts
Tweak warning slightly. |
292 |
self.assertStartsWith(warnings[0], 'Could not start editor "eacces.py"') |
3910.1.1
by Andrew Bennetts
Improve error handling in msgeditor._run_editor. |
293 |
|
2472.4.1
by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added |
294 |
def test__create_temp_file_with_commit_template(self): |
295 |
# check that commit template written properly
|
|
296 |
# and has platform native line-endings (CRLF on win32)
|
|
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
297 |
create_file = msgeditor._create_temp_file_with_commit_template |
2472.4.2
by Alexander Belchenko
unwrapping long lines in tests |
298 |
msgfilename, hasinfo = create_file('infotext','----','start message') |
2472.4.1
by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added |
299 |
self.assertNotEqual(None, msgfilename) |
300 |
self.assertTrue(hasinfo) |
|
2472.4.2
by Alexander Belchenko
unwrapping long lines in tests |
301 |
expected = os.linesep.join(['start message', |
302 |
'', |
|
303 |
'', |
|
304 |
'----', |
|
305 |
'', |
|
306 |
'infotext']) |
|
2472.4.1
by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added |
307 |
self.assertFileEqual(expected, msgfilename) |
308 |
||
3004.1.1
by Daniel Watkins
Added (failing) test to check if a temp file containing a commit message can be created in a folder with a Unicode name. |
309 |
def test__create_temp_file_with_commit_template_in_unicode_dir(self): |
5967.12.1
by Martin Pool
Move all test features into bzrlib.tests.features |
310 |
self.requireFeature(features.UnicodeFilenameFeature) |
3004.1.7
by Daniel Watkins
test__create_temp_file_with_commit_template_in_unicode_dir now uses EncodingTestAdapter info. |
311 |
if hasattr(self, 'info'): |
5537.1.1
by Vincent Ladeuil
Create commit message files in TMPDIR instead of the current dir |
312 |
tmpdir = self.info['directory'] |
313 |
os.mkdir(tmpdir) |
|
314 |
# Force the creation of temp file in a directory whose name
|
|
315 |
# requires some encoding support
|
|
316 |
msgeditor._create_temp_file_with_commit_template('infotext', |
|
317 |
tmpdir=tmpdir) |
|
3004.1.7
by Daniel Watkins
test__create_temp_file_with_commit_template_in_unicode_dir now uses EncodingTestAdapter info. |
318 |
else: |
3004.1.8
by Daniel Watkins
Changed TestSkipped to TestNotApplicable as suggested by Aaron. |
319 |
raise TestNotApplicable('Test run elsewhere with non-ascii data.') |
3004.1.1
by Daniel Watkins
Added (failing) test to check if a temp file containing a commit message can be created in a folder with a Unicode name. |
320 |
|
2472.4.1
by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added |
321 |
def test__create_temp_file_with_empty_commit_template(self): |
322 |
# empty file
|
|
2772.1.1
by Martin Pool
Merge commit --show-diff feature from Goffredo |
323 |
create_file = msgeditor._create_temp_file_with_commit_template |
2472.4.2
by Alexander Belchenko
unwrapping long lines in tests |
324 |
msgfilename, hasinfo = create_file('') |
2472.4.1
by Alexander Belchenko
Bugfix #110901: commit message template written with native line-endings; corresponding unit tests added |
325 |
self.assertNotEqual(None, msgfilename) |
326 |
self.assertFalse(hasinfo) |
|
327 |
self.assertFileEqual('', msgfilename) |
|
2625.9.6
by Daniel Watkins
Added test to ensure correct error message is thrown when an unencodable commit message is entered through the editor. |
328 |
|
329 |
def test_unsupported_encoding_commit_message(self): |
|
5570.3.12
by Vincent Ladeuil
Replace osutils.set_or_unset_env calls with self.overrideEnv. |
330 |
self.overrideEnv('LANG', 'C') |
331 |
# LANG env variable has no effect on Windows
|
|
332 |
# but some characters anyway cannot be represented
|
|
333 |
# in default user encoding
|
|
334 |
char = probe_bad_non_ascii(osutils.get_user_encoding()) |
|
335 |
if char is None: |
|
336 |
raise TestSkipped('Cannot find suitable non-ascii character ' |
|
337 |
'for user_encoding (%s)' % osutils.get_user_encoding()) |
|
338 |
||
339 |
self.make_fake_editor(message=char) |
|
340 |
||
341 |
working_tree = self.make_uncommitted_tree() |
|
342 |
self.assertRaises(errors.BadCommitMessageEncoding, |
|
343 |
msgeditor.edit_commit_message, '') |
|
3642.2.2
by Jelmer Vernooij
Add tests for commit_message_template hooks. |
344 |
|
5912.4.4
by Jonathan Riddell
add test |
345 |
def test_set_commit_message_no_hooks(self): |
346 |
commit_obj = commit.Commit() |
|
347 |
self.assertIs(None, |
|
348 |
msgeditor.set_commit_message(commit_obj)) |
|
349 |
||
350 |
def test_set_commit_message_hook(self): |
|
351 |
msgeditor.hooks.install_named_hook("set_commit_message", |
|
5912.4.10
by Jonathan Riddell
add test for set_commit_message hook |
352 |
lambda commit_obj, existing_message: "save me some typing\n", None) |
5912.4.4
by Jonathan Riddell
add test |
353 |
commit_obj = commit.Commit() |
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
354 |
self.assertEqual("save me some typing\n", |
5912.4.4
by Jonathan Riddell
add test |
355 |
msgeditor.set_commit_message(commit_obj)) |
356 |
||
3642.2.2
by Jelmer Vernooij
Add tests for commit_message_template hooks. |
357 |
def test_generate_commit_message_template_no_hooks(self): |
358 |
commit_obj = commit.Commit() |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
359 |
self.assertIs(None, |
3642.2.2
by Jelmer Vernooij
Add tests for commit_message_template hooks. |
360 |
msgeditor.generate_commit_message_template(commit_obj)) |
361 |
||
362 |
def test_generate_commit_message_template_hook(self): |
|
363 |
msgeditor.hooks.install_named_hook("commit_message_template", |
|
364 |
lambda commit_obj, msg: "save me some typing\n", None) |
|
365 |
commit_obj = commit.Commit() |
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
366 |
self.assertEqual("save me some typing\n", |
3642.2.2
by Jelmer Vernooij
Add tests for commit_message_template hooks. |
367 |
msgeditor.generate_commit_message_template(commit_obj)) |
4789.23.3
by Martin
Test that msgeditor errno workarounds for windows are actually required |
368 |
|
369 |
||
370 |
# GZ 2009-11-17: This wants moving to osutils when the errno checking code is
|
|
371 |
class TestPlatformErrnoWorkarounds(TestCaseInTempDir): |
|
372 |
"""Ensuring workarounds enshrined in code actually serve a purpose"""
|
|
4805.1.4
by John Arbash Meinel
Document a bit more what 193 means, and why we are testing it. |
373 |
|
374 |
def test_subprocess_call_bad_file(self): |
|
4789.23.3
by Martin
Test that msgeditor errno workarounds for windows are actually required |
375 |
if sys.platform != "win32": |
4805.1.4
by John Arbash Meinel
Document a bit more what 193 means, and why we are testing it. |
376 |
raise TestNotApplicable("Workarounds for windows only") |
4789.23.3
by Martin
Test that msgeditor errno workarounds for windows are actually required |
377 |
import subprocess, errno |
378 |
ERROR_BAD_EXE_FORMAT = 193 |
|
379 |
file("textfile.txt", "w").close() |
|
380 |
e = self.assertRaises(WindowsError, subprocess.call, "textfile.txt") |
|
5848.2.1
by John Arbash Meinel
Break compatibility with python <2.6. |
381 |
self.assertEqual(e.errno, errno.ENOEXEC) |
382 |
self.assertEqual(e.winerror, ERROR_BAD_EXE_FORMAT) |