added lint configuration

Author: dmy.berezovskyi

.idea/misc.xml

@@ -0,0 +1,44 @@
+# Ruff Configuration
+line-length = 88
+indent-width = 4
+output-format = "grouped"
+respect-gitignore = true
+target-version = "py312"
+
+[lint]
+select = [
+    "E",  # Error codes
+    "F",  # Pyflakes
+    "W",  # Warnings
+    "C",  # Complexity
+    "N",  # Naming conventions
+    "I",  # Ignored errors
+    "E501",  # Line too long
+    "N805",  # First argument should be 'self'
+]
+fixable = ["ALL"]  # All fixable errors will be automatically corrected
+
+# Ignored Errors
+ignore = [
+    "E731",  # Do not assign a lambda expression, use a def instead
+    "N801",  # Function name should be lowercase
+    "I001",  # Import convention violation
+    "F631",  # Assert should not be used with a literal
+]
+
+# Regular expression for dummy variables
+dummy-variable-rgx = "^(_+|(_+[a-zA-Z0-9_]*[a-zA-Z0-9]+?))$"
+
+# Excluded Directories and Files
+exclude = [
+    ".pytest_cache",
+    "poetry.lock",
+    "__pypackages__",
+    ".git",
+    ".env",
+    ".ruff_cache"
+]
+
+[format]
+exclude = ["__init__.py"]  # Exclude __init__.py from formatting
+skip-magic-trailing-comma = true  # Skip the trailing comma for magic trailing commas

.idea/modules.xml

@@ -1,7 +1,6 @@
 import pytest
 from core import CustomEventListener
 from dotenv import load_dotenv
-from selenium import webdriver
 from selenium.webdriver.support.event_firing_webdriver import EventFiringWebDriver
 
 from src.utils.driver_factory import WebDriverFactory

.idea/simple-python-selenium-framework.iml

@@ -1,6 +1,6 @@
 [pytest]
-#run tests in a row local
-#addopts = -rA -v --html=test_results.html --browser-type=local --env=stag --driver=mac
+#run tests in a row locallocal
+#addopts = -rA -v --html=test_results.html --browser-type= --env=stag --driver=mac
 #run tests parallel with xdist
 #addopts = -rA -v --html=test_results.html --browser-type=firefox --env=stag -n 2
 #run tests in CI

.idea/vcs.xml

@@ -12,4 +12,5 @@ pytest-xdist~=3.3.1
 cryptography~=43.0.1
 beautifulsoup4==4.12.2
 requests~=2.31.0
-setuptools~=68.2.2
+setuptools~=68.2.2
+ruff~=0.6.8

.ruff.toml

@@ -8,13 +8,15 @@
 
 
 class ChromePageScraper:
-    URL_LATEST = 'https://googlechromelabs.github.io/chrome-for-testing/#stable'
-    URL_ALL = "https://googlechromelabs.github.io/chrome-for-testing/latest-versions-per-milestone-with-downloads.json"
+    URL_LATEST = (
+        "https://googlechromelabs.github.io/chrome-for-testing/#stable"
+    )
+    URL_ALL = "https://googlechromelabs.github.io/chrome-for-testing/latest-versions-per-milestone-with-downloads.json" # noqa
 
     @staticmethod
     def __fetch(url: str) -> requests.Response:
         response = requests.get(url)
-        response.raise_for_status()  # Raises an exception if status code is not 200
+        response.raise_for_status()  # Raises an exception if status code is not 200 # noqa
         return response
 
     @staticmethod
@@ -24,18 +26,20 @@ def parse_latest() -> Dict[str, str]:
         drivers = {}
         page = ChromePageScraper.__fetch(ChromePageScraper.URL_LATEST)
 
-        soup = BeautifulSoup(page.text, 'html.parser')
-        element = soup.select_one('section#stable.status-not-ok div.table-wrapper table tbody tr.status-ok')
+        soup = BeautifulSoup(page.text, "html.parser")
+        element = soup.select_one(
+            "section#stable.status-not-ok div.table-wrapper table tbody tr.status-ok" # noqa
+        )
 
         if not element:
             raise Exception("Element not found in the HTML.")
 
-        code_elements = element.find_all('code')
+        code_elements = element.find_all("code")
 
         for el in code_elements:
             text = el.text.strip()
 
-            if text not in ['200', 'chrome', 'chromedriver']:
+            if text not in ["200", "chrome", "chromedriver"]:
                 elements_list.append(text)
 
         for i in range(0, len(elements_list), 2):
@@ -56,28 +60,37 @@ def get_chromedriver(platform=None, version=None, milestone=None):
 
         :param platform: os_name and architecture
         :param version: your chrome browser version
-        :param milestone: first 3 digit of browser version: 116 or 115
+        :param milestone: first 3 digits of a browser version: 129 or etc
         :return:
         """
         if version is None and milestone is None:
-            raise ValueError(f"You must specify version or milestone: version {version}, milestone {milestone}")
+            raise ValueError(
+                f"You must specify version or milestone: version {version},"
+                f" milestone {milestone}"
+            )
         if platform is None:
             platform = OSChecker.check_os()
 
         # Parse the JSON data
-        parsed_data = json.loads(ChromePageScraper.__fetch(ChromePageScraper.URL_ALL).text)
+        parsed_data = json.loads(
+            ChromePageScraper.__fetch(ChromePageScraper.URL_ALL).text
+        )
         milestones_data = parsed_data["milestones"]
 
         for milestone_key, milestone_data in milestones_data.items():
-            if (
-                    (milestone is None or milestone_key == milestone)
-                    and (version is None or milestone_data["version"] == version)
+            if (milestone is None or milestone_key == milestone) and (
+                version is None or milestone_data["version"] == version
             ):
                 if "chromedriver" in milestone_data["downloads"]:
-                    for chromedriver_info in milestone_data["downloads"]["chromedriver"]:
-                        if platform is None or chromedriver_info["platform"] == platform:
+                    for chromedriver_info in milestone_data["downloads"][
+                        "chromedriver"
+                    ]:
+                        if (
+                            platform is None
+                            or chromedriver_info["platform"] == platform
+                        ):
                             return chromedriver_info
 
 
-if __name__ == '__main__':
-    print(ChromePageScraper.get_chromedriver(milestone='116'))
+if __name__ == "__main__":
+    print(ChromePageScraper.get_chromedriver(milestone="129"))

conftest.py

@@ -12,22 +12,22 @@ def check_os():
         if os_name == "Darwin":
             os_name = "mac"
         if os_name == "mac":
-            return '-'.join([os_name, arch])
+            return "-".join([os_name, arch])
         else:
-            return ''.join([os_name, arch])
+            return "".join([os_name, arch])
 
     def print_os_info(self):
         os_name, arch = self.check_os()
         print(f"Operating System: {os_name}, arch={arch}")
 
-    def _get_driver_type(dr_type=None):
+    def _get_driver_type(self=None):
         os_arch_mapping = {
             "arm64": "chrome_arm64",
             "x86_64": "chrome_x86_64",
             "ubuntu": "ubuntu",  # Add additional mappings as needed
         }
 
-        default_driver_type = dr_type
+        default_driver_type = self
         os_arch = platform.machine()
         os_name = platform.system().lower()