~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/blackbox/test_diff.py

  • Committer: Robert Collins
  • Date: 2006-07-20 13:00:31 UTC
  • mto: (1852.9.1 Tree.compare().)
  • mto: This revision was merged to the branch mainline in revision 1890.
  • Revision ID: robertc@robertcollins.net-20060720130031-d26103a427ea10f3
StartĀ treeĀ implementationĀ tests.

Show diffs side-by-side

added added

removed removed

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