Added parameter documentation to model constructors (Azure#1184)

* Constructor Parameter Documentation for models

* Regenerated files

* Added culture invariant to char.tolower

* Removed unneccessary logic in createsignaturedocumentation

* Removed unneeded null or whitespace check

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureCompositeModelClient/Models/Basic.cs

@@ -26,6 +26,7 @@ public partial class Basic
         /// <summary>
         /// Initializes a new instance of the Basic class.
         /// </summary>
+        /// <param name="color">Possible values include: 'cyan', 'Magenta', 'YELLOW', 'blacK'</param>
         public Basic(int? id = default(int?), string name = default(string), string color = default(string))
         {
             Id = id;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureCompositeModelClient/Models/CatalogArray.cs

@@ -26,6 +26,7 @@ public partial class CatalogArray
         /// <summary>
         /// Initializes a new instance of the CatalogArray class.
         /// </summary>
+        /// <param name="productArray">Array of products</param>
         public CatalogArray(IList<Product> productArray = default(IList<Product>))
         {
             ProductArray = productArray;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureCompositeModelClient/Models/CatalogArrayOfDictionary.cs

@@ -26,6 +26,7 @@ public partial class CatalogArrayOfDictionary
         /// <summary>
         /// Initializes a new instance of the CatalogArrayOfDictionary class.
         /// </summary>
+        /// <param name="productArrayOfDictionary">Array of dictionary of products</param>
         public CatalogArrayOfDictionary(IList<IDictionary<string, Product>> productArrayOfDictionary = default(IList<IDictionary<string, Product>>))
         {
             ProductArrayOfDictionary = productArrayOfDictionary;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureCompositeModelClient/Models/CatalogDictionary.cs

@@ -26,6 +26,7 @@ public partial class CatalogDictionary
         /// <summary>
         /// Initializes a new instance of the CatalogDictionary class.
         /// </summary>
+        /// <param name="productDictionary">Dictionary of products</param>
         public CatalogDictionary(IDictionary<string, Product> productDictionary = default(IDictionary<string, Product>))
         {
             ProductDictionary = productDictionary;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureCompositeModelClient/Models/CatalogDictionaryOfArray.cs

@@ -26,6 +26,7 @@ public partial class CatalogDictionaryOfArray
         /// <summary>
         /// Initializes a new instance of the CatalogDictionaryOfArray class.
         /// </summary>
+        /// <param name="productDictionaryOfArray">Dictionary of Array of product</param>
         public CatalogDictionaryOfArray(IDictionary<string, IList<Product>> productDictionaryOfArray = default(IDictionary<string, IList<Product>>))
         {
             ProductDictionaryOfArray = productDictionaryOfArray;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureCompositeModelClient/Models/Product.cs

@@ -29,6 +29,11 @@ public partial class Product
         /// <summary>
         /// Initializes a new instance of the Product class.
         /// </summary>
+        /// <param name="productId">Unique identifier representing a specific product for a given latitude & longitude. For example, uberX in San Francisco will have a different product_id than uberX in Los Angeles.</param>
+        /// <param name="description">Description of product.</param>
+        /// <param name="displayName">Display name of product.</param>
+        /// <param name="capacity">Capacity of product. For example, 4 people.</param>
+        /// <param name="image">Image URL representing the product.</param>
         public Product(string productId = default(string), string description = default(string), string displayName = default(string), string capacity = default(string), string image = default(string))
         {
             ProductId = productId;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureParameterGrouping/Models/FirstParameterGroup.cs

@@ -31,6 +31,7 @@ public partial class FirstParameterGroup
         /// <summary>
         /// Initializes a new instance of the FirstParameterGroup class.
         /// </summary>
+        /// <param name="queryOne">Query parameter with default</param>
         public FirstParameterGroup(string headerOne = default(string), int? queryOne = default(int?))
         {
             HeaderOne = headerOne;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureParameterGrouping/Models/ParameterGroupingPostMultiParamGroupsSecondParamGroup.cs

@@ -32,6 +32,7 @@ public partial class ParameterGroupingPostMultiParamGroupsSecondParamGroup
         /// Initializes a new instance of the
         /// ParameterGroupingPostMultiParamGroupsSecondParamGroup class.
         /// </summary>
+        /// <param name="queryTwo">Query parameter with default</param>
         public ParameterGroupingPostMultiParamGroupsSecondParamGroup(string headerTwo = default(string), int? queryTwo = default(int?))
         {
             HeaderTwo = headerTwo;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureParameterGrouping/Models/ParameterGroupingPostOptionalParameters.cs

@@ -31,6 +31,7 @@ public partial class ParameterGroupingPostOptionalParameters
         /// Initializes a new instance of the
         /// ParameterGroupingPostOptionalParameters class.
         /// </summary>
+        /// <param name="query">Query parameter with default</param>
         public ParameterGroupingPostOptionalParameters(string customHeader = default(string), int? query = default(int?))
         {
             CustomHeader = customHeader;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureParameterGrouping/Models/ParameterGroupingPostRequiredParameters.cs

@@ -31,6 +31,8 @@ public partial class ParameterGroupingPostRequiredParameters
         /// Initializes a new instance of the
         /// ParameterGroupingPostRequiredParameters class.
         /// </summary>
+        /// <param name="path">Path parameter</param>
+        /// <param name="query">Query parameter with default</param>
         public ParameterGroupingPostRequiredParameters(int body, string path, string customHeader = default(string), int? query = default(int?))
         {
             Body = body;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureResource/Models/FlattenedProduct.cs

@@ -27,6 +27,10 @@ public partial class FlattenedProduct : Resource
         /// <summary>
         /// Initializes a new instance of the FlattenedProduct class.
         /// </summary>
+        /// <param name="id">Resource Id</param>
+        /// <param name="type">Resource Type</param>
+        /// <param name="location">Resource Location</param>
+        /// <param name="name">Resource Name</param>
         public FlattenedProduct(string id = default(string), string type = default(string), IDictionary<string, string> tags = default(IDictionary<string, string>), string location = default(string), string name = default(string), string pname = default(string), int? lsize = default(int?), string provisioningState = default(string))
             : base(id, type, tags, location, name)
         {

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureResource/Models/Resource.cs

@@ -30,6 +30,10 @@ public partial class Resource : IResource
         /// <summary>
         /// Initializes a new instance of the Resource class.
         /// </summary>
+        /// <param name="id">Resource Id</param>
+        /// <param name="type">Resource Type</param>
+        /// <param name="location">Resource Location</param>
+        /// <param name="name">Resource Name</param>
         public Resource(string id = default(string), string type = default(string), IDictionary<string, string> tags = default(IDictionary<string, string>), string location = default(string), string name = default(string))
         {
             Id = id;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureSpecials/Models/HeaderCustomNamedRequestIdHeaders.cs

@@ -31,6 +31,7 @@ public partial class HeaderCustomNamedRequestIdHeaders
         /// Initializes a new instance of the
         /// HeaderCustomNamedRequestIdHeaders class.
         /// </summary>
+        /// <param name="fooRequestId">Gets the foo-request-id.</param>
         public HeaderCustomNamedRequestIdHeaders(string fooRequestId = default(string))
         {
             FooRequestId = fooRequestId;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureSpecials/Models/HeaderCustomNamedRequestIdParamGroupingHeaders.cs

@@ -31,6 +31,7 @@ public partial class HeaderCustomNamedRequestIdParamGroupingHeaders
         /// Initializes a new instance of the
         /// HeaderCustomNamedRequestIdParamGroupingHeaders class.
         /// </summary>
+        /// <param name="fooRequestId">Gets the foo-request-id.</param>
         public HeaderCustomNamedRequestIdParamGroupingHeaders(string fooRequestId = default(string))
         {
             FooRequestId = fooRequestId;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/AzureSpecials/Models/HeaderCustomNamedRequestIdParamGroupingParameters.cs

@@ -32,6 +32,7 @@ public partial class HeaderCustomNamedRequestIdParamGroupingParameters
         /// Initializes a new instance of the
         /// HeaderCustomNamedRequestIdParamGroupingParameters class.
         /// </summary>
+        /// <param name="fooClientRequestId">The fooRequestId</param>
         public HeaderCustomNamedRequestIdParamGroupingParameters(string fooClientRequestId)
         {
             FooClientRequestId = fooClientRequestId;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LRORetrysDelete202Retry200Headers.cs

@@ -31,6 +31,8 @@ public partial class LRORetrysDelete202Retry200Headers
         /// Initializes a new instance of the
         /// LRORetrysDelete202Retry200Headers class.
         /// </summary>
+        /// <param name="location">Location to poll for result status: will be set to /lro/retryerror/delete/202/retry/200</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LRORetrysDelete202Retry200Headers(string location = default(string), int? retryAfter = default(int?))
         {
             Location = location;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LRORetrysDeleteAsyncRelativeRetrySucceededHeaders.cs

@@ -31,6 +31,9 @@ public partial class LRORetrysDeleteAsyncRelativeRetrySucceededHeaders
         /// Initializes a new instance of the
         /// LRORetrysDeleteAsyncRelativeRetrySucceededHeaders class.
         /// </summary>
+        /// <param name="azureAsyncOperation">Location to poll for result status: will be set to /lro/retryerror/deleteasync/retry/succeeded/operationResults/200</param>
+        /// <param name="location">Location to poll for result status: will be set to /lro/retryerror/deleteasync/retry/succeeded/operationResults/200</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LRORetrysDeleteAsyncRelativeRetrySucceededHeaders(string azureAsyncOperation = default(string), string location = default(string), int? retryAfter = default(int?))
         {
             AzureAsyncOperation = azureAsyncOperation;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LRORetrysDeleteProvisioning202Accepted200SucceededHeaders.cs

@@ -32,6 +32,8 @@ public partial class LRORetrysDeleteProvisioning202Accepted200SucceededHeaders
         /// Initializes a new instance of the
         /// LRORetrysDeleteProvisioning202Accepted200SucceededHeaders class.
         /// </summary>
+        /// <param name="location">Location to poll for result status: will be set to /lro/retryerror/delete/provisioning/202/accepted/200/succeeded</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LRORetrysDeleteProvisioning202Accepted200SucceededHeaders(string location = default(string), int? retryAfter = default(int?))
         {
             Location = location;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LRORetrysPost202Retry200Headers.cs

@@ -31,6 +31,8 @@ public partial class LRORetrysPost202Retry200Headers
         /// Initializes a new instance of the LRORetrysPost202Retry200Headers
         /// class.
         /// </summary>
+        /// <param name="location">Location to poll for result status: will be set to /lro/retryerror/post/202/retry/200</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LRORetrysPost202Retry200Headers(string location = default(string), int? retryAfter = default(int?))
         {
             Location = location;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LRORetrysPostAsyncRelativeRetrySucceededHeaders.cs

@@ -31,6 +31,9 @@ public partial class LRORetrysPostAsyncRelativeRetrySucceededHeaders
         /// Initializes a new instance of the
         /// LRORetrysPostAsyncRelativeRetrySucceededHeaders class.
         /// </summary>
+        /// <param name="azureAsyncOperation">Location to poll for result status: will be set to /lro/retryerror/putasync/retry/succeeded/operationResults/200</param>
+        /// <param name="location">Location to poll for result status: will be set to /lro/retryerror/putasync/retry/succeeded/operationResults/200</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LRORetrysPostAsyncRelativeRetrySucceededHeaders(string azureAsyncOperation = default(string), string location = default(string), int? retryAfter = default(int?))
         {
             AzureAsyncOperation = azureAsyncOperation;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LRORetrysPutAsyncRelativeRetrySucceededHeaders.cs

@@ -31,6 +31,9 @@ public partial class LRORetrysPutAsyncRelativeRetrySucceededHeaders
         /// Initializes a new instance of the
         /// LRORetrysPutAsyncRelativeRetrySucceededHeaders class.
         /// </summary>
+        /// <param name="azureAsyncOperation">Location to poll for result status: will be set to /lro/retryerror/putasync/retry/succeeded/operationResults/200</param>
+        /// <param name="location">Location to poll for result status: will be set to /lro/retryerror/putasync/retry/succeeded/operationResults/200</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LRORetrysPutAsyncRelativeRetrySucceededHeaders(string azureAsyncOperation = default(string), string location = default(string), int? retryAfter = default(int?))
         {
             AzureAsyncOperation = azureAsyncOperation;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LROSADsDelete202NonRetry400Headers.cs

@@ -31,6 +31,8 @@ public partial class LROSADsDelete202NonRetry400Headers
         /// Initializes a new instance of the
         /// LROSADsDelete202NonRetry400Headers class.
         /// </summary>
+        /// <param name="location">Location to poll for result status: will be set to /lro/retryerror/delete/202/retry/200</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LROSADsDelete202NonRetry400Headers(string location = default(string), int? retryAfter = default(int?))
         {
             Location = location;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LROSADsDelete202RetryInvalidHeaderHeaders.cs

@@ -31,6 +31,8 @@ public partial class LROSADsDelete202RetryInvalidHeaderHeaders
         /// Initializes a new instance of the
         /// LROSADsDelete202RetryInvalidHeaderHeaders class.
         /// </summary>
+        /// <param name="location">Location to poll for result status: will be set to /foo</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to /bar</param>
         public LROSADsDelete202RetryInvalidHeaderHeaders(string location = default(string), int? retryAfter = default(int?))
         {
             Location = location;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LROSADsDeleteAsyncRelativeRetry400Headers.cs

@@ -31,6 +31,9 @@ public partial class LROSADsDeleteAsyncRelativeRetry400Headers
         /// Initializes a new instance of the
         /// LROSADsDeleteAsyncRelativeRetry400Headers class.
         /// </summary>
+        /// <param name="azureAsyncOperation">Location to poll for result status: will be set to /lro/nonretryerror/deleteasync/retry/operationResults/400</param>
+        /// <param name="location">Location to poll for result status: will be set to /lro/nonretryerror/deleteasync/retry/operationResults/400</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LROSADsDeleteAsyncRelativeRetry400Headers(string azureAsyncOperation = default(string), string location = default(string), int? retryAfter = default(int?))
         {
             AzureAsyncOperation = azureAsyncOperation;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LROSADsDeleteAsyncRelativeRetryInvalidHeaderHeaders.cs

@@ -31,6 +31,9 @@ public partial class LROSADsDeleteAsyncRelativeRetryInvalidHeaderHeaders
         /// Initializes a new instance of the
         /// LROSADsDeleteAsyncRelativeRetryInvalidHeaderHeaders class.
         /// </summary>
+        /// <param name="azureAsyncOperation">Location to poll for result status: will be set to /foo</param>
+        /// <param name="location">Location to poll for result status: will be set to /foo</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to /bar</param>
         public LROSADsDeleteAsyncRelativeRetryInvalidHeaderHeaders(string azureAsyncOperation = default(string), string location = default(string), int? retryAfter = default(int?))
         {
             AzureAsyncOperation = azureAsyncOperation;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LROSADsDeleteAsyncRelativeRetryInvalidJsonPollingHeaders.cs

@@ -32,6 +32,9 @@ public partial class LROSADsDeleteAsyncRelativeRetryInvalidJsonPollingHeaders
         /// Initializes a new instance of the
         /// LROSADsDeleteAsyncRelativeRetryInvalidJsonPollingHeaders class.
         /// </summary>
+        /// <param name="azureAsyncOperation">Location to poll for result status: will be set to /lro/error/deleteasync/retry/failed/operationResults/invalidjsonpolling</param>
+        /// <param name="location">Location to poll for result status: will be set to /lro/error/deleteasync/retry/failed/operationResults/invalidjsonpolling</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LROSADsDeleteAsyncRelativeRetryInvalidJsonPollingHeaders(string azureAsyncOperation = default(string), string location = default(string), int? retryAfter = default(int?))
         {
             AzureAsyncOperation = azureAsyncOperation;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LROSADsDeleteAsyncRelativeRetryNoStatusHeaders.cs

@@ -31,6 +31,9 @@ public partial class LROSADsDeleteAsyncRelativeRetryNoStatusHeaders
         /// Initializes a new instance of the
         /// LROSADsDeleteAsyncRelativeRetryNoStatusHeaders class.
         /// </summary>
+        /// <param name="azureAsyncOperation">Location to poll for result status: will be set to /lro/deleteasync/retry/succeeded/operationResults/200</param>
+        /// <param name="location">Location to poll for result status: will be set to /lro/deleteasync/retry/succeeded/operationResults/200</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LROSADsDeleteAsyncRelativeRetryNoStatusHeaders(string azureAsyncOperation = default(string), string location = default(string), int? retryAfter = default(int?))
         {
             AzureAsyncOperation = azureAsyncOperation;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LROSADsDeleteNonRetry400Headers.cs

@@ -31,6 +31,8 @@ public partial class LROSADsDeleteNonRetry400Headers
         /// Initializes a new instance of the LROSADsDeleteNonRetry400Headers
         /// class.
         /// </summary>
+        /// <param name="location">Location to poll for result status: will be set to /lro/retryerror/delete/202/retry/200</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LROSADsDeleteNonRetry400Headers(string location = default(string), int? retryAfter = default(int?))
         {
             Location = location;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LROSADsPost202NoLocationHeaders.cs

@@ -31,6 +31,8 @@ public partial class LROSADsPost202NoLocationHeaders
         /// Initializes a new instance of the LROSADsPost202NoLocationHeaders
         /// class.
         /// </summary>
+        /// <param name="location">Location to poll for result status: will not be set</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LROSADsPost202NoLocationHeaders(string location = default(string), int? retryAfter = default(int?))
         {
             Location = location;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LROSADsPost202NonRetry400Headers.cs

@@ -31,6 +31,8 @@ public partial class LROSADsPost202NonRetry400Headers
         /// Initializes a new instance of the LROSADsPost202NonRetry400Headers
         /// class.
         /// </summary>
+        /// <param name="location">Location to poll for result status: will be set to /lro/retryerror/post/202/retry/200</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to zero</param>
         public LROSADsPost202NonRetry400Headers(string location = default(string), int? retryAfter = default(int?))
         {
             Location = location;

AutoRest/Generators/CSharp/Azure.CSharp.Tests/Expected/AcceptanceTests/Lro/Models/LROSADsPost202RetryInvalidHeaderHeaders.cs

@@ -31,6 +31,8 @@ public partial class LROSADsPost202RetryInvalidHeaderHeaders
         /// Initializes a new instance of the
         /// LROSADsPost202RetryInvalidHeaderHeaders class.
         /// </summary>
+        /// <param name="location">Location to poll for result status: will be set to /foo</param>
+        /// <param name="retryAfter">Number of milliseconds until the next poll should be sent, will be set to /bar</param>
         public LROSADsPost202RetryInvalidHeaderHeaders(string location = default(string), int? retryAfter = default(int?))
         {
             Location = location;