6332beef65
* Start of tests for ansible.module_utils.urls * Start adding file for generic functions throughout urls * Add tests for maybe_add_ssl_handler * Remove commented out line * Improve coverage of maybe_add_ssl_handler, test basic_auth_header * Start tests for open_url * pep8 and ignore urlopen in test_url_open.py tests * Extend auth tests, add test for validate_certs=False * Finish tests for open_url * Add tests for fetch_url * Add fetch_url tests to replace-urlopen ignore * dummy instead of _ * Add BadStatusLine test * Reorganize/rename tests * Add tests for RedirectHandlerFactory * Add POST test to confirm behavior is to convert to GET * Update tests to handle recent changes to RedirectHandlerFactory * Special test, just to confirm that aliasing http_error_308 to http_error_307 does not cause issues with urllib2 type redirects
31 lines
880 B
Python
Executable file
31 lines
880 B
Python
Executable file
#!/usr/bin/env python
|
|
|
|
import os
|
|
import re
|
|
import sys
|
|
|
|
|
|
def main():
|
|
skip = set([
|
|
'test/sanity/code-smell/%s' % os.path.basename(__file__),
|
|
'lib/ansible/module_utils/six/__init__.py',
|
|
'lib/ansible/module_utils/urls.py',
|
|
'test/units/module_utils/urls/test_open_url.py',
|
|
'test/units/module_utils/urls/test_fetch_url.py',
|
|
])
|
|
|
|
for path in sys.argv[1:] or sys.stdin.read().splitlines():
|
|
if path in skip:
|
|
continue
|
|
|
|
with open(path, 'r') as path_fd:
|
|
for line, text in enumerate(path_fd.readlines()):
|
|
match = re.search(r'^(?:[^#]*?)(urlopen)', text)
|
|
|
|
if match:
|
|
print('%s:%d:%d: use `ansible.module_utils.urls.open_url` instead of `urlopen`' % (
|
|
path, line + 1, match.start(1) + 1))
|
|
|
|
|
|
if __name__ == '__main__':
|
|
main()
|