1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
|
# vim: set fileencoding=utf-8 :
#
# (C) 2017 Guido Günther <agx@sigxcpu.org>
#
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation; either version 2 of the License, or
# (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program; if not, please see
# <http://www.gnu.org/licenses/>
import os
from tests.component import (ComponentTestBase)
from tests.component.deb import DEB_TEST_DATA_DIR
from tests.component.deb.fixtures import RepoFixtures
from gbp.scripts.pq import main as pq
from gbp.scripts.import_dsc import main as import_dsc
class TestPq(ComponentTestBase):
"""Test gbp pq"""
def _test_pq(self, repo, action, opts=[]):
args = ['arg0', action] + opts
os.chdir(os.path.abspath(repo.path))
ret = pq(args)
assert ret == 0, "Running gbp pq %s failed" % action
@RepoFixtures.quilt30()
def test_rebase_import(self, repo):
"""Test that rebase imports patches first"""
assert repo.branch == "master"
assert repo.has_branch("patch-queue/master") is False
self._test_pq(repo, "rebase")
assert repo.has_branch("patch-queue/master") is True
@RepoFixtures.quilt30()
def test_switch_import(self, repo):
"""Test that switch imports patches first"""
assert repo.branch == "master"
assert repo.has_branch("patch-queue/master") is False
self._test_pq(repo, "switch")
assert repo.has_branch("patch-queue/master") is True
@RepoFixtures.quilt30()
def test_empty_cycle(self, repo):
assert repo.has_branch("patch-queue/master") is False
assert repo.branch == "master"
self._test_pq(repo, "import")
assert repo.has_branch("patch-queue/master") is True
assert repo.branch == "patch-queue/master"
self._test_pq(repo, "rebase")
assert repo.branch == "patch-queue/master"
self._test_pq(repo, "export")
assert repo.has_branch("patch-queue/master") is True
assert repo.branch == "master"
self._test_pq(repo, "drop")
assert repo.has_branch("patch-queue/master") is False
@RepoFixtures.quilt30()
def test_rename(self, repo):
patch = os.path.join(repo.path, 'debian/patches/0001-Rename.patch')
# gbp's behavior should override the user's configuration.
repo.set_config('diff.renames', 'false')
self._test_pq(repo, 'import')
repo.rename_file('configure.ac', 'renamed')
repo.commit_all("Rename")
self._test_pq(repo, 'export')
self.assertTrue(
os.path.exists(patch))
# Check the file was renamed, not removed and added
with open(patch) as f:
p = f.read()
self.assertTrue('rename from' in p)
self.assertTrue('rename to' in p)
@RepoFixtures.quilt30()
def test_copy(self, repo):
patch = os.path.join(repo.path, 'debian/patches/0001-Copy.patch')
# gbp's behavior should override the user's configuration.
repo.set_config('diff.renames', 'false')
self._test_pq(repo, 'import')
with open(os.path.join(repo.path, 'configure.ac'), mode='rb') as fin:
with open(os.path.join(repo.path, 'copied'), mode='wb') as fout:
fout.write(fin.read())
repo.add_files(['copied'])
repo.commit_all("Copy")
self._test_pq(repo, 'export')
self.assertTrue(os.path.exists(patch))
with open(patch) as f:
p = f.read()
self.assertTrue('copy from' in p)
self.assertTrue('copy to' in p)
@staticmethod
def _dsc_name(pkg, version, dir):
return os.path.join(DEB_TEST_DATA_DIR,
dir,
'%s_%s.dsc' % (pkg, version))
@staticmethod
def _append_patch(repo, name, contents):
with open(os.path.join(repo.path, 'debian/patches/series'), 'a') as series_file:
series_file.write('{}.patch\n'.format(name))
with open(os.path.join(repo.path, 'debian/patches/{}.patch'.format(name)), 'w') as patch:
patch.write(contents)
repo.add_files('debian/patches/{}.patch'.format(name))
repo.commit_files(msg='Add patch: {}.patch'.format(name),
files=['debian/patches/series',
'debian/patches/{}.patch'.format(name)])
@RepoFixtures.quilt30()
def test_import(self, repo):
pkg = "hello-debhelper"
dsc = self._dsc_name(pkg, "2.6-2", "dsc-3.0")
assert import_dsc(["arg0", dsc]) == 0
self._test_pq(repo, "import")
author, subject = repo.get_head_author_subject()
assert author == "Santiago Vila <sanvila@debian.org>"
assert subject == "Modified doc/Makefile.in to avoid " "/usr/share/info/dir.gz"
self._test_pq(repo, 'switch')
self._append_patch(repo, 'foo', '''\
Author: Mr. T. St <t@example.com>
Description: Short DEP-3 description
Long DEP-3 description
.
Continued
--- /dev/null
+++ b/foo
@@ -0,0 +1 @@
+foo
''')
self._test_pq(repo, 'import', ['--force'])
author, subject = repo.get_head_author_subject()
assert subject == "Short DEP-3 description"
assert author == '"Mr. T. St" <t@example.com>'
@RepoFixtures.quilt30()
def test_import_poor_dep3_behaviour(self, repo):
"""Demonstrate the issues with the current DEP-3 support"""
pkg = "hello-debhelper"
dsc = self._dsc_name(pkg, "2.6-2", "dsc-3.0")
assert import_dsc(["arg0", dsc]) == 0
self._append_patch(repo, 'foo', '''\
Author: Mr. T. St <t@example.com>
Description: A very long description with wrapp-
ing to increase readability in the file, which
is currently split into a short and long description.
Origin: https://twitter.com/MrT/status/941789967361097728
Forwarded: not-needed
--- /dev/null
+++ b/foo
@@ -0,0 +1 @@
+foo
''')
self._test_pq(repo, 'import', ['--force'])
_, subject = repo.get_head_author_subject()
assert subject == "A very long description with wrapp-"
self._test_pq(repo, 'export')
relevant_parts_of_patch = ''
with open('debian/patches/foo.patch') as patch_file:
for line in patch_file:
# skip the date as it's currently set to now(),
# not a deterministic value
if line.startswith('Date: '):
continue
# stop reading after the main part of the description;
# i.e. ignore the bit that git(1) fully controls.
if line.startswith('---'):
break
relevant_parts_of_patch += line
assert (
relevant_parts_of_patch == """\
From: "Mr. T. St" <t@example.com>
Subject: A very long description with wrapp-
Origin: https://twitter.com/MrT/status/941789967361097728
Forwarded: not-needed
ing to increase readability in the file, which
is currently split into a short and long description.
"""
)
|
