Add '--hash' option for 'git-pw patch list'

Allow users to filter by hash.

Signed-off-by: Stephen Finucane <stephen@that.guru>
Closes: #55

Author: stephenfin

git_pw/patch.py

@@ -199,23 +199,25 @@ def update_cmd(patch_ids, commit_ref, state, delegate, archived, fmt):
               help='Show only patches by these submitters. Should be an '
               'email, name or ID.')
 @click.option('--delegate', metavar='DELEGATE', multiple=True,
-              help='Show only patches by these delegates. Should be an '
+              help='Show only patches with these delegates. Should be an '
               'email or username.')
+@click.option('--hash', 'hashes', metavar='HASH', multiple=True,
+              help='Show only patches with these hashes.')
 @click.option('--archived', default=False, is_flag=True,
               help='Include patches that are archived.')
 @utils.pagination_options(sort_fields=_sort_fields, default_sort='-date')
 @utils.format_options(headers=_list_headers)
 @click.argument('name', required=False)
 @api.validate_multiple_filter_support
-def list_cmd(state, submitter, delegate, archived, limit, page, sort, fmt,
-             headers, name):
+def list_cmd(state, submitter, delegate, hashes, archived, limit, page, sort,
+             fmt, headers, name):
     """List patches.
 
     List patches on the Patchwork instance.
     """
     LOG.debug('List patches: states=%s, submitters=%s, delegates=%s, '
-              'archived=%r', ','.join(state), ','.join(submitter),
-              ','.join(delegate), archived)
+              'hashes=%s, archived=%r', ','.join(state), ','.join(submitter),
+              ','.join(delegate), ','.join(hashes), archived)
 
     params = []
 
@@ -236,6 +238,9 @@ def list_cmd(state, submitter, delegate, archived, limit, page, sort, fmt,
         else:
             params.extend(api.retrieve_filter_ids('users', 'delegate', delg))
 
+    for hash_ in hashes:
+        params.append(('hash', hash_))
+
     params.extend([
         ('q', name),
         ('archived', 'true' if archived else 'false'),

releasenotes/notes/issue-55-bfcf05e02ad305b1.yaml

@@ -0,0 +1,6 @@
+---
+features:
+  - |
+    It is now possible to filter patches by hash. For example::
+
+      git pw patch list --hash HASH

tests/test_patch.py

@@ -345,7 +345,7 @@ def test_list_with_filters(self, mock_echo, mock_index, mock_version):
         result = runner.invoke(patch.list_cmd, [
             '--state', 'new', '--submitter', 'john@example.com',
             '--submitter', '2', '--delegate', 'doe@example.com',
-            '--delegate', '2', '--archived',
+            '--delegate', '2', '--hash', 'foo', '--archived',
             '--limit', 1, '--page', 1, '--sort', '-name', 'test'])
 
         assert result.exit_code == 0, result
@@ -354,7 +354,8 @@ def test_list_with_filters(self, mock_echo, mock_index, mock_version):
             mock.call('users', [('q', 'doe@example.com')]),
             mock.call('patches', [
                 ('state', 'new'), ('submitter', 1), ('submitter', '2'),
-                ('delegate', 1), ('delegate', '2'), ('q', 'test'),
+                ('delegate', 1), ('delegate', '2'), ('hash', 'foo'),
+                ('q', 'test'),
                 ('archived', 'true'), ('page', 1), ('per_page', 1),
                 ('order', '-name')])]