mirror of
				https://github.com/python/cpython.git
				synced 2025-10-31 13:41:24 +00:00 
			
		
		
		
	bpo-44967: pydoc: return non-zero exit code when query is not found (GH-27868)
This commit is contained in:
		
							parent
							
								
									21fa854792
								
							
						
					
					
						commit
						8868d48712
					
				
					 3 changed files with 25 additions and 21 deletions
				
			
		|  | @ -24,7 +24,7 @@ | |||
| from io import StringIO | ||||
| from collections import namedtuple | ||||
| from test.support import os_helper | ||||
| from test.support.script_helper import assert_python_ok | ||||
| from test.support.script_helper import assert_python_ok, assert_python_failure | ||||
| from test.support import threading_helper | ||||
| from test.support import (reap_children, captured_output, captured_stdout, | ||||
|                           captured_stderr, requires_docstrings) | ||||
|  | @ -345,6 +345,14 @@ def run_pydoc(module_name, *args, **env): | |||
|     rc, out, err = assert_python_ok('-B', pydoc.__file__, *args, **env) | ||||
|     return out.strip() | ||||
| 
 | ||||
| def run_pydoc_fail(module_name, *args, **env): | ||||
|     """ | ||||
|     Runs pydoc on the specified module expecting a failure. | ||||
|     """ | ||||
|     args = args + (module_name,) | ||||
|     rc, out, err = assert_python_failure('-B', pydoc.__file__, *args, **env) | ||||
|     return out.strip() | ||||
| 
 | ||||
| def get_pydoc_html(module): | ||||
|     "Returns pydoc generated output as html" | ||||
|     doc = pydoc.HTMLDoc() | ||||
|  | @ -487,7 +495,7 @@ class B: | |||
| 
 | ||||
|     def test_not_here(self): | ||||
|         missing_module = "test.i_am_not_here" | ||||
|         result = str(run_pydoc(missing_module), 'ascii') | ||||
|         result = str(run_pydoc_fail(missing_module), 'ascii') | ||||
|         expected = missing_pattern % missing_module | ||||
|         self.assertEqual(expected, result, | ||||
|             "documentation for missing module found") | ||||
|  | @ -501,7 +509,7 @@ def test_not_ascii(self): | |||
| 
 | ||||
|     def test_input_strip(self): | ||||
|         missing_module = " test.i_am_not_here " | ||||
|         result = str(run_pydoc(missing_module), 'ascii') | ||||
|         result = str(run_pydoc_fail(missing_module), 'ascii') | ||||
|         expected = missing_pattern % missing_module.strip() | ||||
|         self.assertEqual(expected, result) | ||||
| 
 | ||||
|  | @ -902,7 +910,7 @@ def test_badimport(self): | |||
|         for importstring, expectedinmsg in testpairs: | ||||
|             with open(sourcefn, 'w') as f: | ||||
|                 f.write("import {}\n".format(importstring)) | ||||
|             result = run_pydoc(modname, PYTHONPATH=TESTFN).decode("ascii") | ||||
|             result = run_pydoc_fail(modname, PYTHONPATH=TESTFN).decode("ascii") | ||||
|             expected = badimport_pattern % (modname, expectedinmsg) | ||||
|             self.assertEqual(expected, result) | ||||
| 
 | ||||
|  |  | |||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Gregory Anders
						Gregory Anders