fix: GraphClientFactory baseUrl setting logic

Updates the GraphClientFactory so that the baseUrl of the httpClient is only set when creating a new httpClient.
This ensures that any client which has previously set a baseUrl and is adding middleware to the client will not have the existing baseUrl set

Author: gavinbarron

src/msgraph_core/graph_client_factory.py

@@ -16,7 +16,7 @@
 from .middleware.options import GraphTelemetryHandlerOption
 
 
-class GraphClientFactory(KiotaClientFactory):
+class GraphClientFactory():
     """Constructs httpx.AsyncClient instances configured with either custom or default
     pipeline of graph specific middleware.
     """
@@ -35,10 +35,13 @@ def create_with_default_middleware( # type: ignore
         Args:
             api_version (APIVersion): The Graph API version to be used.
             Defaults to APIVersion.v1.
-            client  (httpx.AsyncClient): The httpx.AsyncClient instance to be used.
-            Defaults to KiotaClientFactory.get_default_client().
+            This is only used if the client parameter is None.
+            client (Optional[httpx.AsyncClient]]): The httpx.AsyncClient instance to be used.
+            Defaults to None.
+            When None, a default client will be created with the base url set to https://{host}/{api_version}.
             host (NationalClouds): The national clound endpoint to be used.
             Defaults to NationalClouds.Global.
+            This is only used if the client parameter is None.
             options (Optional[dict[str, RequestOption]]): The request options to use when
             instantiating default middleware. Defaults to dict[str, RequestOption]=None.
 
@@ -47,7 +50,7 @@ def create_with_default_middleware( # type: ignore
         """
         if client is None:
             client = KiotaClientFactory.get_default_client()
-        client.base_url = GraphClientFactory._get_base_url(host, api_version)  # type: ignore
+            client.base_url = GraphClientFactory._get_base_url(host, api_version)  # type: ignore
         middleware = KiotaClientFactory.get_default_middleware(options)
         telemetry_handler = GraphClientFactory._get_telemetry_handler(options)
         middleware.append(telemetry_handler)
@@ -64,19 +67,23 @@ def create_with_custom_middleware( # type: ignore
         """Applies a custom middleware chain to the HTTP Client
 
         Args:
-            middleware(list[BaseMiddleware]): Custom middleware list that will be used to create
+            middleware(Optional[list[BaseMiddleware]]): Custom middleware list that will be used to create
             a middleware pipeline. The middleware should be arranged in the order in which they will
             modify the request.
-            api_version (APIVersion): The Graph API version to be used.
+            Defaults to None,
+            api_version (APIVersion): The Graph API version to be used. 
             Defaults to APIVersion.v1.
-            client  (httpx.AsyncClient): The httpx.AsyncClient instance to be used.
-            Defaults to KiotaClientFactory.get_default_client().
-            host (NationalClouds): The national clound endpoint to be used.
+            This is only used if the client parameter is None.
+            client (Optional[httpx.AsyncClient]): The httpx.AsyncClient instance to be used.
+            Defaults to None.
+            When None, a default client will be created with the base url set to https://{host}/{api_version}.
+            host (NationalClouds): The national cloud endpoint to be used.
             Defaults to NationalClouds.Global.
+            This is only used if the client parameter is None.
         """
         if client is None:
             client = KiotaClientFactory.get_default_client()
-        client.base_url = GraphClientFactory._get_base_url(host, api_version)  # type: ignore
+            client.base_url = GraphClientFactory._get_base_url(host, api_version)  # type: ignore
         return GraphClientFactory._load_middleware_to_client(client, middleware)
 
     @staticmethod