win_ping: Make full-functional

Small cleanup and backport `ping` functionality.

lib/ansible/modules/windows/win_ping.ps1

@@ -17,13 +17,19 @@
 # WANT_JSON
 # POWERSHELL_COMMON
 
-$params = Parse-Args $args $true;
+$ErrorActionPreference = "Stop"
 
-$data = Get-Attr $params "data" "pong";
+$params = Parse-Args $args -supports_check_mode $true
 
-$result = New-Object psobject @{
+$data = Get-AnsibleParam -obj $params -name "data" -type "str" -default "pong"
+
+if ($data -eq "crash") {
+    throw "boom"
+}
+
+$result = @{
     changed = $false
     ping = $data
-};
+}
 
-Exit-Json $result;
+Exit-Json $result

lib/ansible/modules/windows/win_ping.py

@@ -36,9 +36,7 @@ options:
   data:
     description:
       - Alternate data to return instead of 'pong'
-    required: false
     default: 'pong'
-    aliases: []
 author: "Chris Church (@cchurch)"
 '''
 
@@ -47,6 +45,10 @@ EXAMPLES = r'''
 ansible winserver -m win_ping
 
 # Example from an Ansible Playbook
-- action: win_ping
+- win_ping:
+
+# Induce a crash to see what happens
+- win_ping:
+    data: crash
 '''