← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

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

  • Committer: Martin Pool
  • Date: 2007-04-04 06:17:31 UTC
  • mto: This revision was merged to the branch mainline in revision 2397.
  • Revision ID: mbp@sourcefrog.net-20070404061731-tt2xrzllqhbodn83
Contents of TODO file moved into bug tracker

Show diffs side-by-side

added added

removed removed

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