fix formatting issues

Author: Ndiritu

src/msgraph_core/graph_client_factory.py

@@ -91,7 +91,9 @@ def _get_telemetry_handler(
         options"""
 
         if options:
-            graph_telemetry_options: GraphTelemetryHandlerOption = options.get(GraphTelemetryHandlerOption().get_key()) # type: ignore # Unable to down cast type
+            graph_telemetry_options: GraphTelemetryHandlerOption = options.get(
+                GraphTelemetryHandlerOption().get_key()
+            )  # type: ignore # Unable to down cast type
             if graph_telemetry_options:
                 return GraphTelemetryHandler(options=graph_telemetry_options)
         return GraphTelemetryHandler()

src/msgraph_core/models/page_result.py

@@ -46,8 +46,16 @@ def get_field_deserializers(self) -> Dict[str, Callable[[ParseNode], None]]:
             object where each entry is a property key with its deserialization callback.
         """
         return {
-            "@odata.nextLink": lambda x: setattr(self, "odata_next_link", x.get_str_value()),
-            "value": lambda x: setattr(self, "value", x.get_collection_of_object_values(Parsable)) # type: ignore # Bug. Should get a collection of primitive dictionary objects
+            "@odata.nextLink":
+            lambda x: setattr(self, "odata_next_link", x.get_str_value()),
+            "value":
+            lambda x: setattr(
+                self,
+                "value",
+                x.get_collection_of_object_values(
+                    Parsable  # type: ignore # Bug. Should get a collection of primitive dictionary objects
+                )
+            )
         }
 
     def serialize(self, writer: SerializationWriter) -> None:

src/msgraph_core/requests/batch_request_content.py

@@ -67,10 +67,14 @@ def add_request(self, request_id: Optional[str], request: BatchRequestItem) -> N
         if hasattr(request, 'depends_on') and request.depends_on:
             for dependent_id in request.depends_on:
                 if not self._request_by_id(dependent_id):
-                    raise ValueError(f"Request depends on request id: {dependent_id} which was not found in requests. Add request id: {dependent_id} first")
+                    raise ValueError(
+                        f"Request depends on request id: {dependent_id} which was not found in requests. Add request id: {dependent_id} first"
+                    )
         self._requests[request.id] = request
 
-    def add_request_information(self, request_information: RequestInformation, request_id: Optional[str] = None) -> None:
+    def add_request_information(
+        self, request_information: RequestInformation, request_id: Optional[str] = None
+    ) -> None:
         """
         Adds a request to the batch request content.
         Args:

src/msgraph_core/requests/batch_request_content_collection.py

@@ -53,9 +53,14 @@ def new_batch_with_failed_requests(self) -> Optional[BatchRequestContent]:
         batch_with_failed_responses: Optional[BatchRequestContent] = BatchRequestContent()
         for batch in self.batches:
             for request in batch.requests:
-                if request.status_code not in [200, 201, 202, 203, 204, 205, 206, 207, 208, 226]: # type: ignore # Method should be deprecated
+                if request.status_code not in [ # type: ignore # Method should be deprecated
+                    200, 201, 202, 203, 204, 205, 206, 207, 208, 226
+                ]:
                     if batch_with_failed_responses is not None:
-                        batch_with_failed_responses.add_request(request.id, request) # type: ignore # Bug. Method should be deprecated
+                        batch_with_failed_responses.add_request(
+                            request.id,  # type: ignore # Bug. Method should be deprecated
+                            request  # type: ignore
+                        )
                     else:
                         raise ValueError("batch_with_failed_responses is None")
         return batch_with_failed_responses

src/msgraph_core/requests/batch_request_item.py

@@ -78,8 +78,12 @@ def create_with_urllib_request(
         request_info.headers = RequestHeaders()
         for key, value in request.headers.items():
             request_info.headers.try_add(header_name=key, header_value=value)
-        request_info.content = request.data # type: ignore
-        return BatchRequestItem(request_info, id, depends_on) # type: ignore # union types not analysed correctly
+        request_info.content = request.data  # type: ignore
+        return BatchRequestItem(
+            request_info,
+            id,
+            depends_on  # type: ignore # union types not analysed correctly
+        )
 
     def set_depends_on(self, requests: Optional[List[Union[str, 'BatchRequestItem']]]) -> None:
         """
@@ -254,7 +258,10 @@ def serialize(self, writer: SerializationWriter) -> None:
         writer.write_str_value('method', self.method)
         writer.write_str_value('url', self.url)
         writer.write_collection_of_primitive_values('depends_on', self._depends_on)
-        writer.write_collection_of_object_values('headers', self._headers) # type: ignore # need method to serialize dicts
+        writer.write_collection_of_object_values(
+            'headers',
+            self._headers  # type: ignore # need method to serialize dicts
+        )
         if self._body:
             json_object = json.loads(self._body)
             is_json_string = json_object and isinstance(json_object, dict)

src/msgraph_core/requests/batch_response_content.py

@@ -149,11 +149,7 @@ def set_responses(n: ParseNode):
             else:
                 setattr(self, '_responses', {})
 
-
-        return {
-            'responses':
-            lambda n: set_responses(n)
-        }
+        return {'responses': lambda n: set_responses(n)}
 
     def serialize(self, writer: SerializationWriter) -> None:
         """

src/msgraph_core/requests/batch_response_content_collection.py

@@ -66,10 +66,8 @@ def get_field_deserializers(self) -> Dict[str, Callable[[ParseNode], None]]:
         """
         return {
             'responses':
-            lambda n: setattr(
-                self, "_responses",
-                n.get_collection_of_object_values(BatchResponseItem)
-            )
+            lambda n:
+            setattr(self, "_responses", n.get_collection_of_object_values(BatchResponseItem))
         }
 
     def serialize(self, writer: SerializationWriter) -> None:

src/msgraph_core/requests/batch_response_item.py

@@ -142,7 +142,11 @@ def get_field_deserializers(self) -> Dict[str, Callable[[ParseNode], None]]:
         return {
             "id": lambda x: setattr(self, "id", x.get_str_value()),
             "status": lambda x: setattr(self, "status", x.get_int_value()),
-            "headers": lambda x: setattr(self, "headers", x.try_get_anything(x._json_node)), # type: ignore # need interface to return a dictionary
+            "headers": lambda x: setattr(
+                self,
+                "headers",
+                x.try_get_anything(x._json_node)  # type: ignore
+            ),  # need interface to return a dictionary
             "body": lambda x: setattr(self, "body", x.get_bytes_value()),
         }
 
@@ -153,7 +157,10 @@ def serialize(self, writer: SerializationWriter) -> None:
         writer.write_str_value('id', self._id)
         writer.write_str_value('atomicity_group', self._atomicity_group)
         writer.write_int_value('status', self._status)
-        writer.write_collection_of_primitive_values('headers', self._headers) # type: ignore # need method to serialize dicts
+        writer.write_collection_of_primitive_values(
+            'headers',
+            self._headers  # type: ignore
+        )  # need method to serialize dicts
         if self._body:
             writer.write_bytes_value('body', self._body.getvalue())
         else:

src/msgraph_core/tasks/large_file_upload.py

@@ -16,6 +16,7 @@
 
 T = TypeVar('T', bound=Parsable)
 
+
 # pylint: disable=too-many-instance-attributes
 class LargeFileUploadTask:
 
@@ -145,7 +146,9 @@ def next_range(self):
     def next_range(self, value: Optional[str]) -> None:
         self._next_range = value
 
-    async def next_chunk(self, file: BytesIO, range_start: int = 0, range_end: int = 0) -> LargeFileUploadSession:
+    async def next_chunk(
+        self, file: BytesIO, range_start: int = 0, range_end: int = 0
+    ) -> LargeFileUploadSession:
         upload_url = self.get_validated_upload_url(self.upload_session)
         if not upload_url:
             raise ValueError('The upload session URL must not be empty.')

src/msgraph_core/tasks/page_iterator.py

@@ -148,7 +148,9 @@ async def next(self) -> Optional[PageResult]:
         if self.current_page is not None and not self.current_page.odata_next_link:
             return None
         response = await self.fetch_next_page()
-        next_link = response.odata_next_link if response and hasattr(response, 'odata_next_link') else None
+        next_link = response.odata_next_link if response and hasattr(
+            response, 'odata_next_link'
+        ) else None
         value = response.value if response and hasattr(response, 'value') else None
         return PageResult(next_link, value)
 
@@ -206,7 +208,9 @@ async def fetch_next_page(self) -> Optional[Union[T, PageResult]]:
         if self.request_options:
             request_info.add_request_options(*self.request_options)
         return await self.request_adapter.send_async(
-            request_info, self.parsable_factory, self.error_mapping # type: ignore
+            request_info,
+            self.parsable_factory,  # type: ignore
+            self.error_mapping
         )
 
     def enumerate(self, callback: Optional[Callable] = None) -> bool: