haseebmalik18 opened a new pull request, #59734:
URL: https://github.com/apache/airflow/pull/59734

   Add E2E tests for DAG audit log functionality
   
     This PR implements comprehensive end-to-end tests for the DAG audit log 
feature to ensure proper functionality of the Events tab.
   
     ## Changes
   
     ### New Files
     - `airflow-core/src/airflow/ui/tests/e2e/pages/EventsPage.ts` - Page 
Object Model for the Events/audit log page
     - `airflow-core/src/airflow/ui/tests/e2e/specs/dag-audit-log.spec.ts` - 
E2E test specifications
   
     ### Test Coverage
     The implementation includes 5 test scenarios covering:
     1. **Navigation and Table Visibility** - Verifies users can navigate to 
the audit log tab and the table renders correctly
     2. **Column Verification** - Ensures all expected columns (When, Event, 
User, Extra) are displayed and DAG ID column is hidden in DAG-specific context
     3. **Data Validation** - Validates that audit log entries contain valid 
data with proper null/edge case handling
     4. **Pagination** - Tests navigation between pages and verifies content 
changes appropriately
     5. **Sorting** - Validates column header sorting functionality
   
     ### Implementation Details
     - Follows existing Page Object Model patterns from `DagsPage.ts`
     - Uses `data-testid` selectors for stable test automation
     - Includes proper error handling and edge case coverage (empty data, no 
pagination, insufficient rows)
     - All code passes ESLint with strict alphabetical sorting requirements 
(`perfectionist/sort-classes`)
     - Follows TypeScript strict mode conventions
   
     closes: #59684
   
     ---
     **^ Add meaningful description above**
     Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)**
 for more information.
     In case of fundamental code changes, an Airflow Improvement Proposal 
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals))
 is needed.
     In case of a new dependency, check compliance with the [ASF 3rd Party 
License Policy](https://www.apache.org/legal/resolved.html#category-x).
     In case of backwards incompatible changes please leave a note in a 
newsfragment file, named `{pr_number}.significant.rst` or 
`{issue_number}.significant.rst`, in 
[airflow-core/newsfragments](https://github.com/apache/airflow/tree/main/airflow-core/newsfragments).
   


-- 
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]

Reply via email to