/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 breezy/tests/test_revert.py

  • Committer: Breezy landing bot
  • Author(s): Jelmer Vernooij
  • Date: 2018-11-16 07:18:33 UTC
  • mfrom: (7141.3.3 fix1799482)
  • Revision ID: breezy.the.bot@gmail.com-20181116071833-e01b0833f3hkc3et
Report correct paths when running "brz add" in git repositories.

Merged from https://code.launchpad.net/~jelmer/brz/fix1799482/+merge/357734

Show diffs side-by-side

added added

removed removed

Lines of Context:
breezy/tests/test_revert.py
16
16
 
17
17
import os
18
18
 
19
 
from bzrlib import merge, tests, transform, workingtree
 
19
from breezy import merge, tests, transform, workingtree
20
20
 
21
21
 
22
22
class TestRevert(tests.TestCaseWithTransport):
26
26
        """Reverting a merge that adds a directory deletes the directory"""
27
27
        source_tree = self.make_branch_and_tree('source')
28
28
        source_tree.commit('empty tree')
29
 
        target_tree = source_tree.bzrdir.sprout('target').open_workingtree()
 
29
        target_tree = source_tree.controldir.sprout('target').open_workingtree()
30
30
        self.build_tree(['source/dir/', 'source/dir/contents'])
31
 
        source_tree.add(['dir', 'dir/contents'], ['dir-id', 'contents-id'])
 
31
        source_tree.add(['dir', 'dir/contents'], [b'dir-id', b'contents-id'])
32
32
        source_tree.commit('added dir')
33
33
        target_tree.lock_write()
34
34
        self.addCleanup(target_tree.unlock)
35
35
        merge.merge_inner(target_tree.branch, source_tree.basis_tree(),
36
36
                          target_tree.basis_tree(), this_tree=target_tree)
37
 
        self.failUnlessExists('target/dir')
38
 
        self.failUnlessExists('target/dir/contents')
 
37
        self.assertPathExists('target/dir')
 
38
        self.assertPathExists('target/dir/contents')
39
39
        target_tree.revert()
40
 
        self.failIfExists('target/dir/contents')
41
 
        self.failIfExists('target/dir')
 
40
        self.assertPathDoesNotExist('target/dir/contents')
 
41
        self.assertPathDoesNotExist('target/dir')
42
42
 
43
43
    def test_revert_new(self):
44
44
        """Only locally-changed new files should be preserved when reverting
53
53
        """
54
54
        tree = self.make_branch_and_tree('tree')
55
55
        tree.commit('empty tree')
56
 
        merge_target = tree.bzrdir.sprout('merge_target').open_workingtree()
 
56
        merge_target = tree.controldir.sprout('merge_target').open_workingtree()
57
57
        self.build_tree(['tree/new_file'])
58
58
 
59
59
        # newly-added files should not be deleted
60
60
        tree.add('new_file')
61
61
        basis_tree = tree.branch.repository.revision_tree(tree.last_revision())
62
62
        tree.revert()
63
 
        self.failUnlessExists('tree/new_file')
 
63
        self.assertPathExists('tree/new_file')
64
64
 
65
65
        # unchanged files should be deleted
66
66
        tree.add('new_file')
67
67
        tree.commit('add new_file')
68
68
        tree.revert(old_tree=basis_tree)
69
 
        self.failIfExists('tree/new_file')
 
69
        self.assertPathDoesNotExist('tree/new_file')
70
70
 
71
71
        # files should be deleted if their changes came from merges
72
72
        merge_target.merge_from_branch(tree.branch)
73
 
        self.failUnlessExists('merge_target/new_file')
 
73
        self.assertPathExists('merge_target/new_file')
74
74
        merge_target.revert()
75
 
        self.failIfExists('merge_target/new_file')
 
75
        self.assertPathDoesNotExist('merge_target/new_file')
76
76
 
77
77
        # files should not be deleted if changed after a merge
78
78
        merge_target.merge_from_branch(tree.branch)
79
 
        self.failUnlessExists('merge_target/new_file')
80
 
        self.build_tree_contents([('merge_target/new_file', 'new_contents')])
 
79
        self.assertPathExists('merge_target/new_file')
 
80
        self.build_tree_contents([('merge_target/new_file', b'new_contents')])
81
81
        merge_target.revert()
82
 
        self.failUnlessExists('merge_target/new_file')
 
82
        self.assertPathExists('merge_target/new_file')
83
83
 
84
84
    def tree_with_executable(self):
85
85
        tree = self.make_branch_and_tree('tree')
86
86
        tt = transform.TreeTransform(tree)
87
 
        tt.new_file('newfile', tt.root, 'helooo!', 'newfile-id', True)
 
87
        tt.new_file('newfile', tt.root, [b'helooo!'], b'newfile-id', True)
88
88
        tt.apply()
89
 
        tree.lock_write()
90
 
        try:
91
 
            self.assertTrue(tree.is_executable('newfile-id'))
 
89
        with tree.lock_write():
 
90
            self.assertTrue(tree.is_executable('newfile'))
92
91
            tree.commit('added newfile')
93
 
        finally:
94
 
            tree.unlock()
95
92
        return tree
96
93
 
97
94
    def test_preserve_execute(self):
98
95
        tree = self.tree_with_executable()
99
96
        tt = transform.TreeTransform(tree)
100
 
        newfile = tt.trans_id_tree_file_id('newfile-id')
 
97
        newfile = tt.trans_id_tree_path('newfile')
101
98
        tt.delete_contents(newfile)
102
 
        tt.create_file('Woooorld!', newfile)
 
99
        tt.create_file([b'Woooorld!'], newfile)
103
100
        tt.apply()
104
101
        tree = workingtree.WorkingTree.open('tree')
105
102
        tree.lock_write()
106
103
        self.addCleanup(tree.unlock)
107
 
        self.assertTrue(tree.is_executable('newfile-id'))
 
104
        self.assertTrue(tree.is_executable('newfile'))
108
105
        transform.revert(tree, tree.basis_tree(), None, backups=True)
109
 
        self.assertEqual('helooo!', tree.get_file('newfile-id').read())
110
 
        self.assertTrue(tree.is_executable('newfile-id'))
 
106
        with tree.get_file('newfile', 'rb') as f:
 
107
            self.assertEqual(b'helooo!', f.read())
 
108
        self.assertTrue(tree.is_executable('newfile'))
111
109
 
112
110
    def test_revert_executable(self):
113
111
        tree = self.tree_with_executable()
114
112
        tt = transform.TreeTransform(tree)
115
 
        newfile = tt.trans_id_tree_file_id('newfile-id')
 
113
        newfile = tt.trans_id_tree_path('newfile')
116
114
        tt.set_executability(False, newfile)
117
115
        tt.apply()
118
116
        tree.lock_write()
119
117
        self.addCleanup(tree.unlock)
120
118
        transform.revert(tree, tree.basis_tree(), None)
121
 
        self.assertTrue(tree.is_executable('newfile-id'))
 
119
        self.assertTrue(tree.is_executable('newfile'))
122
120
 
123
121
    def test_revert_deletes_files_from_revert(self):
124
122
        tree = self.make_branch_and_tree('.')
125
123
        self.build_tree(['file'])
126
124
        tree.add('file')
127
 
        tree.commit('added file', rev_id='rev1')
 
125
        tree.commit('added file', rev_id=b'rev1')
128
126
        os.unlink('file')
129
127
        tree.commit('removed file')
130
 
        self.failIfExists('file')
131
 
        tree.revert(old_tree=tree.branch.repository.revision_tree('rev1'))
132
 
        self.failUnlessExists('file')
 
128
        self.assertPathDoesNotExist('file')
 
129
        tree.revert(old_tree=tree.branch.repository.revision_tree(b'rev1'))
 
130
        self.assertPathExists('file')
133
131
        tree.revert()
134
 
        self.failIfExists('file')
 
132
        self.assertPathDoesNotExist('file')
135
133
        self.assertEqual({}, tree.merge_modified())
136
134
 
137
 
    def test_empty_deprecated(self):
138
 
        tree = self.make_branch_and_tree('.')
139
 
        self.build_tree(['file'])
140
 
        tree.add('file')
141
 
        self.callDeprecated(['Using [] to revert all files is deprecated'
142
 
            ' as of bzr 0.91.  Please use None (the default) instead.'],
143
 
            tree.revert, [])
144
 
        self.assertIs(None, tree.path2id('file'))
145
 
 
146
135
    def test_revert_file_in_deleted_dir(self):
147
136
        tree = self.make_branch_and_tree('.')
148
137
        self.build_tree(['dir/', 'dir/file1', 'dir/file2'])
149
138
        tree.add(['dir', 'dir/file1', 'dir/file2'],
150
 
                 ['dir-id', 'file1-id', 'file2-id'])
 
139
                 [b'dir-id', b'file1-id', b'file2-id'])
151
140
        tree.commit("Added files")
152
141
        os.unlink('dir/file1')
153
142
        os.unlink('dir/file2')
154
143
        os.rmdir('dir')
155
144
        tree.remove(['dir/', 'dir/file1', 'dir/file2'])
156
145
        tree.revert(['dir/file1'])
157
 
        self.failUnlessExists('dir/file1')
158
 
        self.failIfExists('dir/file2')
159
 
        self.assertEqual('dir-id', tree.path2id('dir'))
 
146
        self.assertPathExists('dir/file1')
 
147
        self.assertPathDoesNotExist('dir/file2')
 
148
        self.assertEqual(b'dir-id', tree.path2id('dir'))
160
149
 
161
150
    def test_revert_root_id_change(self):
162
151
        tree = self.make_branch_and_tree('.')
163
 
        tree.set_root_id('initial-root-id')
 
152
        tree.set_root_id(b'initial-root-id')
164
153
        self.build_tree(['file1'])
165
154
        tree.add(['file1'])
166
155
        tree.commit('first')
167
 
        tree.set_root_id('temp-root-id')
168
 
        self.assertEqual('temp-root-id', tree.get_root_id())
 
156
        tree.set_root_id(b'temp-root-id')
 
157
        self.assertEqual(b'temp-root-id', tree.get_root_id())
169
158
        tree.revert()
170
 
        self.assertEqual('initial-root-id', tree.get_root_id())
 
159
        self.assertEqual(b'initial-root-id', tree.get_root_id())