mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
Make sure raw doesn't eat key=value arguments.
This commit is contained in:
parent
cf35bdbdf9
commit
6969b5ac8b
2 changed files with 11 additions and 2 deletions
|
@ -181,7 +181,7 @@ class ModuleArgsParser:
|
||||||
args = thing
|
args = thing
|
||||||
elif isinstance(thing, string_types):
|
elif isinstance(thing, string_types):
|
||||||
# form is like: local_action: copy src=a dest=b ... pretty common
|
# form is like: local_action: copy src=a dest=b ... pretty common
|
||||||
check_raw = action in ('command', 'shell', 'script')
|
check_raw = action in ('command', 'shell', 'script', 'raw')
|
||||||
args = parse_kv(thing, check_raw=check_raw)
|
args = parse_kv(thing, check_raw=check_raw)
|
||||||
elif thing is None:
|
elif thing is None:
|
||||||
# this can happen with modules which take no params, like ping:
|
# this can happen with modules which take no params, like ping:
|
||||||
|
@ -218,7 +218,7 @@ class ModuleArgsParser:
|
||||||
elif isinstance(thing, string_types):
|
elif isinstance(thing, string_types):
|
||||||
# form is like: copy: src=a dest=b ... common shorthand throughout ansible
|
# form is like: copy: src=a dest=b ... common shorthand throughout ansible
|
||||||
(action, args) = self._split_module_string(thing)
|
(action, args) = self._split_module_string(thing)
|
||||||
check_raw = action in ('command', 'shell', 'script')
|
check_raw = action in ('command', 'shell', 'script', 'raw')
|
||||||
args = parse_kv(args, check_raw=check_raw)
|
args = parse_kv(args, check_raw=check_raw)
|
||||||
|
|
||||||
else:
|
else:
|
||||||
|
|
|
@ -83,3 +83,12 @@
|
||||||
- "not sleep_command.stderr"
|
- "not sleep_command.stderr"
|
||||||
- "not sleep_command|failed"
|
- "not sleep_command|failed"
|
||||||
- "not sleep_command|changed"
|
- "not sleep_command|changed"
|
||||||
|
|
||||||
|
- name: run a raw command with key=value arguments
|
||||||
|
raw: echo wwe=raw
|
||||||
|
register: raw_result
|
||||||
|
|
||||||
|
- name: make sure raw is really raw and not removing key=value arguments
|
||||||
|
assert:
|
||||||
|
that:
|
||||||
|
- "raw_result.stdout_lines[0] == 'wwe=raw'"
|
||||||
|
|
Loading…
Reference in a new issue