/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: Canonical.com Patch Queue Manager
  • Date: 2007-03-09 02:08:38 UTC
  • mfrom: (2327.1.1 jam-integration)
  • Revision ID: pqm@pqm.ubuntu.com-20070309020838-1dd7e652bc5c30ff
(Dmitry Vasiliev) Support for Putty SSH, and improved vendor support

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2006 by Canonical Ltd
2
 
 
 
1
# Copyright (C) 2005, 2006 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
15
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
19
19
"""
20
20
 
21
21
import os
 
22
import re
22
23
 
23
24
import bzrlib
 
25
from bzrlib import workingtree
24
26
from bzrlib.branch import Branch
 
27
from bzrlib.tests import TestSkipped
25
28
from bzrlib.tests.blackbox import ExternalBase
26
29
 
27
30
 
28
 
class TestDiff(ExternalBase):
29
 
 
30
 
    def make_example_branch(test):
 
31
def subst_dates(string):
 
32
    """Replace date strings with constant values."""
 
33
    return re.sub(r'\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} [-\+]\d{4}',
 
34
                  'YYYY-MM-DD HH:MM:SS +ZZZZ', string)
 
35
 
 
36
 
 
37
class DiffBase(ExternalBase):
 
38
    """Base class with common setup method"""
 
39
 
 
40
    def make_example_branch(self):
31
41
        # FIXME: copied from test_too_much -- share elsewhere?
32
 
        test.runbzr('init')
33
 
        file('hello', 'wt').write('foo')
34
 
        test.runbzr('add hello')
35
 
        test.runbzr('commit -m setup hello')
36
 
        file('goodbye', 'wt').write('baz')
37
 
        test.runbzr('add goodbye')
38
 
        test.runbzr('commit -m setup goodbye')
 
42
        tree = self.make_branch_and_tree('.')
 
43
        open('hello', 'wb').write('foo\n')
 
44
        tree.add(['hello'])
 
45
        tree.commit('setup')
 
46
        open('goodbye', 'wb').write('baz\n')
 
47
        tree.add(['goodbye'])
 
48
        tree.commit('setup')
 
49
 
 
50
 
 
51
class TestDiff(DiffBase):
39
52
 
40
53
    def test_diff(self):
41
54
        self.make_example_branch()
50
63
        os.unlink('moo')
51
64
        self.runbzr('diff')
52
65
 
 
66
    def test_diff_prefix(self):
 
67
        """diff --prefix appends to filenames in output"""
 
68
        self.make_example_branch()
 
69
        file('hello', 'wb').write('hello world!\n')
 
70
        out, err = self.runbzr('diff --prefix old/:new/', retcode=1)
 
71
        self.assertEquals(err, '')
 
72
        self.assertEqualDiff(subst_dates(out), '''\
 
73
=== modified file 'hello'
 
74
--- old/hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
 
75
+++ new/hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
 
76
@@ -1,1 +1,1 @@
 
77
-foo
 
78
+hello world!
 
79
 
 
80
''')
 
81
 
 
82
    def test_diff_illegal_prefix_value(self):
 
83
        # There was an error in error reporting for this option
 
84
        out, err = self.runbzr('diff --prefix old/', retcode=3)
 
85
        self.assertContainsRe(err,
 
86
            '--prefix expects two values separated by a colon')
 
87
 
 
88
    def test_diff_p1(self):
 
89
        """diff -p1 produces lkml-style diffs"""
 
90
        self.make_example_branch()
 
91
        file('hello', 'wb').write('hello world!\n')
 
92
        out, err = self.runbzr('diff -p1', retcode=1)
 
93
        self.assertEquals(err, '')
 
94
        self.assertEqualDiff(subst_dates(out), '''\
 
95
=== modified file 'hello'
 
96
--- old/hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
 
97
+++ new/hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
 
98
@@ -1,1 +1,1 @@
 
99
-foo
 
100
+hello world!
 
101
 
 
102
''')
 
103
 
 
104
    def test_diff_p0(self):
 
105
        """diff -p0 produces diffs with no prefix"""
 
106
        self.make_example_branch()
 
107
        file('hello', 'wb').write('hello world!\n')
 
108
        out, err = self.runbzr('diff -p0', retcode=1)
 
109
        self.assertEquals(err, '')
 
110
        self.assertEqualDiff(subst_dates(out), '''\
 
111
=== modified file 'hello'
 
112
--- hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
 
113
+++ hello\tYYYY-MM-DD HH:MM:SS +ZZZZ
 
114
@@ -1,1 +1,1 @@
 
115
-foo
 
116
+hello world!
 
117
 
 
118
''')
 
119
 
53
120
    def test_diff_nonexistent(self):
54
121
        # Get an error from a file that does not exist at all
55
122
        # (Malone #3619)
57
124
        out, err = self.runbzr('diff does-not-exist', retcode=3)
58
125
        self.assertContainsRe(err, 'not versioned.*does-not-exist')
59
126
 
 
127
    def test_diff_illegal_revision_specifiers(self):
 
128
        out, err = self.runbzr('diff -r 1..23..123', retcode=3)
 
129
        self.assertContainsRe(err, 'one or two revision specifiers')
 
130
 
60
131
    def test_diff_unversioned(self):
61
132
        # Get an error when diffing a non-versioned file.
62
133
        # (Malone #3619)
79
150
    def test_diff_branches(self):
80
151
        self.example_branches()
81
152
        # should open branch1 and diff against branch2, 
82
 
        output = self.run_bzr_captured(['diff', '-r', 'branch:branch2', 
83
 
                                        'branch1'],
84
 
                                       retcode=1)
85
 
        self.assertEquals(("=== modified file 'a/file'\n"
86
 
                           "--- a/file\t\n"
87
 
                           "+++ b/file\t\n"
88
 
                           "@@ -1,1 +1,1 @@\n"
89
 
                           "-new content\n"
90
 
                           "+contents of branch1/file\n"
91
 
                           "\n", ''), output)
92
 
        output = self.run_bzr_captured(['diff', 'branch2', 'branch1'],
93
 
                                       retcode=1)
94
 
        self.assertEqualDiff(("=== modified file 'a/file'\n"
95
 
                              "--- a/file\t\n"
96
 
                              "+++ b/file\t\n"
97
 
                              "@@ -1,1 +1,1 @@\n"
98
 
                              "-new content\n"
99
 
                              "+contents of branch1/file\n"
100
 
                              "\n", ''), output)
 
153
        out, err = self.run_bzr_captured(['diff', '-r', 'branch:branch2', 
 
154
                                          'branch1'],
 
155
                                         retcode=1)
 
156
        self.assertEquals('', err)
 
157
        self.assertEquals("=== modified file 'file'\n"
 
158
                          "--- file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
 
159
                          "+++ file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
 
160
                          "@@ -1,1 +1,1 @@\n"
 
161
                          "-new content\n"
 
162
                          "+contents of branch1/file\n"
 
163
                          "\n", subst_dates(out))
 
164
        out, err = self.run_bzr_captured(['diff', 'branch2', 'branch1'],
 
165
                                         retcode=1)
 
166
        self.assertEquals('', err)
 
167
        self.assertEqualDiff("=== modified file 'file'\n"
 
168
                              "--- file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
 
169
                              "+++ file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
 
170
                              "@@ -1,1 +1,1 @@\n"
 
171
                              "-new content\n"
 
172
                              "+contents of branch1/file\n"
 
173
                              "\n", subst_dates(out))
 
174
 
 
175
    def test_diff_revno_branches(self):
 
176
        self.example_branches()
 
177
        print >> open('branch2/file', 'wb'), 'even newer content'
 
178
        self.run_bzr_captured(['commit', '-m', 
 
179
                               'update file once more', 'branch2'])
 
180
 
 
181
        out, err = self.run_bzr_captured(['diff', '-r',
 
182
                                          'revno:1:branch2..revno:1:branch1'],
 
183
                                         retcode=0)
 
184
        self.assertEquals('', err)
 
185
        self.assertEquals('', out)
 
186
        out, err = self.run_bzr_captured(['diff', '-r', 
 
187
                                          'revno:2:branch2..revno:1:branch1'],
 
188
                                         retcode=1)
 
189
        self.assertEquals('', err)
 
190
        self.assertEqualDiff("=== modified file 'file'\n"
 
191
                              "--- file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
 
192
                              "+++ file\tYYYY-MM-DD HH:MM:SS +ZZZZ\n"
 
193
                              "@@ -1,1 +1,1 @@\n"
 
194
                              "-new content\n"
 
195
                              "+contents of branch1/file\n"
 
196
                              "\n", subst_dates(out))
101
197
 
102
198
    def example_branch2(self):
103
199
        self.build_tree(['branch1/', 'branch1/file1'], line_endings='binary')
113
209
        self.example_branch2()
114
210
        
115
211
        print >> open('branch1/file1', 'wb'), 'new line'
116
 
        output = self.run_bzr_captured(['diff', '-r', '1..', 'branch1'], retcode=1)
 
212
        output = self.run_bzr_captured(['diff', '-r', '1..', 'branch1'],
 
213
                                       retcode=1)
117
214
        self.assertTrue('\n-original line\n+new line\n' in output[0])
118
215
 
 
216
    def test_diff_across_rename(self):
 
217
        """The working tree path should always be considered for diffing"""
 
218
        self.make_example_branch()
 
219
        self.run_bzr('diff', '-r', '0..1', 'hello', retcode=1)
 
220
        wt = workingtree.WorkingTree.open_containing('.')[0]
 
221
        wt.rename_one('hello', 'hello1')
 
222
        self.run_bzr('diff', 'hello1', retcode=1)
 
223
        self.run_bzr('diff', '-r', '0..1', 'hello1', retcode=1)
 
224
 
119
225
 
120
226
class TestCheckoutDiff(TestDiff):
121
227
 
138
244
        os.chdir('checkouts')
139
245
 
140
246
 
141
 
class TestDiffLabels(TestDiff):
 
247
class TestDiffLabels(DiffBase):
142
248
 
143
249
    def test_diff_label_removed(self):
144
250
        super(TestDiffLabels, self).make_example_branch()
145
251
        self.runbzr('remove hello')
146
252
        diff = self.run_bzr_captured(['diff'], retcode=1)
147
 
        self.assertTrue("=== removed file 'a/hello'" in diff[0])
 
253
        self.assertTrue("=== removed file 'hello'" in diff[0])
148
254
 
149
255
    def test_diff_label_added(self):
150
256
        super(TestDiffLabels, self).make_example_branch()
151
257
        file('barbar', 'wt').write('barbar')
152
258
        self.runbzr('add barbar')
153
259
        diff = self.run_bzr_captured(['diff'], retcode=1)
154
 
        self.assertTrue("=== added file 'b/barbar'" in diff[0])
 
260
        self.assertTrue("=== added file 'barbar'" in diff[0])
155
261
 
156
262
    def test_diff_label_modified(self):
157
263
        super(TestDiffLabels, self).make_example_branch()
158
264
        file('hello', 'wt').write('barbar')
159
265
        diff = self.run_bzr_captured(['diff'], retcode=1)
160
 
        self.assertTrue("=== modified file 'a/hello'" in diff[0])
 
266
        self.assertTrue("=== modified file 'hello'" in diff[0])
161
267
 
162
268
    def test_diff_label_renamed(self):
163
269
        super(TestDiffLabels, self).make_example_branch()
164
270
        self.runbzr('rename hello gruezi')
165
271
        diff = self.run_bzr_captured(['diff'], retcode=1)
166
 
        self.assertTrue("=== renamed file 'a/hello' => 'b/gruezi'" in diff[0])
 
272
        self.assertTrue("=== renamed file 'hello' => 'gruezi'" in diff[0])
 
273
 
 
274
 
 
275
class TestExternalDiff(DiffBase):
 
276
 
 
277
    def test_external_diff(self):
 
278
        """Test that we can spawn an external diff process"""
 
279
        # We have to use run_bzr_subprocess, because we need to
 
280
        # test writing directly to stdout, (there was a bug in
 
281
        # subprocess.py that we had to workaround).
 
282
        # However, if 'diff' may not be available
 
283
        self.make_example_branch()
 
284
        orig_progress = os.environ.get('BZR_PROGRESS_BAR')
 
285
        try:
 
286
            os.environ['BZR_PROGRESS_BAR'] = 'none'
 
287
            out, err = self.run_bzr_subprocess('diff', '-r', '1',
 
288
                                               '--diff-options', '-ub',
 
289
                                               universal_newlines=True,
 
290
                                               retcode=None)
 
291
        finally:
 
292
            if orig_progress is None:
 
293
                del os.environ['BZR_PROGRESS_BAR']
 
294
            else:
 
295
                os.environ['BZR_PROGRESS_BAR'] = orig_progress
 
296
            
 
297
        if 'Diff is not installed on this machine' in err:
 
298
            raise TestSkipped("No external 'diff' is available")
 
299
        self.assertEqual('', err)
 
300
        # We have to skip the stuff in the middle, because it depends
 
301
        # on time.time()
 
302
        self.assertStartsWith(out, "=== added file 'goodbye'\n"
 
303
                                   "--- goodbye\t1970-01-01 00:00:00 +0000\n"
 
304
                                   "+++ goodbye\t")
 
305
        self.assertEndsWith(out, "\n@@ -0,0 +1 @@\n"
 
306
                                 "+baz\n\n")
 
307
 
 
308
 
 
309
class TestDiffOutput(DiffBase):
 
310
 
 
311
    def test_diff_output(self):
 
312
        # check that output doesn't mangle line-endings
 
313
        self.make_example_branch()
 
314
        file('hello', 'wb').write('hello world!\n')
 
315
        output = self.run_bzr_subprocess('diff', retcode=1)[0]
 
316
        self.assert_('\n+hello world!\n' in output)