refactor of all the framework

Author: jerryfinol17

.github/workflows/main.yml

@@ -46,7 +46,7 @@ jobs:
         run: |
           mkdir -p reports
           pytest test/ --html=reports/report.html --self-contained-html
-      - name: Upload HTML test report
+      - name: Upload HTML tests report
         uses: actions/upload-artifact@v4
         with:
           name: html-report

.gitignore

@@ -0,0 +1,50 @@
+# Virtual Environment
+.venv/
+venv/
+ENV/
+
+# Python cache
+__pycache__/
+*.py[cod]
+*$py.class
+
+# Coverage reports
+.coverage
+coverage.xml
+htmlcov/
+.coverage.*
+coverage/
+
+# Test reports and artifacts
+reports/
+allure-results/
+allure-report/
+screenshots/
+junit/
+pytest_report/
+
+# PyTest
+.pytest_cache/
+
+# IDEs and editors
+.idea/
+.vscode/
+*.swp
+*.swo
+
+# OS generated files
+.DS_Store
+Thumbs.db
+
+# Logs
+*.log
+logs/
+
+# Temporary files
+*.tmp
+*.temp
+
+# Selenium / WebDriver specific
+geckodriver.log
+msedgedriver.log
+chromedriver.log

README.md

@@ -27,7 +27,7 @@ Demonstrates key skills companies seek: stable Selenium automation, beautiful st
 Automated-testing-project/
 ├── config/               # Credentials & locators per page
 ├── pages/                # POM core (login, inventory, cart, checkout)
-├── test/                 # Test suites (login, inventory, cart/E2E)
+├── tests/                 # Test suites (login, inventory, cart/E2E)
 ├── reports/              # Pytest-HTML reports
 ├── allure-results/       # Raw Allure data
 ├── allure-report/        # Generated Allure dashboard
@@ -45,7 +45,7 @@ pip install -r requirements.txt
 
 ### Run Tests:
 ```bash
-pytest test/ -v                               # Full suite
+pytest tests/ -v                               # Full suite
 pytest --alluredir=allure-results -v          # With Allure
 allure serve allure-results/                  # Open interactive report
 ```

config/config_for_cart_page.py

@@ -27,8 +27,8 @@
         "display_name": "Sauce Labs Bolt T-Shirt"
     },
     "red_tshirt": {
-        "add": (By.ID, "add-to-cart-test.allthethings()-t-shirt-(red)"),
-        "remove": (By.ID, "remove-test.allthethings()-t-shirt-(red)"),
+        "add": (By.ID, "add-to-cart-tests.allthethings()-t-shirt-(red)"),
+        "remove": (By.ID, "remove-tests.allthethings()-t-shirt-(red)"),
         "display_name": "Test.allTheThings() T-Shirt (Red)"
     }
 }

config/config_for_inventory_page.py

@@ -27,8 +27,8 @@
         "display_name": "Sauce Labs Bolt T-Shirt"
     },
     "red_tshirt": {
-        "add": (By.ID, "add-to-cart-test.allthethings()-t-shirt-(red)"),
-        "remove": (By.ID, "remove-test.allthethings()-t-shirt-(red)"),
+        "add": (By.ID, "add-to-cart-tests.allthethings()-t-shirt-(red)"),
+        "remove": (By.ID, "remove-tests.allthethings()-t-shirt-(red)"),
         "display_name": "Test.allTheThings() T-Shirt (Red)"
     }
 }

config/locators.py

@@ -0,0 +1,68 @@
+
+from selenium.webdriver.common.by import By
+
+# ==================== LOGIN PAGE ====================
+LOGIN = {
+    "USERNAME_INPUT": (By.ID, "user-name"),
+    "PASSWORD_INPUT": (By.ID, "password"),
+    "LOGIN_BUTTON": (By.ID, "login-button"),
+    "ERROR_MESSAGE": (By.CSS_SELECTOR, "[data-tests='error']"),
+    "SWAG_LABS_LOGO": (By.CLASS_NAME, "login_logo"),
+    "BURGER_MENU_BUTTON": (By.ID, "react-burger-menu-btn"),
+    "LOGOUT_LINK": (By.CSS_SELECTOR, "[data-tests='logout-sidebar-link']"),
+}
+
+# ==================== INVENTORY PAGE ====================
+INVENTORY = {
+    "TITLE": (By.CLASS_NAME, "title"),
+    "PRIMARY_HEADER": (By.CSS_SELECTOR, "[data-tests='primary-header']"),
+    "SHOPPING_CART_LINK": (By.CSS_SELECTOR, "[data-tests='shopping-cart-link']"),
+    "SHOPPING_CART_BADGE": (By.CLASS_NAME, "shopping_cart_badge"),
+    "BURGER_MENU_BUTTON": (By.ID, "react-burger-menu-btn"),
+    "CROSS_BURGER_BUTTON": (By.ID, "react-burger-cross-btn"),
+    "SORT_DROPDOWN": (By.CLASS_NAME, "product_sort_container"),
+    "PRODUCT_ITEM": (By.CLASS_NAME, "inventory_item"),
+    "PRODUCT_NAME": (By.CLASS_NAME, "inventory_item_name"),
+    "PRODUCT_PRICE": (By.CLASS_NAME, "inventory_item_price"),
+    "PRODUCT_DESCRIPTION": (By.CSS_SELECTOR, "[data-tests='inventory-item-desc']"),
+    "ADD_TO_CART_PREFIX": (By.CSS_SELECTOR, "[data-tests^='add-to-cart-']"),
+}
+
+# ==================== CART PAGE ====================
+CART = {
+    "TITLE": (By.CSS_SELECTOR, "[data-tests='title']"),
+    "CART_ITEMS": (By.CLASS_NAME, "cart_item"),
+    "ITEM_NAME": (By.CLASS_NAME, "inventory_item_name"),
+    "CHECKOUT_BUTTON": (By.ID, "checkout"),
+    "CONTINUE_SHOPPING_BTN": (By.CSS_SELECTOR, "[data-tests='continue-shopping']"),
+    "SHOPPING_CART_BADGE": (By.CLASS_NAME, "shopping_cart_badge"),
+    "REMOVE_BTN_PREFIX": (By.CSS_SELECTOR, "[data-tests^='remove-']"),
+    "CART_QUANTITY": (By.CSS_SELECTOR, "[data-tests='item-quantity']"),
+}
+
+# ==================== CHECKOUT PAGE ====================
+CHECKOUT = {
+    # Step One
+    "FIRST_NAME": (By.ID, "first-name"),
+    "LAST_NAME": (By.ID, "last-name"),
+    "ZIP_CODE": (By.ID, "postal-code"),
+    "CONTINUE_BUTTON": (By.ID, "continue"),
+    "CANCEL_BUTTON": (By.CSS_SELECTOR, "[data-tests='cancel']"),
+    "ERROR_MESSAGE": (By.CSS_SELECTOR, "[data-tests='error']"),
+
+    # Step Two (Overview)
+    "PAYMENT_INFO_LABEL": (By.CSS_SELECTOR, "[data-tests='payment-info-label']"),
+    "PAYMENT_INFO_VALUE": (By.CSS_SELECTOR, "[data-tests='payment-info-value']"),
+    "SHIPPING_INFO_LABEL": (By.CSS_SELECTOR, "[data-tests='shipping-info-label']"),
+    "SHIPPING_INFO_VALUE": (By.CSS_SELECTOR, "[data-tests='shipping-info-value']"),
+    "TOTAL_INFO_LABEL": (By.CSS_SELECTOR, "[data-tests='total-info-label']"),
+    "SUBTOTAL_LABEL": (By.CLASS_NAME, "summary_subtotal_label"),
+    "TAX_LABEL": (By.CLASS_NAME, "summary_tax_label"),
+    "TOTAL_LABEL": (By.CLASS_NAME, "summary_total_label"),
+    "FINISH_BUTTON": (By.ID, "finish"),
+
+    # Step Three (Complete)
+    "COMPLETE_HEADER": (By.CSS_SELECTOR, "[data-tests='complete-header']"),
+    "COMPLETE_TEXT": (By.CSS_SELECTOR, "[data-tests='complete-text']"),
+    "BACK_HOME_BUTTON": (By.CSS_SELECTOR, "[data-tests='back-to-products']"),
+}

pages/Base_Page.py

@@ -0,0 +1,69 @@
+from selenium.webdriver.remote.webdriver import WebDriver
+from selenium.webdriver.remote.webelement import WebElement
+from selenium.webdriver.support.ui import WebDriverWait
+from selenium.webdriver.support import expected_conditions as EC
+from selenium.common.exceptions import TimeoutException, NoSuchElementException, ElementNotInteractableException
+import logging
+from typing import Tuple, Optional
+
+class BasePage:
+    def __init__(self, driver: WebDriver, timeout: int = 15):
+        self.driver = driver
+        self.timeout = timeout
+        self.wait = WebDriverWait(self.driver, self.timeout)
+        self.logger = logging.getLogger(self.__class__.__name__)
+
+# ==============Element Finders ======================================
+    def find_element(self, locator: Tuple[str, str]):
+        try:
+            return self.wait.until(EC.presence_of_element_located(locator))
+        except TimeoutException:
+            self.logger.error(f"Element not found within {self.timeout} seconds: {locator}")
+            raise
+    def find_elements(self, locator: Tuple[str, str]):
+        try:
+            return self.wait.until(EC.presence_of_all_elements_located(locator))
+        except TimeoutException:
+            self.logger.error(f"Element not found within {self.timeout} seconds: {locator}")
+            raise
+
+# =============Actions =======================================================
+
+    def click(self, locator: Tuple[str, str]):
+        try:
+            element = self.wait.until(EC.element_to_be_clickable(locator))
+            element.click()
+        except (TimeoutException, ElementNotInteractableException):
+            try:
+                element = self.find_element(locator)
+                self.driver.execute_script("arguments[0].click();", element)
+                self.logger.info(f"Used JavaScript click for locator: {locator}")
+            except Exception as e:
+                self.logger.error(f"Failed to click element {locator}: {e}")
+                raise
+
+    def fill(self, locator: Tuple[str, str], text: str):
+        try:
+            element = self.wait.until(EC.presence_of_element_located(locator))
+            element.clear()
+            element.send_keys(text)
+        except Exception as e:
+            self.logger.error(f"Failed to fill text in {locator}: {e}")
+            raise
+    def get_text(self, locator: Tuple[str, str]) -> str:
+        element = self.find_element(locator)
+        return element.text.strip()
+
+    def is_visible(self, locator: Tuple[str, str], timeout: Optional[int] = None) -> bool:
+        try:
+            wait_time = timeout or self.timeout
+            WebDriverWait(self.driver, wait_time).until(EC.visibility_of_element_located(locator))
+            return True
+        except TimeoutException:
+            return False
+
+    def get_current_url(self) -> str:
+        return self.driver.current_url
+
+    def go_to ( self , url: str ) :
+        self.driver.get (url)

pages/Cart_Page.py

@@ -0,0 +1,66 @@
+from selenium.webdriver.common.by import By
+from selenium.common.exceptions import TimeoutException , NoSuchElementException , ElementNotInteractableException , \
+    StaleElementReferenceException
+from selenium.webdriver.support import expected_conditions as EC
+from pages.Base_Page import BasePage
+from config.locators import CART
+from config.config_for_cart_page import PRODUCTS
+class CartPage(BasePage):
+
+    def get_cart_items ( self ) -> list :
+        try :
+            self.wait.until (EC.presence_of_all_elements_located (CART["CART_ITEMS"]))
+        except TimeoutException :
+            return []
+
+        present_items = []
+        for product_key in PRODUCTS :
+            try :
+                remove_selector = PRODUCTS[product_key]['remove']
+                self.driver.find_element (*remove_selector)
+                present_items.append (PRODUCTS[product_key]['display_name'])
+            except NoSuchElementException :
+                pass
+        return present_items
+
+    def get_cart_badge_count ( self ) -> str :
+        try :
+            self.wait.until (EC.presence_of_element_located (CART["SHOPPING_CART_BADGE"]))
+            return self.driver.find_element (*CART["SHOPPING_CART_BADGE"]).text
+        except TimeoutException :
+            return "0"
+
+    def is_checkout_button_visible ( self ) -> bool :
+        return self.is_visible (CART["CHECKOUT_BUTTON"])
+
+    def remove_item ( self , product_key: str ) -> bool :
+        if PRODUCTS[product_key]['display_name'] not in self.get_cart_items () :
+            self.logger.warning (f"Product '{product_key}' not found in cart.")
+            return False
+
+        remove_selector = PRODUCTS[product_key]['remove']
+
+        try :
+            self.click (remove_selector)
+            remove_button = self.find_element (remove_selector)
+            self.wait.until (EC.staleness_of (remove_button))
+
+            self.logger.info (
+                f"Product '{product_key}' removed successfully. New badge: {self.get_cart_badge_count ()}")
+            return True
+        except (TimeoutException , StaleElementReferenceException) as e :
+            self.logger.error (f"Failed to remove product '{product_key}': {e}")
+            return False
+
+    def start_checkout ( self ) -> bool :
+        if not self.is_checkout_button_visible () :
+            self.logger.warning ("Checkout button is not visible")
+            return False
+
+        try :
+            self.click (CART["CHECKOUT_BUTTON"])
+            self.wait.until (EC.url_contains ("checkout-step-one"))
+            return True
+        except TimeoutException :
+            self.logger.error ("Failed to navigate to checkout step one")
+            return False

pages/Checkout_Page.py

@@ -0,0 +1,48 @@
+from selenium.webdriver.common.by import By
+from selenium.common.exceptions import TimeoutException
+from selenium.webdriver.support import expected_conditions as EC
+from pages.Base_Page import BasePage
+from config.locators import CHECKOUT
+
+
+class CheckoutPage(BasePage):
+
+    def fill_info(self, first_name: str, last_name: str, postal_code: str) -> bool:
+        try:
+            self.wait.until(EC.url_contains("checkout-step-one"))
+
+            self.fill(CHECKOUT["FIRST_NAME"], first_name)
+            self.fill(CHECKOUT["LAST_NAME"], last_name)
+            self.fill(CHECKOUT["ZIP_CODE"], postal_code)
+
+            self.logger.info(f"Checkout info filled: {first_name} {last_name}")
+            return True
+        except Exception as e:
+            self.logger.error(f"Failed to fill checkout info: {e}")
+            return False
+
+    def continue_to_overview(self) -> bool:
+        try:
+            self.click(CHECKOUT["CONTINUE_BUTTON"])
+            self.wait.until(EC.url_contains("checkout-step-two"))
+            self.logger.info("Navigated to checkout overview (step two)")
+            return True
+        except TimeoutException as e:
+            self.logger.error(f"Failed to continue to overview: {e}")
+            return False
+
+    def complete_checkout(self) -> bool:
+        try:
+            self.click(CHECKOUT["FINISH_BUTTON"])
+            self.wait.until(EC.url_contains("checkout-complete"))
+            self.logger.info("Checkout completed successfully")
+            return True
+        except TimeoutException as e:
+            self.logger.error(f"Failed to complete checkout: {e}")
+            return False
+
+    def is_complete_header_visible(self) -> bool:
+        return self.is_visible(CHECKOUT["COMPLETE_HEADER"])
+
+    def get_complete_header_text(self) -> str:
+        return self.get_text(CHECKOUT["COMPLETE_HEADER"])