Merge pull request #1 from Vamshi-Microsoft/vk-deploy-st

Merge Smoke Testing into Main

Author: Vamshi-Microsoft

tests/e2e-test/base/base.py

@@ -1,10 +1,38 @@
+"""
+Base page module providing common functionality for all page objects.
+"""
+
+
 class BasePage:
+    """Base class for all page objects with common methods."""
+
     def __init__(self, page):
+        """
+        Initialize the BasePage with a Playwright page instance.
+
+        Args:
+            page: Playwright page object
+        """
         self.page = page
 
     def scroll_into_view(self, locator):
+        """
+        Scroll the last element matching the locator into view.
+
+        Args:
+            locator: Playwright locator object
+        """
         reference_list = locator
         locator.nth(reference_list.count() - 1).scroll_into_view_if_needed()
 
     def is_visible(self, locator):
-        locator.is_visible()
+        """
+        Check if an element is visible on the page.
+
+        Args:
+            locator: Playwright locator object
+
+        Returns:
+            bool: True if visible, False otherwise
+        """
+        return locator.is_visible()

tests/e2e-test/config/constants.py

@@ -1,8 +1,12 @@
+"""
+Configuration constants module for test environment settings.
+"""
+
 import os
 
 from dotenv import load_dotenv
 
 load_dotenv()
 URL = os.getenv("url")
-if URL.endswith("/"):
+if URL and URL.endswith("/"):
     URL = URL[:-1]