Merge pull request Azure#4347 from haitch/preview

fix OperationalInsight searchResult backward compatible issue

Author: markcowl

src/ResourceManager/OperationalInsights/Commands.OperationalInsights.Test/ScenarioTests/SearchTests.ps1

@@ -30,6 +30,12 @@ function Test-SearchGetSearchResultsAndUpdate
 	Assert-NotNull $searchResult.Value
 	Assert-AreEqual $searchResult.Value.Count $top
 
+	# Makesure we return each doc as string for backward compatiable.
+	$stringType = "string".GetType()
+	$valueType = $searchResult.Value.GetType()
+	$valueIsString = $valueType.GenericTypeArguments.Contains($stringType)
+	Assert-AreEqual $true $valueIsString
+
 	$idArray = $searchResult.Id.Split("/")
 	$id = $idArray[$idArray.Length-1]
 	$updatedResult = Get-AzureRmOperationalInsightsSearchResults -ResourceGroupName $rgname -WorkspaceName $wsname -Id $id

src/ResourceManager/OperationalInsights/Commands.OperationalInsights/Models/PSSearchGetSearchResultsResponse.cs

@@ -14,6 +14,7 @@
 
 using Microsoft.Azure.Management.OperationalInsights.Models;
 using System;
+using System.Linq;
 using System.Collections.Generic;
 
 namespace Microsoft.Azure.Commands.OperationalInsights.Models
@@ -30,7 +31,7 @@ public PSSearchGetSearchResultsResponse(SearchResultsResponse response)
             {
                 this.Id = response.Id;
                 this.Metadata = new PSSearchMetadata(response.Metadata);
-                this.Value = response.Value;
+                this.Value = response.Value.Select(jObj=>jObj.ToString()).ToList();
 
                 if (response.Error != null)
                 {