Apply black formatting

Author: guillaume-dequenne

src/py_sonar_scanner/configuration.py

@@ -32,13 +32,13 @@ class Configuration:
     scan_arguments: list[str]
 
     def __init__(self):
-        self.sonar_scanner_path = '.scanner'
-        self.sonar_scanner_version = '4.6.2.2472'
-        self.sonar_scanner_executable_path = ''
+        self.sonar_scanner_path = ".scanner"
+        self.sonar_scanner_version = "4.6.2.2472"
+        self.sonar_scanner_executable_path = ""
         self.scan_arguments = []
 
     def setup(self):
-        """ This is executed when run from the command line """
+        """This is executed when run from the command line"""
         parser = argparse.ArgumentParser()
 
         # Required positional argument
@@ -51,12 +51,7 @@ def setup(self):
         parser.add_argument("-n", "--name", action="store", dest="name")
 
         # Optional verbosity counter (eg. -v, -vv, -vvv, etc.)
-        parser.add_argument(
-            "-v",
-            "--verbose",
-            action="count",
-            default=0,
-            help="Verbosity (-v, -vv, etc)")
+        parser.add_argument("-v", "--verbose", action="count", default=0, help="Verbosity (-v, -vv, etc)")
 
         # Specify output of "--version"
         # parser.add_argument(
@@ -82,14 +77,14 @@ def _read_jvm_args(self) -> list[str]:
     def _read_toml_args(self) -> list[str]:
         scan_arguments: list[str] = []
         try:
-            if os.path.isfile('pyproject.toml'):
-                with open('pyproject.toml', 'r') as file:
+            if os.path.isfile("pyproject.toml"):
+                with open("pyproject.toml", "r") as file:
                     # TODO: actually search for pyproject.toml
                     toml_data = file.read()
                     parsed_data = toml.loads(toml_data)
                     print(parsed_data)
-                    if 'sonar' in parsed_data:
-                        sonar_properties = parsed_data['sonar']
+                    if "sonar" in parsed_data:
+                        sonar_properties = parsed_data["sonar"]
                         for key, value in sonar_properties.items():
                             self._add_parameter_to_scanner_args(scan_arguments, key, value)
         except BaseException as e:

src/py_sonar_scanner/logger.py

@@ -21,16 +21,16 @@
 from logging import Logger
 from typing import Optional
 
-class ApplicationLogger():
 
+class ApplicationLogger:
     _log: Optional[Logger] = None
 
     @classmethod
     def get_logger(cls) -> Logger:
         if not cls._log:
             cls._log = logging.getLogger("main")
             cls._setup_logger(cls._log)
-        return cls._log 
+        return cls._log
 
     @staticmethod
     def _setup_logger(log: Logger):

src/py_sonar_scanner/utils/binaries_utils.py

@@ -19,10 +19,12 @@
 #
 import zipfile
 
+
 def write_binaries(scanner_res: bytes, destination: str):
     with open(destination, "wb") as output:
         output.write(scanner_res.read())
 
+
 def unzip_binaries(scanner_zip_path: str, destination: str):
     with zipfile.ZipFile(scanner_zip_path, "r") as zip_ref:
         zip_ref.extractall(destination)

tests/test_environment.py

@@ -25,7 +25,6 @@
 
 
 class TestEnvironment(unittest.TestCase):
-
     @patch("py_sonar_scanner.environment.write_binaries")
     @patch("py_sonar_scanner.environment.urllib.request.urlopen")
     def test_download_scanner(self, mock_urlopen, mock_write_binaries):
@@ -79,22 +78,18 @@ def test_install_scanner(self, mock_os, mock_unzip_binaries):
         environment._install_scanner(system_name)
 
         mock_os.mkdir.assert_called_once_with(scanner_path)
-        environment._download_scanner_binaries.assert_called_once_with(
-            scanner_path, scanner_version, system_name
-        )
+        environment._download_scanner_binaries.assert_called_once_with(scanner_path, scanner_version, system_name)
         mock_unzip_binaries.assert_called_once_with(download_destination, scanner_path)
 
         mock_os.remove.assert_called_once_with(download_destination)
-        environment._change_permissions_recursive.assert_called_once_with(
-           scanner_path, 0o777 
-        )
+        environment._change_permissions_recursive.assert_called_once_with(scanner_path, 0o777)
 
     def test_setup_when_scanner_is_on_path(self):
         cfg = Configuration()
         environment = Environment(cfg)
         environment.cleanup = Mock()
         environment._is_sonar_scanner_on_path = Mock(return_value=True)
-        
+
         environment.setup()
 
         environment.cleanup.assert_called_once()
@@ -145,7 +140,7 @@ def test_cleanup_when_scanner_path_does_not_exist(self, mock_shutil, mock_os_pat
         environment = Environment(cfg)
         mock_os_path.exists = Mock(return_value=False)
         mock_shutil.rmtree = Mock()
-        
+
         environment.cleanup()
 
         mock_os_path.exists.assert_called_once_with(scanner_path)
@@ -158,7 +153,7 @@ def test_is_sonar_scanner_on_path(self, mock_shutil):
         cfg.sonar_scanner_path = scanner_path
         environment = Environment(cfg)
         mock_shutil.which = Mock()
-        
+
         environment._is_sonar_scanner_on_path()
 
         mock_shutil.which.assert_called_once_with("sonar-scanner")

tests/test_scanner.py

@@ -25,7 +25,6 @@
 
 
 class TestScanner(unittest.TestCase):
-
     def test_scanner_compute_command(self):
         cfg = Configuration()
         cfg.sonar_scanner_executable_path = "test"