Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 1 | # -*- coding: utf-8 -*- |
Tom Rini | 83d290c | 2018-05-06 17:58:06 -0400 | [diff] [blame] | 2 | # SPDX-License-Identifier: GPL-2.0+ |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 3 | # |
| 4 | # Copyright 2017 Google, Inc |
| 5 | # |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 6 | |
| 7 | import contextlib |
| 8 | import os |
| 9 | import re |
| 10 | import shutil |
| 11 | import sys |
| 12 | import tempfile |
| 13 | import unittest |
| 14 | |
Simon Glass | ade1e38 | 2019-05-14 15:53:49 -0600 | [diff] [blame] | 15 | try: |
| 16 | from StringIO import StringIO |
| 17 | except ImportError: |
| 18 | from io import StringIO |
| 19 | |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 20 | import gitutil |
| 21 | import patchstream |
| 22 | import settings |
Simon Glass | ade1e38 | 2019-05-14 15:53:49 -0600 | [diff] [blame] | 23 | import tools |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 24 | |
| 25 | |
| 26 | @contextlib.contextmanager |
| 27 | def capture(): |
| 28 | import sys |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 29 | oldout,olderr = sys.stdout, sys.stderr |
| 30 | try: |
| 31 | out=[StringIO(), StringIO()] |
| 32 | sys.stdout,sys.stderr = out |
| 33 | yield out |
| 34 | finally: |
| 35 | sys.stdout,sys.stderr = oldout, olderr |
| 36 | out[0] = out[0].getvalue() |
| 37 | out[1] = out[1].getvalue() |
| 38 | |
| 39 | |
| 40 | class TestFunctional(unittest.TestCase): |
| 41 | def setUp(self): |
| 42 | self.tmpdir = tempfile.mkdtemp(prefix='patman.') |
| 43 | |
| 44 | def tearDown(self): |
| 45 | shutil.rmtree(self.tmpdir) |
| 46 | |
| 47 | @staticmethod |
| 48 | def GetPath(fname): |
| 49 | return os.path.join(os.path.dirname(os.path.realpath(sys.argv[0])), |
| 50 | 'test', fname) |
| 51 | |
| 52 | @classmethod |
| 53 | def GetText(self, fname): |
| 54 | return open(self.GetPath(fname)).read() |
| 55 | |
| 56 | @classmethod |
| 57 | def GetPatchName(self, subject): |
| 58 | fname = re.sub('[ :]', '-', subject) |
| 59 | return fname.replace('--', '-') |
| 60 | |
| 61 | def CreatePatchesForTest(self, series): |
| 62 | cover_fname = None |
| 63 | fname_list = [] |
| 64 | for i, commit in enumerate(series.commits): |
| 65 | clean_subject = self.GetPatchName(commit.subject) |
| 66 | src_fname = '%04d-%s.patch' % (i + 1, clean_subject[:52]) |
| 67 | fname = os.path.join(self.tmpdir, src_fname) |
| 68 | shutil.copy(self.GetPath(src_fname), fname) |
| 69 | fname_list.append(fname) |
| 70 | if series.get('cover'): |
| 71 | src_fname = '0000-cover-letter.patch' |
| 72 | cover_fname = os.path.join(self.tmpdir, src_fname) |
| 73 | fname = os.path.join(self.tmpdir, src_fname) |
| 74 | shutil.copy(self.GetPath(src_fname), fname) |
| 75 | |
| 76 | return cover_fname, fname_list |
| 77 | |
| 78 | def testBasic(self): |
| 79 | """Tests the basic flow of patman |
| 80 | |
| 81 | This creates a series from some hard-coded patches build from a simple |
| 82 | tree with the following metadata in the top commit: |
| 83 | |
| 84 | Series-to: u-boot |
| 85 | Series-prefix: RFC |
| 86 | Series-cc: Stefan Brüns <stefan.bruens@rwth-aachen.de> |
| 87 | Cover-letter-cc: Lord Mëlchett <clergy@palace.gov> |
| 88 | Series-version: 2 |
| 89 | Series-changes: 4 |
| 90 | - Some changes |
| 91 | |
| 92 | Cover-letter: |
| 93 | test: A test patch series |
| 94 | This is a test of how the cover |
| 95 | leter |
| 96 | works |
| 97 | END |
| 98 | |
| 99 | and this in the first commit: |
| 100 | |
| 101 | Series-notes: |
| 102 | some notes |
| 103 | about some things |
| 104 | from the first commit |
| 105 | END |
| 106 | |
| 107 | Commit-notes: |
| 108 | Some notes about |
| 109 | the first commit |
| 110 | END |
| 111 | |
| 112 | with the following commands: |
| 113 | |
| 114 | git log -n2 --reverse >/path/to/tools/patman/test/test01.txt |
| 115 | git format-patch --subject-prefix RFC --cover-letter HEAD~2 |
| 116 | mv 00* /path/to/tools/patman/test |
| 117 | |
| 118 | It checks these aspects: |
| 119 | - git log can be processed by patchstream |
| 120 | - emailing patches uses the correct command |
| 121 | - CC file has information on each commit |
| 122 | - cover letter has the expected text and subject |
| 123 | - each patch has the correct subject |
| 124 | - dry-run information prints out correctly |
| 125 | - unicode is handled correctly |
| 126 | - Series-to, Series-cc, Series-prefix, Cover-letter |
| 127 | - Cover-letter-cc, Series-version, Series-changes, Series-notes |
| 128 | - Commit-notes |
| 129 | """ |
| 130 | process_tags = True |
| 131 | ignore_bad_tags = True |
| 132 | stefan = u'Stefan Brüns <stefan.bruens@rwth-aachen.de>' |
| 133 | rick = 'Richard III <richard@palace.gov>' |
| 134 | mel = u'Lord Mëlchett <clergy@palace.gov>' |
| 135 | ed = u'Lond Edmund Blackaddër <weasel@blackadder.org' |
| 136 | fred = 'Fred Bloggs <f.bloggs@napier.net>' |
| 137 | add_maintainers = [stefan, rick] |
| 138 | dry_run = True |
| 139 | in_reply_to = mel |
| 140 | count = 2 |
| 141 | settings.alias = { |
| 142 | 'fdt': ['simon'], |
| 143 | 'u-boot': ['u-boot@lists.denx.de'], |
| 144 | 'simon': [ed], |
| 145 | 'fred': [fred], |
| 146 | } |
| 147 | |
| 148 | text = self.GetText('test01.txt') |
| 149 | series = patchstream.GetMetaDataForTest(text) |
| 150 | cover_fname, args = self.CreatePatchesForTest(series) |
| 151 | with capture() as out: |
| 152 | patchstream.FixPatches(series, args) |
| 153 | if cover_fname and series.get('cover'): |
| 154 | patchstream.InsertCoverLetter(cover_fname, series, count) |
| 155 | series.DoChecks() |
| 156 | cc_file = series.MakeCcFile(process_tags, cover_fname, |
Chris Packham | 4fb3502 | 2018-06-07 20:45:06 +1200 | [diff] [blame] | 157 | not ignore_bad_tags, add_maintainers, |
| 158 | None) |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 159 | cmd = gitutil.EmailPatches(series, cover_fname, args, |
| 160 | dry_run, not ignore_bad_tags, cc_file, |
| 161 | in_reply_to=in_reply_to, thread=None) |
| 162 | series.ShowActions(args, cmd, process_tags) |
| 163 | cc_lines = open(cc_file).read().splitlines() |
| 164 | os.remove(cc_file) |
| 165 | |
| 166 | lines = out[0].splitlines() |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 167 | self.assertEqual('Cleaned %s patches' % len(series.commits), lines[0]) |
| 168 | self.assertEqual('Change log missing for v2', lines[1]) |
| 169 | self.assertEqual('Change log missing for v3', lines[2]) |
| 170 | self.assertEqual('Change log for unknown version v4', lines[3]) |
| 171 | self.assertEqual("Alias 'pci' not found", lines[4]) |
| 172 | self.assertIn('Dry run', lines[5]) |
| 173 | self.assertIn('Send a total of %d patches' % count, lines[7]) |
| 174 | line = 8 |
| 175 | for i, commit in enumerate(series.commits): |
| 176 | self.assertEqual(' %s' % args[i], lines[line + 0]) |
| 177 | line += 1 |
| 178 | while 'Cc:' in lines[line]: |
| 179 | line += 1 |
| 180 | self.assertEqual('To: u-boot@lists.denx.de', lines[line]) |
| 181 | self.assertEqual('Cc: %s' % stefan.encode('utf-8'), lines[line + 1]) |
| 182 | self.assertEqual('Version: 3', lines[line + 2]) |
| 183 | self.assertEqual('Prefix:\t RFC', lines[line + 3]) |
| 184 | self.assertEqual('Cover: 4 lines', lines[line + 4]) |
| 185 | line += 5 |
Simon Glass | b644c66 | 2019-05-14 15:53:51 -0600 | [diff] [blame^] | 186 | self.assertEqual(' Cc: %s' % fred, lines[line + 0]) |
| 187 | self.assertEqual(' Cc: %s' % ed.encode('utf-8'), lines[line + 1]) |
| 188 | self.assertEqual(' Cc: %s' % mel.encode('utf-8'), lines[line + 2]) |
| 189 | self.assertEqual(' Cc: %s' % rick, lines[line + 3]) |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 190 | expected = ('Git command: git send-email --annotate ' |
| 191 | '--in-reply-to="%s" --to "u-boot@lists.denx.de" ' |
| 192 | '--cc "%s" --cc-cmd "%s --cc-cmd %s" %s %s' |
| 193 | % (in_reply_to, stefan, sys.argv[0], cc_file, cover_fname, |
| 194 | ' '.join(args))).encode('utf-8') |
| 195 | line += 4 |
| 196 | self.assertEqual(expected, lines[line]) |
| 197 | |
| 198 | self.assertEqual(('%s %s, %s' % (args[0], rick, stefan)) |
| 199 | .encode('utf-8'), cc_lines[0]) |
Simon Glass | b644c66 | 2019-05-14 15:53:51 -0600 | [diff] [blame^] | 200 | self.assertEqual(('%s %s, %s, %s, %s' % (args[1], fred, ed, rick, |
| 201 | stefan)).encode('utf-8'), cc_lines[1]) |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 202 | |
| 203 | expected = ''' |
| 204 | This is a test of how the cover |
| 205 | leter |
| 206 | works |
| 207 | |
| 208 | some notes |
| 209 | about some things |
| 210 | from the first commit |
| 211 | |
| 212 | Changes in v4: |
| 213 | - Some changes |
| 214 | |
| 215 | Simon Glass (2): |
| 216 | pci: Correct cast for sandbox |
Siva Durga Prasad Paladugu | 12308b1 | 2018-07-16 15:56:11 +0530 | [diff] [blame] | 217 | fdt: Correct cast for sandbox in fdtdec_setup_mem_size_base() |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 218 | |
| 219 | cmd/pci.c | 3 ++- |
| 220 | fs/fat/fat.c | 1 + |
| 221 | lib/efi_loader/efi_memory.c | 1 + |
| 222 | lib/fdtdec.c | 3 ++- |
| 223 | 4 files changed, 6 insertions(+), 2 deletions(-) |
| 224 | |
| 225 | --\x20 |
| 226 | 2.7.4 |
| 227 | |
| 228 | ''' |
| 229 | lines = open(cover_fname).read().splitlines() |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 230 | self.assertEqual( |
| 231 | 'Subject: [RFC PATCH v3 0/2] test: A test patch series', |
| 232 | lines[3]) |
| 233 | self.assertEqual(expected.splitlines(), lines[7:]) |
| 234 | |
| 235 | for i, fname in enumerate(args): |
| 236 | lines = open(fname).read().splitlines() |
Simon Glass | 6e87ae1 | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 237 | subject = [line for line in lines if line.startswith('Subject')] |
| 238 | self.assertEqual('Subject: [RFC %d/%d]' % (i + 1, count), |
| 239 | subject[0][:18]) |
| 240 | if i == 0: |
| 241 | # Check that we got our commit notes |
| 242 | self.assertEqual('---', lines[17]) |
| 243 | self.assertEqual('Some notes about', lines[18]) |
| 244 | self.assertEqual('the first commit', lines[19]) |