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

Fix logic in process_common_errors for unbalanced quotes

Browse source
This commit is contained in:
Matt Martz 2014-03-19 13:49:31 -05:00
parent 67517e96d3
commit 2aaecc5afa
2 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/ansible/utils/__init__.py
View file

@ -424,7 +424,7 @@ Or:
match = True match = True
elif middle.startswith('"') and not middle.endswith('"'): elif middle.startswith('"') and not middle.endswith('"'):
match = True match = True
if len(middle) > 0 and middle[0] in [ '"', "'" ] and middle[-1] in [ '"', "'" ] and probline.count("'") > 2 or probline.count("'") > 2: if len(middle) > 0 and middle[0] in [ '"', "'" ] and middle[-1] in [ '"', "'" ] and probline.count("'") > 2 or probline.count('"') > 2:
unbalanced = True unbalanced = True
if match: if match:
msg = msg + """ msg = msg + """

3
test/units/TestUtils.py
View file

@ -301,8 +301,7 @@ class TestUtils(unittest.TestCase):
self.assertTrue('same kind of quote' in ansible.utils.process_common_errors('', "foo: '{{bar}}'baz", 6)) self.assertTrue('same kind of quote' in ansible.utils.process_common_errors('', "foo: '{{bar}}'baz", 6))
# unbalanced # unbalanced
# The first test fails and is commented out for now, logic is wrong and the test fails self.assertTrue('We could be wrong' in ansible.utils.process_common_errors('', 'foo: "bad" "wolf"', 6))
#self.assertTrue('We could be wrong' in ansible.utils.process_common_errors('', 'foo: "bad" "wolf"', 6))
self.assertTrue('We could be wrong' in ansible.utils.process_common_errors('', "foo: 'bad' 'wolf'", 6)) self.assertTrue('We could be wrong' in ansible.utils.process_common_errors('', "foo: 'bad' 'wolf'", 6))