Repository: apache/airflow
File: airflow-core/src/airflow/ui/tests/e2e/specs/dag-audit-log.spec.ts (TypeScript)

Code before:
 * software distributed under the License is distributed on an
 * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 * KIND, either express or implied.  See the License for the
 * specific language governing permissions and limitations
 * under the License.
 */
import { expect, test } from "@playwright/test";
import { AUTH_FILE, testConfig } from "playwright.config";
import { DagsPage } from "tests/e2e/pages/DagsPage";
import { EventsPage } from "tests/e2e/pages/EventsPage";

test.describe("DAG Audit Log", () => {
  let eventsPage: EventsPage;

  const testDagId = testConfig.testDag.id;

  test.setTimeout(60_000);

  test.beforeAll(async ({ browser }) => {
    test.setTimeout(7 * 60 * 1000);
    const context = await browser.newContext({ storageState: AUTH_FILE });
    const page = await context.newPage();
    const setupDagsPage = new DagsPage(page);
    const setupEventsPage = new EventsPage(page);

    for (let i = 0; i < 10; i++) {
      await setupDagsPage.triggerDag(testDagId);
    }

    await setupEventsPage.navigateToAuditLog(testDagId);
    await context.close();
  });

  test.beforeEach(({ page }) => {
    eventsPage = new EventsPage(page);
  });

  test("should navigate to audit log tab and display table", async () => {
    await eventsPage.navigateToAuditLog(testDagId);

    const rowCount = await eventsPage.getEventLogCount();

    await expect(eventsPage.eventsTable).toBeVisible();

    const isTableVisible = await eventsPage.isAuditLogTableVisible();

    expect(isTableVisible).toBe(true);
    expect(rowCount).toBeGreaterThanOrEqual(0);
  });

  test("should display all expected columns in audit log table", async () => {

Review comment (bug):
Very slow (could take 70+ minutes with 7-minute timeout per trigger)
If one trigger fails, all tests fail
We can Reduce to 2-3 triggers

Code after:
 * software distributed under the License is distributed on an
 * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 * KIND, either express or implied.  See the License for the
 * specific language governing permissions and limitations
 * under the License.
 */
import { expect, test } from "@playwright/test";
import { AUTH_FILE, testConfig } from "playwright.config";
import { DagsPage } from "tests/e2e/pages/DagsPage";
import { EventsPage } from "tests/e2e/pages/EventsPage";

test.describe("DAG Audit Log", () => {
  let eventsPage: EventsPage;

  const testDagId = testConfig.testDag.id;
  const triggerCount = 3;
  const expectedEventCount = triggerCount + 1;

  test.setTimeout(60_000);

  test.beforeAll(async ({ browser }) => {
    test.setTimeout(3 * 60 * 1000);
    const context = await browser.newContext({ storageState: AUTH_FILE });
    const page = await context.newPage();
    const setupDagsPage = new DagsPage(page);
    const setupEventsPage = new EventsPage(page);

    for (let i = 0; i < triggerCount; i++) {
      await setupDagsPage.triggerDag(testDagId);
    }

    await setupEventsPage.navigateToAuditLog(testDagId);
    await page.waitForFunction(
      (minCount) => {
        const table = document.querySelector('[data-testid="table-list"]');

        if (!table) {
          return false;
        }
        const rows = table.querySelectorAll("tbody tr");

        return rows.length >= minCount;
      },
      expectedEventCount,
      { timeout: 60_000 },
    );

    await context.close();
  });

  test.beforeEach(({ page }) => {