Add new 'gerrit project commit file-content show' command and API

This patch adds new 'gerrit project commit file-content show' command
and respective API that allows to gets the content of a file from
a certain commit:

usage: gerrit project commit file-content show [-h] --commit COMMIT --file-id
                                               FILE_ID
                                               name

Gets the content of a file from a certain commit.

positional arguments:
  name               Name of the project.

optional arguments:
  -h, --help         show this help message and exit
  --commit COMMIT    Commit ID.
  --file-id FILE_ID  The path to the file.

e.g.:

  gerrit project commit file-content show foo/bar \
         --commit 6c94b7dfb4e32af1d99ff8f41f3f2c86bbc0e3ad \
         --file-id some/path/to/file.vhd > /tmp/file.vhd

Author: tivaliy

ChangeLog

@@ -1,6 +1,7 @@
 CHANGES
 =======
 
+* Add new 'gerrit project commit file-content show' command and API
 * Add new 'gerrit project commit included-in' command and API
 * Add new 'gerrit project commit show' command and API
 * Add new 'gerrit project tag create' command and API

gerritclient/commands/project.py

@@ -837,6 +837,35 @@ def take_action(self, parsed_args):
         return self.columns, data
 
 
+class ProjectCommitFileContentShow(ProjectMixIn, base.BaseCommand):
+    """Gets the content of a file from a certain commit."""
+
+    def get_parser(self, prog_name):
+        parser = super(ProjectCommitFileContentShow, self).get_parser(
+            prog_name)
+        parser.add_argument(
+            'name',
+            help='Name of the project.'
+        )
+        parser.add_argument(
+            '--commit',
+            required=True,
+            help='Commit ID.'
+        )
+        parser.add_argument(
+            '--file-id',
+            required=True,
+            help='The path to the file.'
+        )
+        return parser
+
+    def take_action(self, parsed_args):
+        response = self.client.get_file_content(parsed_args.name,
+                                                parsed_args.commit,
+                                                parsed_args.file_id)
+        self.app.stdout.write(response)
+
+
 def debug(argv=None):
     """Helper to debug the required command."""
 

gerritclient/tests/unit/cli/test_project.py

@@ -731,3 +731,25 @@ def test_project_commit_affiliation_show_fail(self, mocked_stderr):
         self.assertRaises(SystemExit, self.exec_command, args)
         self.assertIn('project commit included-in: error:',
                       mocked_stderr.write.call_args_list[-1][0][0])
+
+    def test_project_commit_file_content_show(self):
+        commit_id = "184ebe53805e102605d11f6b143486d15c23a09c"
+        project_name = 'fake/fake-project'
+        file_id = 'foo/bar/file.vhd'
+        fake_file_content = 'Some fake file content'
+        args = ('project commit file-content show {0} '
+                '--commit {1} --file-id {2}'.format(project_name,
+                                                    commit_id, file_id))
+        self.m_client.get_file_content.return_value = fake_file_content
+        self.exec_command(args)
+
+        self.m_get_client.assert_called_once_with('project', mock.ANY)
+        self.m_client.get_file_content.assert_called_once_with(
+            project_name, commit_id, file_id)
+
+    @mock.patch('sys.stderr')
+    def test_project_commit_file_content_show_fail(self, mocked_stderr):
+        args = 'project commit file-content show'
+        self.assertRaises(SystemExit, self.exec_command, args)
+        self.assertIn('project commit file-content show: error:',
+                      mocked_stderr.write.call_args_list[-1][0][0])

gerritclient/v1/project.py

@@ -315,6 +315,16 @@ def get_commit_affiliation(self, name, commit):
             commit=commit)
         return self.connection.get_request(request_path)
 
+    def get_file_content(self, name, commit, file_id):
+        request_path = (
+            "{api_path}{name}/commits/{commit}/files/{file_id}/content".format(
+                api_path=self.api_path,
+                name=requests_utils.quote(name, safe=''),
+                commit=commit,
+                file_id=requests_utils.quote(file_id, safe=''))
+        )
+        return self.connection.get_request(request_path)
+
 
 def get_client(connection):
     return ProjectClient(connection)

setup.cfg

@@ -96,6 +96,7 @@ gerritclient =
     project_branch_reflog_show=gerritclient.commands.project:ProjectBranchReflogShow
     project_branch_show=gerritclient.commands.project:ProjectBranchShow
     project_child_list=gerritclient.commands.project:ProjectChildList
+    project_commit_file-content_show=gerritclient.commands.project:ProjectCommitFileContentShow
     project_commit_included-in=gerritclient.commands.project:ProjectCommitIncludedIn
     project_commit_show=gerritclient.commands.project:ProjectCommitShow
     project_configuration_download=gerritclient.commands.project:ProjectConfigDownload