pulled in @stankovski 's fix for the comment problem (fixes #1228) (#…

…1243)

[Reviewed and merged]

src/dev/TestServer/swagger/body-complex.json

@@ -1121,10 +1121,15 @@
     "basic": {
       "properties": {
         "id": {
-          "type": "integer"
+          "type": "integer",
+          "description": "Basic Id"
         },
         "name": {
-          "type": "string"
+          "type": "string",
+          "description": "Name property with a very 
+          long description that 
+          does not fit on a single line 
+          and a line break."
         },
         "color": {
           "type": "string",

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

@@ -26,7 +26,13 @@ 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>
+        /// <param name="id">Basic Id</param>
+        /// <param name="name">Name property with a very
+        /// long description that
+        /// does not fit on a single line
+        /// and a line break.</param>
+        /// <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;
@@ -35,11 +41,16 @@ public Basic(int? id = default(int?), string name = default(string), string colo
         }
 
         /// <summary>
+        /// Gets or sets basic Id
         /// </summary>
         [JsonProperty(PropertyName = "id")]
         public int? Id { get; set; }
 
         /// <summary>
+        /// Gets or sets name property with a very
+        /// long description that
+        /// does not fit on a single line
+        /// and a line break.
         /// </summary>
         [JsonProperty(PropertyName = "name")]
         public string Name { get; set; }

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

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

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

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

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

@@ -29,10 +29,14 @@ 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="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="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))
         {

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

@@ -31,8 +31,10 @@ 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>
+        /// <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;

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

@@ -31,9 +31,14 @@ 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>
+        /// <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;

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

@@ -32,8 +32,11 @@ 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>
+        /// <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;

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

@@ -31,8 +31,10 @@ 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>
+        /// <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;

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

@@ -31,9 +31,14 @@ 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>
+        /// <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;

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

@@ -31,9 +31,14 @@ 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>
+        /// <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;

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

@@ -31,8 +31,10 @@ 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>
+        /// <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;

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

@@ -31,8 +31,10 @@ 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>
+        /// <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;

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

@@ -31,9 +31,14 @@ 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>
+        /// <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;

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

@@ -31,9 +31,12 @@ 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>
+        /// <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;

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

@@ -32,9 +32,14 @@ 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>
+        /// <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;

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

@@ -31,9 +31,14 @@ 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>
+        /// <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;

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

@@ -31,8 +31,10 @@ 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>
+        /// <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;

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

@@ -31,8 +31,10 @@ 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>
+        /// <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;