Skip to content

Commit 61db3ee

Browse files
committed
Use early returns
1 parent f190e83 commit 61db3ee

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

msgraphcore/graph_client.py

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -87,19 +87,16 @@ def _graph_url(self, url: str) -> str:
8787
def get_graph_session(**kwargs):
8888
"""Method to always return a single instance of a HTTP Client"""
8989

90-
session = None
91-
9290
credential = kwargs.get('credential')
9391
middleware = kwargs.get('middleware')
9492

9593
if credential and middleware:
96-
raise Exception(
94+
return Exception(
9795
"Invalid parameters! Both TokenCredential and middleware cannot be passed"
9896
)
9997
if not credential and not middleware:
100-
raise ValueError("Invalid parameters!. Missing TokenCredential or middleware")
98+
return ValueError("Invalid parameters!. Missing TokenCredential or middleware")
99+
101100
if credential:
102-
session = HTTPClientFactory(**kwargs).create_with_default_middleware(credential)
103-
elif middleware:
104-
session = HTTPClientFactory(**kwargs).create_with_custom_middleware(middleware)
105-
return session
101+
return HTTPClientFactory(**kwargs).create_with_default_middleware(credential)
102+
return HTTPClientFactory(**kwargs).create_with_custom_middleware(middleware)

0 commit comments

Comments
 (0)