win_uri: fix creates/removes option (#36016)
This commit is contained in:
parent
6a4f6a6490
commit
f8d1c97279
2 changed files with 29 additions and 5 deletions
|
@ -37,6 +37,11 @@ $validate_certs = Get-AnsibleParam -obj $params -name "validate_certs" -type "bo
|
||||||
$client_cert = Get-AnsibleParam -obj $params -name "client_cert" -type "path"
|
$client_cert = Get-AnsibleParam -obj $params -name "client_cert" -type "path"
|
||||||
$client_cert_password = Get-AnsibleParam -obj $params -name "client_cert_password" -type "str"
|
$client_cert_password = Get-AnsibleParam -obj $params -name "client_cert_password" -type "str"
|
||||||
|
|
||||||
|
$result = @{
|
||||||
|
changed = $false
|
||||||
|
url = $url
|
||||||
|
}
|
||||||
|
|
||||||
if ($creates -and (Test-AnsiblePath -Path $creates)) {
|
if ($creates -and (Test-AnsiblePath -Path $creates)) {
|
||||||
$result.skipped = $true
|
$result.skipped = $true
|
||||||
Exit-Json -obj $result -message "The 'creates' file or directory ($creates) already exists."
|
Exit-Json -obj $result -message "The 'creates' file or directory ($creates) already exists."
|
||||||
|
@ -47,11 +52,6 @@ if ($removes -and -not (Test-AnsiblePath -Path $removes)) {
|
||||||
Exit-Json -obj $result -message "The 'removes' file or directory ($removes) does not exist."
|
Exit-Json -obj $result -message "The 'removes' file or directory ($removes) does not exist."
|
||||||
}
|
}
|
||||||
|
|
||||||
$result = @{
|
|
||||||
changed = $false
|
|
||||||
url = $url
|
|
||||||
}
|
|
||||||
|
|
||||||
if ($use_basic_parsing) {
|
if ($use_basic_parsing) {
|
||||||
Add-DeprecationWarning -obj $result -message "Since Ansible 2.5, use_basic_parsing does not change any behaviour, this option will be removed" -version 2.7
|
Add-DeprecationWarning -obj $result -message "Since Ansible 2.5, use_basic_parsing does not change any behaviour, this option will be removed" -version 2.7
|
||||||
}
|
}
|
||||||
|
|
|
@ -100,6 +100,30 @@
|
||||||
that:
|
that:
|
||||||
- not get_request_with_dest_again.changed
|
- not get_request_with_dest_again.changed
|
||||||
|
|
||||||
|
- name: test request with creates option should skip
|
||||||
|
win_uri:
|
||||||
|
url: http://{{httpbin_host}}/get
|
||||||
|
creates: '{{test_uri_path}}\get.json'
|
||||||
|
register: request_with_creates_skipped
|
||||||
|
|
||||||
|
- name: assert test request with creates option should skip
|
||||||
|
assert:
|
||||||
|
that:
|
||||||
|
- not request_with_creates_skipped.changed
|
||||||
|
- request_with_creates_skipped.skipped
|
||||||
|
|
||||||
|
- name: test request with creates option should not skip
|
||||||
|
win_uri:
|
||||||
|
url: http://{{httpbin_host}}/get
|
||||||
|
creates: '{{test_uri_path}}\fake.json'
|
||||||
|
register: request_with_creates_not_skipped
|
||||||
|
|
||||||
|
- name: assert test request with creates option should not skip
|
||||||
|
assert:
|
||||||
|
that:
|
||||||
|
- not request_with_creates_not_skipped.changed
|
||||||
|
- request_with_creates_not_skipped.skipped is not defined
|
||||||
|
|
||||||
- name: post request with return_content, dest and different content
|
- name: post request with return_content, dest and different content
|
||||||
win_uri:
|
win_uri:
|
||||||
url: http://{{httpbin_host}}/post
|
url: http://{{httpbin_host}}/post
|
||||||
|
|
Loading…
Reference in a new issue