Don't log to the root logger

Author: s0undt3ch

pytest_reportportal/plugin.py

@@ -16,6 +16,8 @@
 except ImportError:
     PYTEST_HAS_LOGGING_PLUGIN = False
 
+log = logging.getLogger(__name__)
+
 
 def is_master(config):
     """
@@ -109,7 +111,7 @@ def pytest_unconfigure(config):
         reporter = config._reporter
         del config._reporter
         config.pluginmanager.unregister(reporter)
-        logging.debug('RP is unconfigured')
+        log.debug('RP is unconfigured')
 
 
 def pytest_addoption(parser):

pytest_reportportal/rp_logging.py

@@ -73,6 +73,9 @@ def filter(self, record):
             # Don't send reportportal_client logs.
             # Specially because we'll hit a max recursion issue
             return False
+        if record.name.startswith('pytest_reportportal'):
+            # Don't send pytest_reportportal logs.
+            return False
         return True
 
     def emit(self, record):

pytest_reportportal/service.py

@@ -9,6 +9,8 @@
 
 from reportportal_client import ReportPortalServiceAsync
 
+log = logging.getLogger(__name__)
+
 
 def timestamp():
     return str(int(time() * 1000))
@@ -39,8 +41,8 @@ def init_service(self, endpoint, project, uuid, log_batch_size,
         if self.RP is None:
             self.ignore_errors = ignore_errors
             self.ignored_tags = ignored_tags
-            logging.debug('ReportPortal - Init service: endpoint=%s, '
-                          'project=%s, uuid=%s', endpoint, project, uuid)
+            log.debug('ReportPortal - Init service: endpoint=%s, '
+                      'project=%s, uuid=%s', endpoint, project, uuid)
             self.RP = ReportPortalServiceAsync(
                 endpoint=endpoint,
                 project=project,
@@ -49,7 +51,7 @@ def init_service(self, endpoint, project, uuid, log_batch_size,
                 log_batch_size=log_batch_size
             )
         else:
-            logging.debug('The pytest is already initialized')
+            log.debug('The pytest is already initialized')
         return self.RP
 
     def async_error_handler(self, exc_info):
@@ -84,11 +86,9 @@ def start_launch(
             'mode': mode,
             'tags': tags
         }
-        logging.debug('ReportPortal - Start launch: '
-                      'request_body=%s', sl_pt)
+        log.debug('ReportPortal - Start launch: equest_body=%s', sl_pt)
         req_data = self.RP.start_launch(**sl_pt)
-        logging.debug('ReportPortal - Launch started: '
-                      'response_body=%s', req_data)
+        log.debug('ReportPortal - Launch started: response_body=%s', req_data)
 
     def start_pytest_item(self, test_item=None):
         self._stop_if_necessary()
@@ -103,8 +103,7 @@ def start_pytest_item(self, test_item=None):
             'item_type': 'STEP'
         }
 
-        logging.debug('ReportPortal - Start TestItem: '
-                      'request_body=%s', start_rq)
+        log.debug('ReportPortal - Start TestItem: request_body=%s', start_rq)
         self.RP.start_test_item(**start_rq)
 
     def _get_tags(self, item):
@@ -124,8 +123,7 @@ def finish_pytest_item(self, status, issue=None):
             'issue': issue
         }
 
-        logging.debug('ReportPortal - Finish TestItem: '
-                      'request_body=%s', fta_rq)
+        log.debug('ReportPortal - Finish TestItem: request_body=%s', fta_rq)
         self.RP.finish_test_item(**fta_rq)
 
     def finish_launch(self, launch=None, status='rp_launch'):
@@ -138,7 +136,7 @@ def finish_launch(self, launch=None, status='rp_launch'):
             'end_time': timestamp(),
             'status': status
         }
-        logging.debug('ReportPortal - Finish launch: request_body=%s', fl_rq)
+        log.debug('ReportPortal - Finish launch: request_body=%s', fl_rq)
         self.RP.finish_launch(**fl_rq)
 
     def post_log(self, message, loglevel='INFO', attachment=None):
@@ -147,8 +145,8 @@ def post_log(self, message, loglevel='INFO', attachment=None):
             return
 
         if loglevel not in self._loglevels:
-            logging.warning('Incorrect loglevel = %s. Force set to INFO. '
-                            'Available levels: %s.', loglevel, self._loglevels)
+            log.warning('Incorrect loglevel = %s. Force set to INFO. '
+                        'Available levels: %s.', loglevel, self._loglevels)
             loglevel = 'INFO'
 
         sl_rq = {