Update azure-autorust to append path and parameters via URL lib (#1663)

* Update azure-autorust to append path and parameters via URL lib

When returning a URL from Client::endpoint the URL contains a tailing slash
in any case. This is handled by the URL lib which will append the tailing
slash to the end upon calling Display::fmt. Therefore formatting the returned
URL from Client::endpoint will result in a duplicated slash, which could cause
some (not all) Azure endpoints to have some trouble.

For example:
Base URL: https://some-app-conf.azconfig.io/
AppConfigUrl: https://some-app-conf.azconfig.io//keys?api-version=2023-10-01
This resulted in a 404 response.

Removing the tailing slash from the URL manually does not solve the problem,
due to the reason mentioned above.

This patch applies a fix to append (set) the formatted path via rust URL lib,
which will handle the duplicated slash (or any other problem).

* Fix Code Style issue

* Fix Code Style issues for doc-comments

---------

Co-authored-by: Heath Stewart <[email protected]>

Author: andras-pinter

services/autorust/codegen/src/codegen_operations/request_builder_send.rs

@@ -38,8 +38,7 @@ impl ToTokens for RequestBuilderSendCode {
             quote! { &self.#url_arg }
         });
         let url_str_args = quote! { #(#url_args),* };
-
-        let fpath = format!("{{}}{}", &format_path(&new_request_code.path));
+        let fpath = PARAM_RE.replace_all(&new_request_code.path, "{}");
 
         let mut match_status = TokenStream::new();
         for status_response in &self.response_code.status_responses {
@@ -58,7 +57,8 @@ impl ToTokens for RequestBuilderSendCode {
             let api_version = &self.request_builder.api_version;
             quote! {
                 fn url(&self) -> azure_core::Result<azure_core::Url> {
-                    let mut url = azure_core::Url::parse(&format!(#fpath, self.client.endpoint(), #url_str_args))?;
+                    let mut url = self.client.endpoint().clone();
+                    url.set_path(&format!(#fpath, #url_str_args));
 
                     let has_api_version_already = url.query_pairs().any(|(k, _)| k == azure_core::query_param::API_VERSION);
                     if !has_api_version_already {
@@ -70,7 +70,9 @@ impl ToTokens for RequestBuilderSendCode {
         } else {
             quote! {
                 fn url(&self) -> azure_core::Result<azure_core::Url> {
-                    let url = azure_core::Url::parse(&format!(#fpath, self.client.endpoint(), #url_str_args))?;
+                    let mut url = self.client.endpoint().clone();
+                    url.set_path(&format!(#fpath, #url_str_args));
+
                     Ok(url)
                 }
             }
@@ -211,10 +213,6 @@ impl ToTokens for RequestBuilderSendCode {
     }
 }
 
-fn format_path(path: &str) -> String {
-    PARAM_RE.replace_all(path, "{}").to_string()
-}
-
 fn get_continuable_param(next_link_name: &str, request_builder: &SetRequestCode) -> Option<String> {
     let next_link_name = next_link_name.to_snake_case();
     let link_name = next_link_name.strip_prefix("next_");