diff options
author | Guido Günther <agx@sigxcpu.org> | 2015-06-01 22:50:50 +0200 |
---|---|---|
committer | Guido Günther <agx@sigxcpu.org> | 2015-06-02 19:58:11 +0200 |
commit | 8241459f773607a7c967eb46ca23a1ae3eae31ae (patch) | |
tree | dbb0429ee3d5fa8e10d6539cadc23d10f0324486 /tests/13_test_gbp_pq.py | |
parent | 58ea0677d6ef0ea0dfc5bc93c0517967db49c656 (diff) |
pq: Allow to preserve the patch name on import/export
Closes: #761161
Diffstat (limited to 'tests/13_test_gbp_pq.py')
-rw-r--r-- | tests/13_test_gbp_pq.py | 35 |
1 files changed, 23 insertions, 12 deletions
diff --git a/tests/13_test_gbp_pq.py b/tests/13_test_gbp_pq.py index f7a9bc77..4e729cae 100644 --- a/tests/13_test_gbp_pq.py +++ b/tests/13_test_gbp_pq.py @@ -19,7 +19,6 @@ from . import context from . import testutils import os -import logging import unittest from gbp.scripts.pq import generate_patches, export_patches @@ -49,6 +48,14 @@ class TestApplyAndCommit(testutils.DebianGitTestRepo): info = self.repo.get_commit_info('HEAD') self.assertIn('Gbp-Pq: Topic foobar', info['body']) + def test_name(self): + """Test if setting a name works""" + patch = gbp.patch_series.Patch(_patch_path('foo.patch')) + + pq.apply_and_commit_patch(self.repo, patch, None, name='foobar') + info = self.repo.get_commit_info('HEAD') + self.assertIn('Gbp-Pq: Name foobar', info['body']) + @unittest.skipIf(not os.path.exists('/usr/bin/dpkg'), 'Dpkg not found') def test_debian_missing_author(self): """ @@ -112,26 +119,30 @@ class TestWritePatch(testutils.DebianGitTestRepo): class opts: patch_numbers = False + d = context.new_tmpdir(__name__) + expected_paths = [os.path.join(str(d), 'gbptest', n) for n in + ['added-foo.patch', 'patchname.diff']] # Add test file with topic: msg = ("added foo\n\n" "Gbp-Pq: Topic gbptest") self.add_file('foo', 'foo', msg) + msg = ("added bar\n\n" + "Gbp-Pq: Topic gbptest\n" + "Gbp-Pq: Name patchname.diff") + self.add_file('baz', 'baz', msg) # Write it out as patch and check it's existence - d = context.new_tmpdir(__name__) - patchfile = generate_patches(self.repo, 'HEAD^', 'HEAD', str(d), - opts)[0] - expected = os.path.join(str(d), 'gbptest', 'added-foo.patch') - - self.assertEqual(os.path.basename(patchfile), - 'added-foo.patch') - self.assertTrue(os.path.exists(expected)) - logging.debug(open(expected).read()) + patchfiles = generate_patches(self.repo, 'HEAD^^', 'HEAD', str(d), + opts) + for expected in expected_paths: + self.assertIn(expected, patchfiles) + self.assertTrue(os.path.exists(expected)) # Reapply the patch to a new branch - self.repo.create_branch('testapply', 'HEAD^') + self.repo.create_branch('testapply', 'HEAD^^') self.repo.set_branch('testapply') - self.repo.apply_patch(expected) + for expected in expected_paths: + self.repo.apply_patch(expected) self.repo.commit_all("foo") diff = self.repo.diff('master', 'testapply') # Branches must be identical afterwards |