cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fixes for missing aliases files sanity test. (#21798)

Browse source 
* Refine test for missing aliases files.
* Fix formatting of missing aliases message.
This commit is contained in:
Matt Clay 2017-02-22 12:04:09 -08:00 committed by GitHub
parent f8157d579a
commit b5e0eb0985
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
test/sanity/code-smell/integration-aliases.py
View file

@ -22,7 +22,7 @@ def main():
continue continue
# don't require aliases for support directories # don't require aliases for support directories
if any(os.path.splitext(f)[0] == 'test' for f in files): if any(os.path.splitext(f)[0] == 'test' and os.access(os.path.join(target_dir, f), os.X_OK) for f in files):
continue continue
# don't require aliases for setup_ directories # don't require aliases for setup_ directories
@ -41,7 +41,7 @@ def main():
missing_aliases.append(target_dir) missing_aliases.append(target_dir)
if missing_aliases: if missing_aliases:
message = ''' message = textwrap.dedent('''
The following integration target directories are missing `aliases` files: The following integration target directories are missing `aliases` files:
%s %s
@ -66,9 +66,9 @@ def main():
skip/python3 skip/python3
Take a look at existing `aliases` files to see what aliases are available and how they're used. Take a look at existing `aliases` files to see what aliases are available and how they're used.
''' % '\n'.join(missing_aliases) ''').strip() % '\n'.join(missing_aliases)
print(textwrap.dedent(message).strip()) print(message)
exit(1) exit(1)