Added gemini-cli capabilities, refactored fixture to testdata and minor refactor

Author: Tahanima

.gemini/skills/code-reviewer/SKILL.md

@@ -0,0 +1,13 @@
+---
+name: code-reviewer
+description: Acts as a Senior QA Lead to review Playwright-Java code, ensuring architectural integrity and test stability.
+---
+# Code Reviewer Instructions
+When this skill is active:
+1. **Analyze the Role:** Determine if the file is a Page (ui/page), a Component (ui/component), a TestData class (testData), or a Test (e2e).
+2. **Apply Architectural Rules:** Use `review-rules.md` to ensure the logic is placed in the correct layer.
+3. **Feedback Format:**
+    - ❌ **Violation:** Serious architectural or stability issues.
+    - ⚠️ **Improvement:** Code smells or optimization opportunities.
+    - ✅ **Correct:** Praise for good implementation.
+4. **Correction:** Always provide the refactored code snippet following the `io.github.tahanima` package standards.

.gemini/skills/code-reviewer/review-rules.md

@@ -0,0 +1,26 @@
+# Architecture Review Rules
+
+## 1. UI Layer (Pages & Components)
+- **Inheritance:** `BasePage` must be extended by all pages; `BaseComponent` by all components.
+- **Locators:** Must be `private`. Use `page.getByRole`, `page.getByLabel`, or `page.getByTestId`.
+- **Method Scope:** Page methods should return `void` or a new Page instance (Fluent interface).
+- **Separation:** Ensure `Header` or `SideNavMenu` elements are not duplicated in `LoginPage`. They must stay in the `component` package.
+
+## 2. Test Data Layer (The Bridge)
+- **Role:** Review `LoginTestData` or `ProductsTestData`. These should act as POJOs or Logic wrappers that interpret CSV data for the UI.
+- **Factory usage:** Ensure these classes don't instantiate pages using `new`. They should rely on the `BasePageFactory`.
+
+## 3. E2E Test Layer
+- **Inheritance:** All tests must extend `BaseTest`.
+- **Annotations:**
+    - Mandatory: `@Epic`, `@Feature`, and `@Story` (Allure).
+    - Mandatory: Category annotations like `@Smoke` or `@Regression`.
+- **Data Driven:** - Verify `@ParameterizedTest` is used with `@TestDataSource(fileName = "xxx.csv")`.
+    - Flag any hardcoded test data strings.
+- **Assertions:** Use Playwright's `assertThat()` for auto-retrying. Avoid generic JUnit `Assertions.assertEquals()` for UI elements.
+
+## 4. Framework Consistency
+- **Configuration:** Use `ConfigurationManager.configuration()` to access properties; never use `System.getProperty` directly.
+- **Logging:** Use `AllureManager` for any custom logging or attachments within the test flow.
+- **Clean Code:** - No `Thread.sleep()`.
+    - No `System.out.println()` (Use Allure steps).

.gemini/skills/healing-agent/SKILL.md

@@ -0,0 +1,9 @@
+---
+name: healing-agent
+description: Expert in diagnosing Playwright failures and refactoring brittle locators into robust, accessibility-first selectors.
+---
+# Healing Agent Instructions
+When a test fails (especially a TimeoutError or 'element not found'):
+1. Load the corresponding Page Object or Component class from `src/main/java/io/github/tahanima/ui/`.
+2. Apply the "Healing Logic" defined in `locator-healer.md`.
+3. Propose the corrected Java code and explain why the new locator is more stable.

.gemini/skills/healing-agent/locator-healer.md

@@ -0,0 +1,23 @@
+# Playwright Locator Healing Logic
+
+## 1. Diagnose the Brittle Locator
+Identify if the current locator uses any of these "Anti-Patterns":
+- Long XPath strings (e.g., `/html/body/div[1]/...`)
+- CSS classes that look autogenerated (e.g., `.css-19v7kv`)
+- Locators relying on specific DOM nesting that is likely to change.
+
+## 2. Healing Strategy (Priority Order)
+Refactor the broken locator using the following Playwright priority list:
+1.  **Role (Preferred):** `page.getByRole(AriaRole.BUTTON, new Page.GetByRoleOptions().setName("Login"))`
+2.  **Label:** `page.getByLabel("Username")`
+3.  **Placeholder:** `page.getByPlaceholder("Enter password")`
+4.  **Text:** `page.getByText("Welcome back")`
+5.  **Alt Text:** `page.getByAltText("Company Logo")`
+6.  **TestID:** `page.getByTestId("submit-button")` (If `data-testid` is present in your HTML).
+
+## 3. Example Transformation
+**Broken:**
+`private Locator loginBtn = page.locator("//div[@id='login-form']//button[2]");`
+
+**Healed:**
+`private Locator loginBtn = page.getByRole(AriaRole.BUTTON, new Page.GetByRoleOptions().setName("Login"));`

GEMINI.md

@@ -0,0 +1,78 @@
+# Project: Playwright Java Test Automation Architecture
+
+## Context & Tech Stack
+- **Build Tool:** Gradle (using `build.gradle`)
+- **Language:** Java 17+
+- **Browser Automation Library:** Playwright
+- **Architecture:** Data-driven test automation using POM to model and reuse web page functionality.
+- **Base Package:** `io.github.tahanima`
+- **Test Runner:** JUnit 5 with custom annotations (`@TestDataSource`, `@Validation`, `@Smoke`, `@Regression`).
+
+---
+
+## Project Mapping
+- **Pages:** `src/main/java/io/github/tahanima/ui/page/` – Encapsulate low-level page actions and locators.
+- **Components:** `src/main/java/io/github/tahanima/ui/component/` – Reusable UI elements (Header, SideNav, etc.).
+- **TestData POJOs:** `src/main/java/io/github/tahanima/testdata/` – Business logic layer that interprets CSV data and interacts with Pages.
+- **Tests:** `src/test/java/io/github/tahanima/e2e/` – High-level orchestration; must extend `BaseTest`.
+- **Test Data Files:** `src/test/resources/testdata/` – CSV files for data-driven inputs.
+- **Utilities:**
+    - `BrowserManager.java` – Browser lifecycle management.
+    - `CsvLoader.java` – CSV loading for test data.
+- **Factories:**
+    - `BasePageFactory.java` – Centralized page initialization.
+    - `BrowserFactory.java` – Abstracts browser setup and creation.
+- **Reporting:** `AllureManager.java` – Custom logging, screenshots, and test reporting.
+
+---
+
+## Development Rules
+1. **Instantiation:** Do not use `new LoginPage()`. Always use the `BasePageFactory` to initialize pages.
+2. **Logic Placement:**
+    - **Pages/Components:** Handle locators and atomic actions (e.g., `typeUsername`).
+    - **TestData POJOs:** Handle business workflows (e.g., `loginWith(LoginTestData data)`).
+    - **Tests:** Handle assertions and test flow. Minimal direct Playwright API calls here.
+3. **Stability & Waits:**
+    - **No Hard Sleeps:** Never use `Thread.sleep()`. Rely on Playwright’s auto-waiting or `page.waitForCondition()`.
+    - **Web-First Assertions:** Use `PlaywrightAssertions.assertThat()` for UI states to enable automatic retries.
+4. **Configuration:** Use `ConfigurationManager` to access `config.properties`. Do not hardcode URLs or credentials.
+5. **Data Loading:** Use the custom `@TestDataSource` annotation for CSV data-driven tests.
+6. **Naming Conventions:**
+    - Pages → `*Page.java`
+    - Components → `*Component.java`
+    - Test Data POJOs → `*TestData.java` (must extend `BaseTestData`).
+
+---
+
+## Locator Strategy (Priority Order)
+1. `page.getByRole()` (Prioritize accessibility-first selectors).
+2. `page.getByLabel()` / `page.getByPlaceholder()`.
+3. `page.getByTestId()` (Use when `data-testid` attributes are available).
+4. CSS/XPath (Use only as a last resort for complex structural selectors).
+
+---
+
+## Test Data Flow
+- CSV files → `TestDataArgumentsProvider` → Test method parameters → Test execution.
+- POJOs (`testdata` package) represent structured test data for readability and business logic reuse.
+
+---
+
+## Running Tests
+- Run all tests: `./gradlew clean test`
+- Run a specific test class: `./gradlew test --tests *LoginTest`
+- Generate Allure report: `./gradlew allureServe`
+
+---
+
+## Available Skills (AI Toolbelt)
+1. **code-reviewer**:
+    - **Trigger:** When providing new code or asking for architectural feedback.
+    - **Focus:** Enforcing layer separation (Page vs. TestData vs. E2E) and checking Factory usage.
+2. **healing-agent**:
+    - **Trigger:** When reporting a `TimeoutError`, broken locator, or execution failure.
+    - **Focus:** Converting brittle selectors to the "Locator Strategy" priority list above.
+
+### Usage Guidelines
+- **Skill Activation:** If a task matches the triggers above, ask for permission to load the skill using `/skills load <name>`.
+- **Contextual Awareness:** Before proposing code changes, refer to `src/main/resources/config.properties` for environment-specific settings.

README.md

@@ -11,6 +11,8 @@ Ready-to-use UI Test Automation Architecture using Java and Playwright.
 - Records video of test execution
 - Provides detailed test report
 - Supports parallel test execution
+- Supports gemini-cli for AI-assisted coding
+---
 
 ## Installation Steps
 
@@ -28,6 +30,7 @@ git clone https://github.com/[your_username]/playwright-java-test-automation-arc
 ```
 ./gradlew allureServe
 ```
+---
 
 ## Languages and Frameworks
 
@@ -41,13 +44,23 @@ The project uses the following:
 - *[Allure Report](https://qameta.io/allure-report/)* as the test reporting strategy.
 - *[Gradle](https://gradle.org/)* as the Java build tool.
 - *[IntelliJ IDEA](https://www.jetbrains.com/idea/)* as the IDE.
+- *[Gemini CLI](https://github.com/google-gemini/gemini-cli)* as an AI collaborator for code reviews, locator healing, and test generation.
+---
 
 ## Project Structure
 
 The project is structured as follows:
 
 ```bash
 📦 playwright-java-test-automation-architecture
+├─ .gemini
+│  └─ skills
+│     └─ code-reviewer
+|        └─ review-rules.md
+|        └─ SKILL.md
+|     └─ healing-agent
+|        └─ locator-healer.md
+|        └─ SKILL.md
 ├─ .github
 │  ├─ FUNDING.yml
 │  ├─ dependabot.yml
@@ -56,6 +69,7 @@ The project is structured as follows:
 ├─ .gitignore
 ├─ LICENSE
 ├─ README.md
+├─ GEMINI.md
 ├─ build.gradle
 ├─ gradle
 │  └─ wrapper
@@ -76,10 +90,10 @@ The project is structured as follows:
    │  │           ├─ factory
    │  │           │  ├─ BasePageFactory.java
    │  │           │  └─ BrowserFactory.java
-   │  │           ├─ fixture
-   │  │           │  ├─ BaseFixture.java
-   │  │           │  ├─ LoginFixture.java
-   │  │           │  └─ ProductsFixture.java
+   │  │           ├─ testData
+   │  │           │  ├─ BaseTestData.java
+   │  │           │  ├─ LoginTestData.java
+   │  │           │  └─ ProductsTestData.java
    │  │           ├─ report
    │  │           │  └─ AllureManager.java
    │  │           ├─ ui
@@ -112,13 +126,14 @@ The project is structured as follows:
       │           │  └─ ProductsTest.java
       │           └─ util
       │              ├─ TestDataArgumentsProvider.java
-      │              └─ TestFixtureCsvLoader.java
+      │              └─ CsvLoader.java
       └─ resources
          ├─ junit-platform.properties
          └─ testdata
             ├─ login.csv
             └─ products.csv
 ```
+---
 
 ## Basic Usage
 
@@ -162,10 +177,10 @@ The project is structured as follows:
 - ### Test Data
   The project uses *csv* file to store test data and [*univocity-parsers*](https://github.com/uniVocity/univocity-parsers) to retrieve the data and map it to a Java bean.
 
-  To add configurations for new test data, add a new Java bean in the [*fixture*](./src/main/java/io/github/tahanima/fixture) package. For example, let's say I want to add test data for a `User` with the attributes `First Name` and `Last Name`. The code for this is as follows:
+  To add configurations for new test data, add a new Java bean in the [*testData*](./src/main/java/io/github/tahanima/testdata) package. For example, let's say I want to add test data for a `User` with the attributes `First Name` and `Last Name`. The code for this is as follows:
 
    ```java
-   package io.github.tahanima.fixture;
+   package io.github.tahanima.testdata;
 
    import com.univocity.parsers.annotations.Parsed;
 
@@ -174,7 +189,7 @@ The project is structured as follows:
 
    @Getter
    @ToString(callSuper = true)
-   public class UserFixture extends BaseFixture {
+   public class UsertestData extends BasetestData {
 
        @Parsed(field = "First Name", defaultNullRead = "")
        private String firstName;
@@ -183,17 +198,47 @@ The project is structured as follows:
        private String lastName;
    }
    ```
-   Note that the class extends from BaseFixture and thus, inherits the attribute `Test Case ID`.
+   Note that the class extends from BasetestData and thus, inherits the attribute `Test Case ID`.
 
    Now, in the [*testdata*](./src/test/resources/testdata) folder you can add a csv file `user.csv` for `User` with the below contents and use it in your tests.
    ```
    Test Case ID,First Name,Last Name
    TC-1,Tahanima,Chowdhury
    ```
-   For reference, check [this](./src/main/java/io/github/tahanima/fixture/LoginFixture.java), [this](./src/test/resources/testdata/login.csv) and [this](./src/test/java/io/github/tahanima/e2e/LoginTest.java).
+   For reference, check [this](./src/main/java/io/github/tahanima/testdata/LogintestData.java), [this](./src/test/resources/testdata/login.csv) and [this](./src/test/java/io/github/tahanima/e2e/LoginTest.java).
 
 - ### Page Objects and Page Component Objects
   The project uses [*Page Objects* and *Page Component Objects*](https://www.selenium.dev/documentation/test_practices/encouraged/page_object_models/) to capture the relevant behaviors of a web page. Check the [*ui*](./src/main/java/io/github/tahanima/ui) package for reference.
 
 - ### Tests
   The project uses *JUnit 5* as the test runner. Check [this implementation](./src/test/java/io/github/tahanima/e2e/LoginTest.java) for reference.
+---
+
+## AI-Augmented Workflow
+
+This project is optimized for use with **Gemini CLI**, providing a specialized AI agent that understands the architecture.
+
+### Prerequisites
+- Gemini CLI: -[Installed and authenticated](https://github.com/google-gemini/gemini-cli) via your Google Account.
+- Project Root: Ensure you run the CLI from the project root to allow it to parse the .gemini/ configuration.
+
+### Project Context (GEMINI.md)
+The project includes a `GEMINI.md` file which serves as the AI's "Source of Truth." This ensures that any code generated or reviewed by the AI adheres to:
+- **BasePageFactory** initialization patterns.
+- **Layer Separation** (Pages vs. TestData POJOs vs. E2E Tests).
+- **Playwright Best Practices** (Accessibility-first locators over brittle XPaths).
+
+### Specialized AI Skills
+We have implemented custom **Skills** in the `.gemini/skills/` directory to automate common QA tasks:
+
+- **`code-reviewer`**: Automatically audits new Page Objects, POJOs, and Tests to ensure they follow the project's layer separation and naming conventions.
+- **`healing-agent`**: Diagnoses test failures (like `TimeoutError`) and suggests robust, accessibility-first Playwright locators (e.g., `getByRole`) to replace brittle XPaths.
+
+### How to Use
+Simply run `gemini` in the project root. You can trigger skills manually or let the AI suggest them:
+```bash
+# To perform a code review
+gemini "Review this new test class: @src/test/java/io/github/tahanima/e2e/NewFeatureTest.java"
+
+# To fix a failing locator
+gemini "The login button locator is failing. Use the healing-agent to suggest a fix for @src/main/java/io/github/tahanima/ui/page/LoginPage.java"

src/main/java/io/github/tahanima/factory/BasePageFactory.java

@@ -1,7 +1,9 @@
 package io.github.tahanima.factory;
 
 import com.microsoft.playwright.Page;
+
 import io.github.tahanima.ui.page.BasePage;
+
 import lombok.extern.slf4j.Slf4j;
 
 /**
@@ -12,7 +14,7 @@ public final class BasePageFactory {
 
     private BasePageFactory() {}
 
-    public static <T extends BasePage> T createInstance(final Page page, final Class<T> clazz) {
+    public static <T extends BasePage> T createInstance(Page page, Class<T> clazz) {
         try {
             BasePage instance = clazz.getDeclaredConstructor().newInstance();
 

src/main/java/io/github/tahanima/factory/BrowserFactory.java

@@ -14,19 +14,19 @@ public enum BrowserFactory {
 
     CHROMIUM {
         @Override
-        public Browser createInstance(final Playwright playwright) {
+        public Browser createInstance(Playwright playwright) {
             return playwright.chromium().launch(options());
         }
     },
     FIREFOX {
         @Override
-        public Browser createInstance(final Playwright playwright) {
+        public Browser createInstance(Playwright playwright) {
             return playwright.firefox().launch(options());
         }
     },
     WEBKIT {
         @Override
-        public Browser createInstance(final Playwright playwright) {
+        public Browser createInstance(Playwright playwright) {
             return playwright.webkit().launch(options());
         }
     };
@@ -37,5 +37,5 @@ public LaunchOptions options() {
                 .setSlowMo(config().slowMotion());
     }
 
-    public abstract Browser createInstance(final Playwright playwright);
+    public abstract Browser createInstance(Playwright playwright);
 }

…github/tahanima/fixture/BaseFixture.java …thub/tahanima/testdata/BaseTestData.javasrc/main/java/io/github/tahanima/fixture/BaseFixture.java renamed to src/main/java/io/github/tahanima/testdata/BaseTestData.java src/main/java/io/github/tahanima/fixture/BaseFixture.java renamed to src/main/java/io/github/tahanima/testdata/BaseTestData.java

@@ -1,4 +1,4 @@
-package io.github.tahanima.fixture;
+package io.github.tahanima.testdata;
 
 import com.univocity.parsers.annotations.Parsed;
 import lombok.Getter;
@@ -9,7 +9,7 @@
  */
 @Getter
 @ToString
-public class BaseFixture {
+public class BaseTestData {
 
     @Parsed(field = "Test Case ID", defaultNullRead = "")
     private String testCaseId;

…ithub/tahanima/fixture/LoginFixture.java …hub/tahanima/testdata/LoginTestData.javasrc/main/java/io/github/tahanima/fixture/LoginFixture.java renamed to src/main/java/io/github/tahanima/testdata/LoginTestData.java src/main/java/io/github/tahanima/fixture/LoginFixture.java renamed to src/main/java/io/github/tahanima/testdata/LoginTestData.java

@@ -1,4 +1,4 @@
-package io.github.tahanima.fixture;
+package io.github.tahanima.testdata;
 
 import com.univocity.parsers.annotations.Parsed;
 import lombok.Getter;
@@ -9,7 +9,7 @@
  */
 @Getter
 @ToString(callSuper = true)
-public class LoginFixture extends BaseFixture {
+public class LoginTestData extends BaseTestData {
 
     @Parsed(field = "Username", defaultNullRead = "")
     private String username;