add/edit comments

Author: VasuBhog

samples/samples-powershell/AddProductParams/run.ps1

@@ -6,6 +6,7 @@ param($Request)
 # Write to the Azure Functions log stream.
 Write-Host "PowerShell function with SQL Output Binding processed a request."
 
+# Update req_query with the query of the request
 $req_query = @{ 
     "productId"= $Request.QUERY.productId;
     "name"= $Request.QUERY.name;
@@ -20,5 +21,5 @@ Push-OutputBinding -Name product -Value $req_query
 # The -Name value matches the name property in the function.json for the binding
 Push-OutputBinding -Name response -Value ([HttpResponseContext]@{
     StatusCode = [HttpStatusCode]::OK
-    Body = $req_body
+    Body = $req_query
 })

samples/samples-powershell/AddProductWithDefaultPK/function.json

@@ -6,7 +6,6 @@
         "direction": "in",
         "type": "httpTrigger",
         "methods": [
-          "get",
           "post"
         ],
         "route": "addproductwithdefaultpk"

samples/samples-powershell/AddProductWithIdentityColumn/run.ps1

@@ -6,21 +6,19 @@ param($Request)
 # Write to the Azure Functions log stream.
 Write-Host "PowerShell function with SQL Output Binding processed a request."
 
-# Update req_body with the body of the request
-# Note that this expects the body to be a JSON object or array of objects 
-# which have a property matching each of the columns in the table to upsert to.
-$req_body = @{
+# Update req_query with the query of the request
+$req_query = @{
     name=$Request.QUERY.name;
     cost=$Request.QUERY.cost;
 };
 
 # Assign the value we want to pass to the SQL Output binding. 
 # The -Name value corresponds to the name property in the function.json for the binding
-Push-OutputBinding -Name product -Value $req_body
+Push-OutputBinding -Name product -Value $req_query
 
 # Assign the value to return as the HTTP response. 
 # The -Name value matches the name property in the function.json for the binding
 Push-OutputBinding -Name response -Value ([HttpResponseContext]@{
     StatusCode = [HttpStatusCode]::OK
-    Body = $req_body
+    Body = $req_query
 })

samples/samples-powershell/AddProductWithIdentityColumnIncluded/run.ps1

@@ -6,22 +6,20 @@ param($Request)
 # Write to the Azure Functions log stream.
 Write-Host "PowerShell function with SQL Output Binding processed a request."
 
-# Update req_body with the body of the request
-# Note that this expects the body to be a JSON object or array of objects 
-# which have a property matching each of the columns in the table to upsert to.
-$req_body = @{
+# Update req_query with the body of the request
+$req_query = @{
     productId= if($Request.QUERY.productId) { $Request.QUERY.productId } else { $null };
     name=$Request.QUERY.name;
     cost=$Request.QUERY.cost;
 };
 
 # Assign the value we want to pass to the SQL Output binding. 
 # The -Name value corresponds to the name property in the function.json for the binding
-Push-OutputBinding -Name product -Value $req_body
+Push-OutputBinding -Name product -Value $req_query
 
 # Assign the value to return as the HTTP response. 
 # The -Name value matches the name property in the function.json for the binding
 Push-OutputBinding -Name response -Value ([HttpResponseContext]@{
     StatusCode = [HttpStatusCode]::OK
-    Body = $req_body
+    Body = $req_query
 })

samples/samples-powershell/AddProductWithMultiplePrimaryColumnsAndIdentity/run.ps1

@@ -7,8 +7,6 @@ param($Request)
 Write-Host "PowerShell function with SQL Output Binding processed a request."
 
 # Update req_query with the body of the request
-# Note that this expects the body to be a JSON object or array of objects 
-# which have a property matching each of the columns in the table to upsert to.
 $req_query = @{
     externalId=$Request.QUERY.externalId;
     name=$Request.QUERY.name;

test/Integration/SqlOutputBindingIntegrationTests.cs

@@ -45,7 +45,7 @@ public void AddProductTest(int id, string name, int cost, SupportedLanguages lan
         [SqlInlineData(1, "Test", 5)]
         [SqlInlineData(0, "", 0)]
         [SqlInlineData(-500, "ABCD", 580)]
-        // Currently PowerShell returns null if the output
+        // Currently PowerShell returns null when the parameter for name is an empty string
         // Issue link: https://github.com/Azure/azure-functions-sql-extension/issues/443
         [UnsupportedLanguages(SupportedLanguages.PowerShell)]
         public void AddProductParamsTest(int id, string name, int cost, SupportedLanguages lang)