Print help if pager is not on PATH

This checks to see if the PAGER is available on POSIX systems
before attempting to run it. If it is not found, then the help
information will just be printed. This adds support for the
rare system that does not have `less` installed.

Fixes aws#1957

awscli/help.py

@@ -114,6 +114,11 @@ def _convert_doc_content(self, contents):
 
     def _send_output_to_pager(self, output):
         cmdline = self.get_pager_cmdline()
+        if not self._exists_on_path(cmdline[0]):
+            LOG.debug("Pager '%s' not found in PATH, printing raw help." %
+                      cmdline[0])
+            print(output.decode('utf-8'))
+            return
         LOG.debug("Running command: %s", cmdline)
         with ignore_ctrl_c():
             # We can't rely on the KeyboardInterrupt from

tests/unit/test_help.py

@@ -19,6 +19,7 @@
 
 import mock
 
+from awscli.compat import six
 from awscli.help import PosixHelpRenderer, ExecutableNotFoundError
 from awscli.help import WindowsHelpRenderer, ProviderHelpCommand, HelpCommand
 from awscli.help import TopicListerCommand, TopicHelpCommand
@@ -96,10 +97,28 @@ def test_pager_with_args(self):
     def test_no_groff_exists(self):
         renderer = FakePosixHelpRenderer()
         renderer.exists_on_path['groff'] = False
-        with self.assertRaisesRegexp(ExecutableNotFoundError,
-                                     'Could not find executable named "groff"'):
+        expected_error = 'Could not find executable named "groff"'
+        with self.assertRaisesRegexp(ExecutableNotFoundError, expected_error):
             renderer.render('foo')
 
+    @skip_if_windows('Requires POSIX system.')
+    def test_no_pager_exists(self):
+        fake_pager = 'foobar'
+        os.environ['MANPAGER'] = fake_pager
+        renderer = FakePosixHelpRenderer()
+        renderer.exists_on_path[fake_pager] = False
+
+        renderer.exists_on_path['groff'] = True
+        renderer.mock_popen.communicate.return_value = (b'foo', '')
+
+        stdout = six.StringIO()
+        captured_stdout = sys.stdout
+        sys.stdout = stdout
+
+        renderer.render('foo')
+        self.assertEqual(stdout.getvalue(), 'foo\n')
+        sys.stdout = captured_stdout
+
     def test_shlex_split_for_pager_var(self):
         pager_cmd = '/bin/sh -c "col -bx | vim -c \'set ft=man\' -"'
         os.environ['PAGER'] = pager_cmd
@@ -109,6 +128,7 @@ def test_shlex_split_for_pager_var(self):
     def test_can_render_contents(self):
         renderer = FakePosixHelpRenderer()
         renderer.exists_on_path['groff'] = True
+        renderer.exists_on_path['less'] = True
         renderer.mock_popen.communicate.return_value = ('rendered', '')
         renderer.render('foo')
         self.assertEqual(renderer.popen_calls[-1][0], (['less', '-R'],))
@@ -133,6 +153,7 @@ def _is_pager_call(self, args):
         renderer = CtrlCRenderer()
         renderer.mock_popen.communicate.return_value = ('send to pager', '')
         renderer.exists_on_path['groff'] = True
+        renderer.exists_on_path['less'] = True
         renderer.render('foo')
         last_call = renderer.mock_popen.communicate.call_args_list[-1]
         self.assertEqual(last_call, mock.call(input='send to pager'))