From b64e042e0e4397ec0f05e563fcc4d64375972da7 Mon Sep 17 00:00:00 2001 From: Dapeng Zhang Date: Thu, 12 Dec 2024 10:32:39 +0800 Subject: [PATCH] resolve comments --- .../test/Utilities/FormattableStringHelpersTests.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/http-client-csharp/generator/Microsoft.Generator.CSharp/test/Utilities/FormattableStringHelpersTests.cs b/packages/http-client-csharp/generator/Microsoft.Generator.CSharp/test/Utilities/FormattableStringHelpersTests.cs index e184dd9d39..3c8988e7de 100644 --- a/packages/http-client-csharp/generator/Microsoft.Generator.CSharp/test/Utilities/FormattableStringHelpersTests.cs +++ b/packages/http-client-csharp/generator/Microsoft.Generator.CSharp/test/Utilities/FormattableStringHelpersTests.cs @@ -234,7 +234,7 @@ public static IEnumerable TestBuildBreakLines $"third{null}" }).SetName("TestBreakLines_MultipleLineBreaks"); - // TODO: Check if this is valid after we update logic in FormattableStringHelpers to handle FormatSpecifier and \n in one argument + // current solution of format specifier in argument is that we ignore them during the process of line breaking. yield return new TestCaseData( (FormattableString)$"first{"x\ny":L}second\nthird{null}", new List {