/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 bzrlib/tests/blackbox/test_diff.py

  • Committer: Michael Ellerman
  • Date: 2006-05-31 08:44:29 UTC
  • mto: (1711.2.63 jam-integration)
  • mto: This revision was merged to the branch mainline in revision 1792.
  • Revision ID: michael@ellerman.id.au-20060531084429-35e5429abda9f560
Add optional location to ancestry and fix behaviour for checkouts.

This adds an optional location parameter to the ancestry command. It also
changes the behaviour of ancestry on checkouts such that if they have
been created with a subset of the branch history, only the subset is
shown by 'bzr ancestry'. Tests for all of that as well.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006-2010 Canonical Ltd
2
 
#
 
1
# Copyright (C) 2005, 2006 by Canonical Ltd
 
2
 
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
5
5
# the Free Software Foundation; either version 2 of the License, or
6
6
# (at your option) any later version.
7
 
#
 
7
 
8
8
# This program is distributed in the hope that it will be useful,
9
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
11
# GNU General Public License for more details.
12
 
#
 
12
 
13
13
# You should have received a copy of the GNU General Public License
14
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
 
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
16
 
17
17
 
18
18
"""Black-box tests for bzr diff.
19
19
"""
20
20
 
21
21
import os
22
 
import re
23
 
 
24
 
from bzrlib import (
25
 
    tests,
26
 
    workingtree,
27
 
    )
28
 
from bzrlib.diff import (
29
 
    DiffTree,
30
 
    format_registry as diff_format_registry,
31
 
    )
32
 
 
33
 
 
34
 
def subst_dates(string):
35
 
    """Replace date strings with constant values."""
36
 
    return re.sub(r'\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} [-\+]\d{4}',
37
 
                  'YYYY-MM-DD HH:MM:SS +ZZZZ', string)
38
 
 
39
 
 
40
 
class DiffBase(tests.TestCaseWithTransport):
41
 
    """Base class with common setup method"""
42
 
 
43
 
    def make_example_branch(self):
44
 
        tree = self.make_branch_and_tree('.')
45
 
        self.build_tree_contents([
46
 
            ('hello', 'foo\n'),
47
 
            ('goodbye', 'baz\n')])
48
 
        tree.add(['hello'])
49
 
        tree.commit('setup')
50
 
        tree.add(['goodbye'])
51
 
        tree.commit('setup')
52
 
        return tree
53
 
 
54
 
 
55
 
class TestDiff(DiffBase):
 
22
 
 
23
import bzrlib
 
24
from bzrlib.branch import Branch
 
25
from bzrlib.tests.blackbox import ExternalBase
 
26
 
 
27
 
 
28
class TestDiff(ExternalBase):
 
29
 
 
30
    def make_example_branch(test):
 
31
        # FIXME: copied from test_too_much -- share elsewhere?
 
32
        test.runbzr('init')
 
33
        file('hello', 'wt').write('foo\n')
 
34
        test.runbzr('add hello')
 
35
        test.runbzr('commit -m setup hello')
 
36
        file('goodbye', 'wt').write('baz\n')
 
37
        test.runbzr('add goodbye')
 
38
        test.runbzr('commit -m setup goodbye')
56
39
 
57
40
    def test_diff(self):
58
 
        tree = self.make_example_branch()
59
 
        self.build_tree_contents([('hello', 'hello world!')])
60
 
        tree.commit(message='fixing hello')
61
 
        output = self.run_bzr('diff -r 2..3', retcode=1)[0]
62
 
        self.assert_('\n+hello world!' in output)
63
 
        output = self.run_bzr('diff -c 3', retcode=1)[0]
64
 
        self.assert_('\n+hello world!' in output)
65
 
        output = self.run_bzr('diff -r last:3..last:1', retcode=1)[0]
66
 
        self.assert_('\n+baz' in output)
67
 
        output = self.run_bzr('diff -c last:2', retcode=1)[0]
68
 
        self.assert_('\n+baz' in output)
69
 
        self.build_tree(['moo'])
70
 
        tree.add('moo')
 
41
        self.make_example_branch()
 
42
        file('hello', 'wt').write('hello world!')
 
43
        self.runbzr('commit -m fixing hello')
 
44
        output = self.runbzr('diff -r 2..3', backtick=1, retcode=1)
 
45
        self.assert_('\n+hello world!' in output)
 
46
        output = self.runbzr('diff -r last:3..last:1', backtick=1, retcode=1)
 
47
        self.assert_('\n+baz' in output)
 
48
        file('moo', 'wb').write('moo')
 
49
        self.runbzr('add moo')
71
50
        os.unlink('moo')
72
 
        self.run_bzr('diff')
 
51
        self.runbzr('diff')
73
52
 
74
53
    def test_diff_prefix(self):
75
54
        """diff --prefix appends to filenames in output"""
76
55
        self.make_example_branch()
77
 
        self.build_tree_contents([('hello', 'hello world!\n')])
78
 
        out, err = self.run_bzr('diff --prefix old/:new/', retcode=1)
 
56
        file('hello', 'wt').write('hello world!\n')
 
57
        out, err = self.runbzr('diff --prefix old/:new/', retcode=1)
79
58
        self.assertEquals(err, '')
80
 
        self.assertEqualDiff(subst_dates(out), '''\
 
59
        self.assertEqualDiff(out, '''\
81
60
=== modified file 'hello'
82
 
--- old/hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
83
 
+++ new/hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
 
61
--- old/hello\t
 
62
+++ new/hello\t
84
63
@@ -1,1 +1,1 @@
85
64
-foo
86
65
+hello world!
87
66
 
88
67
''')
89
68
 
90
 
    def test_diff_illegal_prefix_value(self):
91
 
        # There was an error in error reporting for this option
92
 
        out, err = self.run_bzr('diff --prefix old/', retcode=3)
93
 
        self.assertContainsRe(err,
94
 
            '--prefix expects two values separated by a colon')
95
 
 
96
69
    def test_diff_p1(self):
97
70
        """diff -p1 produces lkml-style diffs"""
98
71
        self.make_example_branch()
99
 
        self.build_tree_contents([('hello', 'hello world!\n')])
100
 
        out, err = self.run_bzr('diff -p1', retcode=1)
 
72
        file('hello', 'wt').write('hello world!\n')
 
73
        out, err = self.runbzr('diff -p1', retcode=1)
101
74
        self.assertEquals(err, '')
102
 
        self.assertEqualDiff(subst_dates(out), '''\
 
75
        self.assertEqualDiff(out, '''\
103
76
=== modified file 'hello'
104
 
--- old/hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
105
 
+++ new/hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
 
77
--- old/hello\t
 
78
+++ new/hello\t
106
79
@@ -1,1 +1,1 @@
107
80
-foo
108
81
+hello world!
112
85
    def test_diff_p0(self):
113
86
        """diff -p0 produces diffs with no prefix"""
114
87
        self.make_example_branch()
115
 
        self.build_tree_contents([('hello', 'hello world!\n')])
116
 
        out, err = self.run_bzr('diff -p0', retcode=1)
 
88
        file('hello', 'wt').write('hello world!\n')
 
89
        out, err = self.runbzr('diff -p0', retcode=1)
117
90
        self.assertEquals(err, '')
118
 
        self.assertEqualDiff(subst_dates(out), '''\
 
91
        self.assertEqualDiff(out, '''\
119
92
=== modified file 'hello'
120
 
--- hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
121
 
+++ hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
 
93
--- hello\t
 
94
+++ hello\t
122
95
@@ -1,1 +1,1 @@
123
96
-foo
124
97
+hello world!
129
102
        # Get an error from a file that does not exist at all
130
103
        # (Malone #3619)
131
104
        self.make_example_branch()
132
 
        out, err = self.run_bzr('diff does-not-exist', retcode=3,
133
 
            error_regexes=('not versioned.*does-not-exist',))
134
 
 
135
 
    def test_diff_illegal_revision_specifiers(self):
136
 
        out, err = self.run_bzr('diff -r 1..23..123', retcode=3,
137
 
            error_regexes=('one or two revision specifiers',))
138
 
 
139
 
    def test_diff_using_and_format(self):
140
 
        out, err = self.run_bzr('diff --format=default --using=mydi', retcode=3,
141
 
            error_regexes=('are mutually exclusive',))
142
 
 
143
 
    def test_diff_nonexistent_revision(self):
144
 
        out, err = self.run_bzr('diff -r 123', retcode=3,
145
 
            error_regexes=("Requested revision: '123' does not "
146
 
                "exist in branch:",))
147
 
 
148
 
    def test_diff_nonexistent_dotted_revision(self):
149
 
        out, err = self.run_bzr('diff -r 1.1', retcode=3)
150
 
        self.assertContainsRe(err,
151
 
            "Requested revision: '1.1' does not exist in branch:")
152
 
 
153
 
    def test_diff_nonexistent_dotted_revision_change(self):
154
 
        out, err = self.run_bzr('diff -c 1.1', retcode=3)
155
 
        self.assertContainsRe(err,
156
 
            "Requested revision: '1.1' does not exist in branch:")
 
105
        out, err = self.runbzr('diff does-not-exist', retcode=3)
 
106
        self.assertContainsRe(err, 'not versioned.*does-not-exist')
157
107
 
158
108
    def test_diff_unversioned(self):
159
109
        # Get an error when diffing a non-versioned file.
160
110
        # (Malone #3619)
161
111
        self.make_example_branch()
162
112
        self.build_tree(['unversioned-file'])
163
 
        out, err = self.run_bzr('diff unversioned-file', retcode=3)
 
113
        out, err = self.runbzr('diff unversioned-file', retcode=3)
164
114
        self.assertContainsRe(err, 'not versioned.*unversioned-file')
165
115
 
166
116
    # TODO: What should diff say for a file deleted in working tree?
167
117
 
168
118
    def example_branches(self):
169
 
        branch1_tree = self.make_branch_and_tree('branch1')
170
 
        self.build_tree(['branch1/file'], line_endings='binary')
171
 
        self.build_tree(['branch1/file2'], line_endings='binary')
172
 
        branch1_tree.add('file')
173
 
        branch1_tree.add('file2')
174
 
        branch1_tree.commit(message='add file and file2')
175
 
        branch2_tree = branch1_tree.bzrdir.sprout('branch2').open_workingtree()
176
 
        self.build_tree_contents([('branch2/file', 'new content\n')])
177
 
        branch2_tree.commit(message='update file')
178
 
        return branch1_tree, branch2_tree
179
 
 
180
 
    def check_b2_vs_b1(self, cmd):
181
 
        # Compare branch2 vs branch1 using cmd and check the result
182
 
        out, err = self.run_bzr(cmd, retcode=1)
183
 
        self.assertEquals('', err)
184
 
        self.assertEquals("=== modified file 'file'\n"
185
 
                          "--- file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
186
 
                          "+++ file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
187
 
                          "@@ -1,1 +1,1 @@\n"
188
 
                          "-new content\n"
189
 
                          "+contents of branch1/file\n"
190
 
                          "\n", subst_dates(out))
191
 
 
192
 
    def check_b1_vs_b2(self, cmd):
193
 
        # Compare branch1 vs branch2 using cmd and check the result
194
 
        out, err = self.run_bzr(cmd, retcode=1)
195
 
        self.assertEquals('', err)
196
 
        self.assertEqualDiff("=== modified file 'file'\n"
197
 
                              "--- file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
198
 
                              "+++ file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
199
 
                              "@@ -1,1 +1,1 @@\n"
200
 
                              "-contents of branch1/file\n"
201
 
                              "+new content\n"
202
 
                              "\n", subst_dates(out))
203
 
 
204
 
    def check_no_diffs(self, cmd):
205
 
        # Check that running cmd returns an empty diff
206
 
        out, err = self.run_bzr(cmd, retcode=0)
207
 
        self.assertEquals('', err)
208
 
        self.assertEquals('', out)
 
119
        self.build_tree(['branch1/', 'branch1/file'], line_endings='binary')
 
120
        self.capture('init branch1')
 
121
        self.capture('add branch1/file')
 
122
        self.run_bzr_captured(['commit', '-m', 'add file', 'branch1'])
 
123
        self.capture('branch branch1 branch2')
 
124
        print >> open('branch2/file', 'wb'), 'new content'
 
125
        self.run_bzr_captured(['commit', '-m', 'update file', 'branch2'])
209
126
 
210
127
    def test_diff_branches(self):
211
128
        self.example_branches()
212
 
        # should open branch1 and diff against branch2,
213
 
        self.check_b2_vs_b1('diff -r branch:branch2 branch1')
214
 
        # Compare two working trees using various syntax forms
215
 
        self.check_b2_vs_b1('diff --old branch2 --new branch1')
216
 
        self.check_b2_vs_b1('diff --old branch2 branch1')
217
 
        self.check_b2_vs_b1('diff branch2 --new branch1')
218
 
        # Test with a selected file that was changed
219
 
        self.check_b2_vs_b1('diff --old branch2 --new branch1 file')
220
 
        self.check_b2_vs_b1('diff --old branch2 branch1/file')
221
 
        self.check_b2_vs_b1('diff branch2/file --new branch1')
222
 
        # Test with a selected file that was not changed
223
 
        self.check_no_diffs('diff --old branch2 --new branch1 file2')
224
 
        self.check_no_diffs('diff --old branch2 branch1/file2')
225
 
        self.check_no_diffs('diff branch2/file2 --new branch1')
226
 
 
227
 
    def test_diff_branches_no_working_trees(self):
228
 
        branch1_tree, branch2_tree = self.example_branches()
229
 
        # Compare a working tree to a branch without a WT
230
 
        dir1 = branch1_tree.bzrdir
231
 
        dir1.destroy_workingtree()
232
 
        self.assertFalse(dir1.has_workingtree())
233
 
        self.check_b2_vs_b1('diff --old branch2 --new branch1')
234
 
        self.check_b2_vs_b1('diff --old branch2 branch1')
235
 
        self.check_b2_vs_b1('diff branch2 --new branch1')
236
 
        # Compare a branch without a WT to one with a WT
237
 
        self.check_b1_vs_b2('diff --old branch1 --new branch2')
238
 
        self.check_b1_vs_b2('diff --old branch1 branch2')
239
 
        self.check_b1_vs_b2('diff branch1 --new branch2')
240
 
        # Compare a branch with a WT against another without a WT
241
 
        dir2 = branch2_tree.bzrdir
242
 
        dir2.destroy_workingtree()
243
 
        self.assertFalse(dir2.has_workingtree())
244
 
        self.check_b1_vs_b2('diff --old branch1 --new branch2')
245
 
        self.check_b1_vs_b2('diff --old branch1 branch2')
246
 
        self.check_b1_vs_b2('diff branch1 --new branch2')
247
 
 
248
 
    def test_diff_revno_branches(self):
249
 
        self.example_branches()
250
 
        branch2_tree = workingtree.WorkingTree.open_containing('branch2')[0]
251
 
        self.build_tree_contents([('branch2/file', 'even newer content')])
252
 
        branch2_tree.commit(message='update file once more')
253
 
 
254
 
        out, err = self.run_bzr('diff -r revno:1:branch2..revno:1:branch1',
255
 
                                )
256
 
        self.assertEquals('', err)
257
 
        self.assertEquals('', out)
258
 
        out, err = self.run_bzr('diff -r revno:2:branch2..revno:1:branch1',
259
 
                                retcode=1)
260
 
        self.assertEquals('', err)
261
 
        self.assertEqualDiff("=== modified file 'file'\n"
262
 
                              "--- file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
263
 
                              "+++ file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
 
129
        # should open branch1 and diff against branch2, 
 
130
        output = self.run_bzr_captured(['diff', '-r', 'branch:branch2', 
 
131
                                        'branch1'],
 
132
                                       retcode=1)
 
133
        self.assertEquals(("=== modified file 'file'\n"
 
134
                           "--- file\t\n"
 
135
                           "+++ file\t\n"
 
136
                           "@@ -1,1 +1,1 @@\n"
 
137
                           "-new content\n"
 
138
                           "+contents of branch1/file\n"
 
139
                           "\n", ''), output)
 
140
        output = self.run_bzr_captured(['diff', 'branch2', 'branch1'],
 
141
                                       retcode=1)
 
142
        self.assertEqualDiff(("=== modified file 'file'\n"
 
143
                              "--- file\t\n"
 
144
                              "+++ file\t\n"
264
145
                              "@@ -1,1 +1,1 @@\n"
265
146
                              "-new content\n"
266
147
                              "+contents of branch1/file\n"
267
 
                              "\n", subst_dates(out))
 
148
                              "\n", ''), output)
268
149
 
269
150
    def example_branch2(self):
270
 
        branch1_tree = self.make_branch_and_tree('branch1')
271
 
        self.build_tree_contents([('branch1/file1', 'original line\n')])
272
 
        branch1_tree.add('file1')
273
 
        branch1_tree.commit(message='first commit')
274
 
        self.build_tree_contents([('branch1/file1', 'repo line\n')])
275
 
        branch1_tree.commit(message='second commit')
276
 
        return branch1_tree
 
151
        self.build_tree(['branch1/', 'branch1/file1'], line_endings='binary')
 
152
        self.capture('init branch1')
 
153
        self.capture('add branch1/file1')
 
154
        print >> open('branch1/file1', 'wb'), 'original line'
 
155
        self.run_bzr_captured(['commit', '-m', 'first commit', 'branch1'])
 
156
        
 
157
        print >> open('branch1/file1', 'wb'), 'repo line'
 
158
        self.run_bzr_captured(['commit', '-m', 'second commit', 'branch1'])
277
159
 
278
160
    def test_diff_to_working_tree(self):
279
161
        self.example_branch2()
280
 
        self.build_tree_contents([('branch1/file1', 'new line')])
281
 
        output = self.run_bzr('diff -r 1.. branch1', retcode=1)
282
 
        self.assertContainsRe(output[0], '\n\\-original line\n\\+new line\n')
283
 
 
284
 
    def test_diff_to_working_tree_in_subdir(self):
285
 
        self.example_branch2()
286
 
        self.build_tree_contents([('branch1/file1', 'new line')])
287
 
        os.mkdir('branch1/dir1')
288
 
        os.chdir('branch1/dir1')
289
 
        output = self.run_bzr('diff -r 1..', retcode=1)
290
 
        self.assertContainsRe(output[0], '\n\\-original line\n\\+new line\n')
291
 
 
292
 
    def test_diff_across_rename(self):
293
 
        """The working tree path should always be considered for diffing"""
294
 
        tree = self.make_example_branch()
295
 
        self.run_bzr('diff -r 0..1 hello', retcode=1)
296
 
        tree.rename_one('hello', 'hello1')
297
 
        self.run_bzr('diff hello1', retcode=1)
298
 
        self.run_bzr('diff -r 0..1 hello1', retcode=1)
299
 
 
300
 
    def test_diff_to_branch_no_working_tree(self):
301
 
        branch1_tree = self.example_branch2()
302
 
        dir1 = branch1_tree.bzrdir
303
 
        dir1.destroy_workingtree()
304
 
        self.assertFalse(dir1.has_workingtree())
305
 
        output = self.run_bzr('diff -r 1.. branch1', retcode=1)
306
 
        self.assertContainsRe(output[0], '\n\\-original line\n\\+repo line\n')
307
 
 
308
 
    def test_custom_format(self):
309
 
        class BooDiffTree(DiffTree):
310
 
 
311
 
            def show_diff(self, specific_files, extra_trees=None):
312
 
                self.to_file.write("BOO!\n")
313
 
                return super(BooDiffTree, self).show_diff(specific_files,
314
 
                    extra_trees)
315
 
 
316
 
        diff_format_registry.register("boo", BooDiffTree, 
317
 
            "Scary diff format")
318
 
        self.addCleanup(diff_format_registry.remove, "boo")
319
 
        self.make_example_branch()
320
 
        self.build_tree_contents([('hello', 'hello world!\n')])
321
 
        output = self.run_bzr('diff --format=boo', retcode=1)
322
 
        self.assertTrue("BOO!" in output[0])
 
162
        
 
163
        print >> open('branch1/file1', 'wb'), 'new line'
 
164
        output = self.run_bzr_captured(['diff', '-r', '1..', 'branch1'], retcode=1)
 
165
        self.assertTrue('\n-original line\n+new line\n' in output[0])
323
166
 
324
167
 
325
168
class TestCheckoutDiff(TestDiff):
326
169
 
327
170
    def make_example_branch(self):
328
 
        tree = super(TestCheckoutDiff, self).make_example_branch()
329
 
        tree = tree.branch.create_checkout('checkout')
 
171
        super(TestCheckoutDiff, self).make_example_branch()
 
172
        self.runbzr('checkout . checkout')
330
173
        os.chdir('checkout')
331
 
        return tree
332
174
 
333
175
    def example_branch2(self):
334
 
        tree = super(TestCheckoutDiff, self).example_branch2()
 
176
        super(TestCheckoutDiff, self).example_branch2()
335
177
        os.mkdir('checkouts')
336
 
        tree = tree.branch.create_checkout('checkouts/branch1')
 
178
        self.runbzr('checkout branch1 checkouts/branch1')
337
179
        os.chdir('checkouts')
338
 
        return tree
339
180
 
340
181
    def example_branches(self):
341
 
        branch1_tree, branch2_tree = super(TestCheckoutDiff, self).example_branches()
 
182
        super(TestCheckoutDiff, self).example_branches()
342
183
        os.mkdir('checkouts')
343
 
        branch1_tree = branch1_tree.branch.create_checkout('checkouts/branch1')
344
 
        branch2_tree = branch2_tree.branch.create_checkout('checkouts/branch2')
 
184
        self.runbzr('checkout branch1 checkouts/branch1')
 
185
        self.runbzr('checkout branch2 checkouts/branch2')
345
186
        os.chdir('checkouts')
346
 
        return branch1_tree, branch2_tree
347
 
 
348
 
 
349
 
class TestDiffLabels(DiffBase):
 
187
 
 
188
 
 
189
class TestDiffLabels(TestDiff):
350
190
 
351
191
    def test_diff_label_removed(self):
352
 
        tree = super(TestDiffLabels, self).make_example_branch()
353
 
        tree.remove('hello', keep_files=False)
354
 
        diff = self.run_bzr('diff', retcode=1)
 
192
        super(TestDiffLabels, self).make_example_branch()
 
193
        self.runbzr('remove hello')
 
194
        diff = self.run_bzr_captured(['diff'], retcode=1)
355
195
        self.assertTrue("=== removed file 'hello'" in diff[0])
356
196
 
357
197
    def test_diff_label_added(self):
358
 
        tree = super(TestDiffLabels, self).make_example_branch()
359
 
        self.build_tree_contents([('barbar', 'barbar')])
360
 
        tree.add('barbar')
361
 
        diff = self.run_bzr('diff', retcode=1)
 
198
        super(TestDiffLabels, self).make_example_branch()
 
199
        file('barbar', 'wt').write('barbar')
 
200
        self.runbzr('add barbar')
 
201
        diff = self.run_bzr_captured(['diff'], retcode=1)
362
202
        self.assertTrue("=== added file 'barbar'" in diff[0])
363
203
 
364
204
    def test_diff_label_modified(self):
365
205
        super(TestDiffLabels, self).make_example_branch()
366
 
        self.build_tree_contents([('hello', 'barbar')])
367
 
        diff = self.run_bzr('diff', retcode=1)
 
206
        file('hello', 'wt').write('barbar')
 
207
        diff = self.run_bzr_captured(['diff'], retcode=1)
368
208
        self.assertTrue("=== modified file 'hello'" in diff[0])
369
209
 
370
210
    def test_diff_label_renamed(self):
371
 
        tree = super(TestDiffLabels, self).make_example_branch()
372
 
        tree.rename_one('hello', 'gruezi')
373
 
        diff = self.run_bzr('diff', retcode=1)
 
211
        super(TestDiffLabels, self).make_example_branch()
 
212
        self.runbzr('rename hello gruezi')
 
213
        diff = self.run_bzr_captured(['diff'], retcode=1)
374
214
        self.assertTrue("=== renamed file 'hello' => 'gruezi'" in diff[0])
375
 
 
376
 
 
377
 
class TestExternalDiff(DiffBase):
378
 
 
379
 
    def test_external_diff(self):
380
 
        """Test that we can spawn an external diff process"""
381
 
        self.disable_missing_extensions_warning()
382
 
        # We have to use run_bzr_subprocess, because we need to
383
 
        # test writing directly to stdout, (there was a bug in
384
 
        # subprocess.py that we had to workaround).
385
 
        # However, if 'diff' may not be available
386
 
        self.make_example_branch()
387
 
        # this will be automatically restored by the base bzr test class
388
 
        os.environ['BZR_PROGRESS_BAR'] = 'none'
389
 
        out, err = self.run_bzr_subprocess('diff -r 1 --diff-options -ub',
390
 
                                           universal_newlines=True,
391
 
                                           retcode=None)
392
 
        if 'Diff is not installed on this machine' in err:
393
 
            raise tests.TestSkipped("No external 'diff' is available")
394
 
        self.assertEqual('', err)
395
 
        # We have to skip the stuff in the middle, because it depends
396
 
        # on time.time()
397
 
        self.assertStartsWith(out, "=== added file 'goodbye'\n"
398
 
                                   "--- goodbye\t1970-01-01 00:00:00 +0000\n"
399
 
                                   "+++ goodbye\t")
400
 
        self.assertEndsWith(out, "\n@@ -0,0 +1 @@\n"
401
 
                                 "+baz\n\n")
402
 
 
403
 
 
404
 
class TestDiffOutput(DiffBase):
405
 
 
406
 
    def test_diff_output(self):
407
 
        # check that output doesn't mangle line-endings
408
 
        self.make_example_branch()
409
 
        self.build_tree_contents([('hello', 'hello world!\n')])
410
 
        output = self.run_bzr_subprocess('diff', retcode=1)[0]
411
 
        self.assert_('\n+hello world!\n' in output)