Merge pull request #211 from reportportal/develop

Release

Author: HardNorth

CHANGELOG.md

@@ -1,6 +1,10 @@
 # Changelog
 
 ## [Unreleased]
+### Removed
+- Logging in the `listener.__post_log_message` method, to avoid logging the same message recursively, by @HardNorth
+
+## [5.6.2]
 ### Fixed
 - SETUP / TEARDOWN keyword removing, by @HardNorth
 

robotframework_reportportal/listener.py

@@ -199,10 +199,6 @@ def __post_log_message(self, message: LogMessage) -> None:
 
         :param message: Internal message object to send
         """
-        if message.attachment:
-            logger.debug(f"ReportPortal - Log Message with Attachment: {message}")
-        else:
-            logger.debug(f"ReportPortal - Log Message: {message}")
         self.service.log(message=message)
 
     def __post_skipped_keyword(self, kwd: Keyword, clean_data_remove: bool) -> None:

setup.py

@@ -18,7 +18,7 @@
 
 from setuptools import setup
 
-__version__ = "5.6.2"
+__version__ = "5.6.3"
 
 
 def read_file(fname):

tests/integration/test_logger.py

@@ -34,7 +34,7 @@ def test_launch_log(mock_client_init):
 
 @mock.patch(REPORT_PORTAL_SERVICE)
 def test_binary_file_log(mock_client_init):
-    if platform.system() == "Linux" and platform.release() == "6.8.0-1017-azure":
+    if platform.system() == "Linux" and platform.release().endswith("-azure"):
         # GitHub Actions Linux runner has an issue with binary data reading
         data_type = "application/octet-stream"
     else: