VedantMadane opened a new pull request, #60586: URL: https://github.com/apache/airflow/pull/60586
## Summary This PR adds comprehensive E2E tests for the Variables page (`/variables`) functionality. Closes #60565 ## What to test (from issue requirements) - [x] Verify variables list displays - [x] Create variable - [x] Edit variable - [x] Delete variable - [x] Search variables - [x] Import variables from file - [x] Verify pagination works - [x] Verify sorting behaviour ## Implementation Details ### Page Object: `VariablesPage.ts` Created a new page object following the Page Object Model (POM) pattern with: - Locators for all interactive elements (buttons, inputs, table cells) - Methods for all CRUD operations - Navigation and wait helpers - Search and pagination utilities ### Test Specs: `variables.spec.ts` Test suites include: 1. **List Display** - Verifies page structure, column headers, and basic layout 2. **CRUD Operations** - Create, edit, and delete individual variables 3. **Search** - Filter variables by search term, clear search, handle no results 4. **Import** - Import variables from JSON file 5. **Pagination** - Display pagination controls, navigate between pages 6. **Sorting** - Verify sortable column headers ### Acceptance Criteria Compliance - All tests follow Page Object Model (POM) pattern - Tests use `testConfig` compatible pattern - Tests create/cleanup their own test data in `beforeAll`/`afterAll` - Tests work across Chromium, Firefox, and WebKit browsers (using Playwright) - No hardcoded values - uses dynamic data with unique keys ## Testing These tests can be run locally with: ```bash breeze testing ui-e2e-tests --test-pattern "variables.spec.ts" ``` Or directly: ```bash cd airflow-core/src/airflow/ui pnpm install pnpm test:e2e:install pnpm test:e2e -- --grep "Variables" ``` ## Related Issues - Closes #60565 - Related to meta issue #59028 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
