Fix: Remove errors and warning raised by pylint 2.17.0

.pylintrc

@@ -364,13 +364,6 @@ max-line-length=80
 # Maximum number of lines in a module
 max-module-lines=1000
 
-# List of optional constructs for which whitespace checking is disabled. `dict-
-# separator` is used to allow tabulation in dicts, etc.: {1  : 1,\n222: 2}.
-# `trailing-comma` allows a space between comma and closing bracket: (a, ).
-# `empty-line` allows space-only lines.
-no-space-check=trailing-comma,
-               dict-separator
-
 # Allow the body of a class to be on the same line as the declaration if body
 # contains single statement.
 single-line-class-stmt=no
@@ -472,4 +465,4 @@ valid-metaclass-classmethod-first-arg=mcs
 
 # Exceptions that will emit a warning when being caught. Defaults to
 # "Exception"
-overgeneral-exceptions=Exception
+overgeneral-exceptions=builtins.Exception

scripts/.pylintrc

@@ -362,13 +362,6 @@ max-line-length=80
 # Maximum number of lines in a module
 max-module-lines=1000
 
-# List of optional constructs for which whitespace checking is disabled. `dict-
-# separator` is used to allow tabulation in dicts, etc.: {1  : 1,\n222: 2}.
-# `trailing-comma` allows a space between comma and closing bracket: (a, ).
-# `empty-line` allows space-only lines.
-no-space-check=trailing-comma,
-               dict-separator
-
 # Allow the body of a class to be on the same line as the declaration if body
 # contains single statement.
 single-line-class-stmt=no
@@ -470,4 +463,4 @@ valid-metaclass-classmethod-first-arg=mcs
 
 # Exceptions that will emit a warning when being caught. Defaults to
 # "Exception"
-overgeneral-exceptions=Exception
+overgeneral-exceptions=builtins.Exception

scripts/check-gmp.gmp.py

@@ -185,7 +185,7 @@ def is_old_report(self, last_scan_end, params_used):
 
         # Retrieve the scan_end value
         self.cursor.execute(
-            "SELECT scan_end, params_used FROM Report WHERE" " host=?",
+            "SELECT scan_end, params_used FROM Report WHERE host=?",
             (self.host,),
         )
         db_entry = self.cursor.fetchone()
@@ -202,7 +202,7 @@ def is_old_report(self, last_scan_end, params_used):
             new = parse_date(last_scan_end)
 
             logger.debug(
-                "Old time (from db): %s\n" "New time (from rp): %s", old, new
+                "Old time (from db): %s\nNew time (from rp): %s", old, new
             )
 
             if new <= old and params_used == db_entry[1]:

scripts/monthly-report.gmp.py

@@ -31,7 +31,7 @@ def check_args(args: Namespace) -> None:
         message = """
         This script will display all vulnerabilities from the hosts of the
         reports in a given month!
-        
+
         1. <month>  -- month of the monthly report
         2. <year>   -- year of the monthly report
 
@@ -97,7 +97,7 @@ def print_result_tables(gmp: Gmp, reports_xml: Element) -> None:
 
         for host in res.xpath("report/report/host"):
             hostname = host.xpath(
-                'detail/name[text()="hostname"]/../' "value/text()"
+                'detail/name[text()="hostname"]/../value/text()'
             )
             if len(hostname) > 0:
                 hostname = str(hostname[0])

scripts/send-delta-emails.gmp.py

@@ -91,7 +91,7 @@ def execute_send_delta_emails(sc: sched.scheduler, **kwargs: dict) -> None:
             continue
 
         if reports[0].xpath(
-            "report/user_tags/tag/" 'name[text()="delta_alert_sent"]'
+            'report/user_tags/tag/name[text()="delta_alert_sent"]'
         ):
             print("  Delta report for latest finished report already sent")
             continue

tests/test_parser.py

@@ -112,7 +112,7 @@ def resolve_raises_error():
     @patch("gvmtools.parser.Config")
     def test_config_load_raises_error(self, config_mock, path_mock):
         def config_load_error():
-            raise Exception
+            raise Exception  # pylint: disable=broad-exception-raised
 
         config = unittest.mock.MagicMock()
         config.load = unittest.mock.MagicMock(side_effect=config_load_error)