Added ClientPipeline.CreateMessage helper overload so codegen looks nicer (Azure#51491)

* added createmessage helper

* typo

* typo

* fixed doc comments

* updated api file

Author: KrzysztofCwalina

sdk/core/System.ClientModel/api/System.ClientModel.net8.0.cs

@@ -171,6 +171,7 @@ internal ClientPipeline() { }
         public static System.ClientModel.Primitives.ClientPipeline Create(System.ClientModel.Primitives.ClientPipelineOptions? options = null) { throw null; }
         public static System.ClientModel.Primitives.ClientPipeline Create(System.ClientModel.Primitives.ClientPipelineOptions options, System.ReadOnlySpan<System.ClientModel.Primitives.PipelinePolicy> perCallPolicies, System.ReadOnlySpan<System.ClientModel.Primitives.PipelinePolicy> perTryPolicies, System.ReadOnlySpan<System.ClientModel.Primitives.PipelinePolicy> beforeTransportPolicies) { throw null; }
         public System.ClientModel.Primitives.PipelineMessage CreateMessage() { throw null; }
+        public System.ClientModel.Primitives.PipelineMessage CreateMessage(System.Uri uri, string method, System.ClientModel.Primitives.PipelineMessageClassifier? classifier = null) { throw null; }
         public void Send(System.ClientModel.Primitives.PipelineMessage message) { }
         public System.Threading.Tasks.ValueTask SendAsync(System.ClientModel.Primitives.PipelineMessage message) { throw null; }
     }

sdk/core/System.ClientModel/api/System.ClientModel.netstandard2.0.cs

@@ -170,6 +170,7 @@ internal ClientPipeline() { }
         public static System.ClientModel.Primitives.ClientPipeline Create(System.ClientModel.Primitives.ClientPipelineOptions? options = null) { throw null; }
         public static System.ClientModel.Primitives.ClientPipeline Create(System.ClientModel.Primitives.ClientPipelineOptions options, System.ReadOnlySpan<System.ClientModel.Primitives.PipelinePolicy> perCallPolicies, System.ReadOnlySpan<System.ClientModel.Primitives.PipelinePolicy> perTryPolicies, System.ReadOnlySpan<System.ClientModel.Primitives.PipelinePolicy> beforeTransportPolicies) { throw null; }
         public System.ClientModel.Primitives.PipelineMessage CreateMessage() { throw null; }
+        public System.ClientModel.Primitives.PipelineMessage CreateMessage(System.Uri uri, string method, System.ClientModel.Primitives.PipelineMessageClassifier? classifier = null) { throw null; }
         public void Send(System.ClientModel.Primitives.PipelineMessage message) { }
         public System.Threading.Tasks.ValueTask SendAsync(System.ClientModel.Primitives.PipelineMessage message) { throw null; }
     }

sdk/core/System.ClientModel/src/Message/PipelineRequest.cs

@@ -10,7 +10,7 @@ namespace System.ClientModel.Primitives;
 /// The type of a <see cref="PipelineRequest"/> is specific to the type of the
 /// <see cref="PipelineTransport"/> used by the <see cref="ClientPipeline"/>
 /// that sends the request.  Because of this,
-/// <see cref="ClientPipeline.CreateMessage"/> is used to create an instance of
+/// <see cref="ClientPipeline.CreateMessage()"/> is used to create an instance of
 /// <see cref="PipelineRequest"/> for a given pipeline.
 /// </summary>
 public abstract class PipelineRequest : IDisposable

sdk/core/System.ClientModel/src/Pipeline/ClientPipeline.cs

@@ -194,6 +194,30 @@ public PipelineMessage CreateMessage()
         return message;
     }
 
+    /// <summary>
+    /// Creates a <see cref="PipelineMessage"/> that can be sent using this
+    /// pipeline instance with the specified URI, HTTP method, and response
+    /// classifier.
+    /// </summary>
+    /// <param name="uri">The <see cref="Uri"/> for the HTTP request.</param>
+    /// <param name="method">The HTTP method for the request.</param>
+    /// <param name="classifier">The <see cref="PipelineMessageClassifier"/>
+    /// to use for determining response success and retry behavior. If not provided,
+    /// <see cref="PipelineMessageClassifier.Default"/> will be used.</param>
+    /// <returns>The created <see cref="PipelineMessage"/> with the specified
+    /// URI, method, and classifier configured.</returns>
+    public PipelineMessage CreateMessage(Uri uri, string method, PipelineMessageClassifier? classifier = default)
+    {
+        Argument.AssertNotNull(uri, nameof(uri));
+        Argument.AssertNotNull(method, nameof(method));
+
+        PipelineMessage message = CreateMessage();
+        message.Request.Uri = uri;
+        message.Request.Method = method;
+        message.ResponseClassifier = classifier ?? PipelineMessageClassifier.Default;
+        return message;
+    }
+
     /// <summary>
     /// Send the provided <see cref="PipelineMessage"/>.
     /// </summary>

sdk/core/System.ClientModel/src/Pipeline/PipelineTransport.cs

@@ -45,7 +45,7 @@ protected PipelineTransport(bool enableLogging, ILoggerFactory? loggerFactory)
     /// <summary>
     /// Create an instance of <see cref="PipelineMessage"/> that can be sent
     /// using this transport instance. This method will rarely be called directly;
-    /// <see cref="ClientPipeline.CreateMessage"/> should be called instead.
+    /// <see cref="ClientPipeline.CreateMessage()"/> should be called instead.
     /// </summary>
     /// <returns>A <see cref="PipelineMessage"/> that can be passed to
     /// <see cref="Process(PipelineMessage)"/>.</returns>

sdk/core/System.ClientModel/tests/Pipeline/ClientPipelineTests.cs

@@ -331,4 +331,51 @@ public async Task RequestOptionsCanCustomizePipeline()
         Assert.AreEqual("Response:RetryPolicy", observations[index++]);
         Assert.AreEqual("Response:A", observations[index++]);
     }
+
+        [Test]
+    public void CreateMessageWithUriMethodAndClassifierSetsProperties()
+    {
+        ClientPipeline pipeline = ClientPipeline.Create();
+        Uri testUri = new Uri("https://example.com/test");
+        string testMethod = "POST";
+        PipelineMessageClassifier testClassifier = PipelineMessageClassifier.Create(ReadOnlySpan<ushort>.Empty);
+
+        PipelineMessage message = pipeline.CreateMessage(testUri, testMethod, testClassifier);
+
+        Assert.IsNotNull(message);
+        Assert.AreEqual(testUri, message.Request.Uri);
+        Assert.AreEqual(testMethod, message.Request.Method);
+        Assert.AreEqual(testClassifier, message.ResponseClassifier);
+        Assert.IsNotNull(message.NetworkTimeout);
+    }
+
+    [Test]
+    public void CreateMessageThrowsOnNullUriOrMethod()
+    {
+        ClientPipeline pipeline = ClientPipeline.Create();
+        Uri testUri = new Uri("https://example.com/test");
+        string testMethod = "GET";
+        PipelineMessageClassifier testClassifier = PipelineMessageClassifier.Default;
+
+        Assert.Throws<ArgumentNullException>(() =>
+            pipeline.CreateMessage(null!, testMethod, testClassifier));
+
+        Assert.Throws<ArgumentNullException>(() =>
+            pipeline.CreateMessage(testUri, null!, testClassifier));
+    }
+
+    [Test]
+    public void CreateMessageWithUriAndMethodUsesDefaultClassifier()
+    {
+        ClientPipeline pipeline = ClientPipeline.Create();
+        Uri testUri = new Uri("https://example.com/test");
+        string testMethod = "DELETE";
+
+        PipelineMessage message = pipeline.CreateMessage(testUri, testMethod);
+
+        Assert.IsNotNull(message);
+        Assert.AreEqual(testUri, message.Request.Uri);
+        Assert.AreEqual(testMethod, message.Request.Method);
+        Assert.AreEqual(PipelineMessageClassifier.Default, message.ResponseClassifier);
+    }
 }