Implicitly return None everywhere

Author: HardNorth

reportportal_client/client.py

@@ -587,7 +587,7 @@ def start_launch(
             http_timeout=self.http_timeout,
         ).make()
         if not response:
-            return
+            return None
 
         if not self._skip_analytics:
             send_event("start_launch", *agent_name_version(attributes))
@@ -638,7 +638,7 @@ def start_test_item(
         """
         if parent_item_id is NOT_FOUND:
             logger.warning("Attempt to start item for non-existent parent item.")
-            return
+            return None
         if parent_item_id:
             url = uri_join(self.base_url_v2, "item", parent_item_id)
         else:
@@ -667,7 +667,7 @@ def start_test_item(
             http_timeout=self.http_timeout,
         ).make()
         if not response:
-            return
+            return None
         item_id = response.id
         if item_id is not NOT_FOUND:
             logger.debug("start_test_item - ID: %s", item_id)
@@ -707,7 +707,7 @@ def finish_test_item(
         """
         if item_id is NOT_FOUND or not item_id:
             logger.warning("Attempt to finish non-existent item")
-            return
+            return None
         url = uri_join(self.base_url_v2, "item", item_id)
         request_payload = ItemFinishRequest(
             end_time,
@@ -725,7 +725,7 @@ def finish_test_item(
             self.session.put, url=url, json=request_payload, verify_ssl=self.verify_ssl, http_timeout=self.http_timeout
         ).make()
         if not response:
-            return
+            return None
         self._remove_current_item()
         logger.debug("finish_test_item - ID: %s", item_id)
         logger.debug("response message: %s", response.message)
@@ -748,7 +748,7 @@ def finish_launch(
         if self.use_own_launch:
             if self.launch_uuid is NOT_FOUND or not self.launch_uuid:
                 logger.warning("Attempt to finish non-existent launch")
-                return
+                return None
             url = uri_join(self.base_url_v2, "launch", self.launch_uuid, "finish")
             request_payload = LaunchFinishRequest(
                 end_time,
@@ -765,7 +765,7 @@ def finish_launch(
                 http_timeout=self.http_timeout,
             ).make()
             if not response:
-                return
+                return None
             logger.debug("finish_launch - ID: %s", self.launch_uuid)
             logger.debug("response message: %s", response.message)
             message = response.message
@@ -794,7 +794,7 @@ def update_test_item(
             self.session.put, url=url, json=data, verify_ssl=self.verify_ssl, http_timeout=self.http_timeout
         ).make()
         if not response:
-            return
+            return None
         logger.debug("update_test_item - Item: %s", item_id)
         return response.message
 
@@ -833,7 +833,7 @@ def log(
         """
         if item_id is NOT_FOUND:
             logger.warning("Attempt to log to non-existent item")
-            return
+            return None
         rp_file = RPFile(**attachment) if attachment else None
         rp_log = RPRequestLog(self.launch_uuid, time, rp_file, item_id, level, message)
         return self._log(self._log_batcher.append(rp_log))
@@ -863,7 +863,7 @@ def get_launch_info(self) -> Optional[dict]:
             self.session.get, url=url, verify_ssl=self.verify_ssl, http_timeout=self.http_timeout
         ).make()
         if not response:
-            return
+            return None
         launch_info = None
         if response.is_success:
             launch_info = response.json
@@ -888,7 +888,7 @@ def get_launch_ui_url(self) -> Optional[str]:
         launch_info = self.get_launch_info()
         ui_id = launch_info.get("id") if launch_info else None
         if not ui_id:
-            return
+            return None
         mode = launch_info.get("mode") if launch_info else None
         if not mode:
             mode = self.mode
@@ -925,7 +925,7 @@ def _remove_current_item(self) -> Optional[str]:
         try:
             return self._item_stack.get()
         except queue.Empty:
-            return
+            return None
 
     def current_item(self) -> Optional[str]:
         """Retrieve the last item reported by the client (based on the internal FILO queue).