From 568b6e0ad786f9ca97d79595dad94eeffe2689a6 Mon Sep 17 00:00:00 2001
From: Hynek Mlnarik <hmlnarik@redhat.com>
Date: Fri, 16 Feb 2024 10:40:11 +0100
Subject: [PATCH] Add device activity tests

Fixes: #21247
Signed-off-by: Hynek Mlnarik <hmlnarik@redhat.com>
---
 .../src/account-security/DeviceActivity.tsx   |  4 +-
 .../account-security/device-activity.spec.ts  | 81 +++++++++++++++++++
 2 files changed, 83 insertions(+), 2 deletions(-)
 create mode 100644 js/apps/account-ui/test/account-security/device-activity.spec.ts

diff --git a/js/apps/account-ui/src/account-security/DeviceActivity.tsx b/js/apps/account-ui/src/account-security/DeviceActivity.tsx
index 22ccf1fd34..fa333a3c5b 100644
--- a/js/apps/account-ui/src/account-security/DeviceActivity.tsx
+++ b/js/apps/account-ui/src/account-security/DeviceActivity.tsx
@@ -145,8 +145,8 @@ export const DeviceActivity = () => {
       >
         <DataListItem aria-labelledby={`sessions-${key}`}>
           {devices.map((device) =>
-            device.sessions.map((session) => (
-              <DataListItemRow key={device.id}>
+            device.sessions.map((session, index) => (
+              <DataListItemRow key={device.id} data-testid={`row-${index}`}>
                 <DataListContent
                   aria-label="device-sessions-content"
                   className="pf-u-flex-grow-1"
diff --git a/js/apps/account-ui/test/account-security/device-activity.spec.ts b/js/apps/account-ui/test/account-security/device-activity.spec.ts
new file mode 100644
index 0000000000..60dd08990e
--- /dev/null
+++ b/js/apps/account-ui/test/account-security/device-activity.spec.ts
@@ -0,0 +1,81 @@
+import { expect, test } from "@playwright/test";
+import { login } from "../login";
+
+test.describe("Sign out test", () => {
+  test("Sign out one device", async ({ browser }) => {
+    const context1 = await browser.newContext({
+      userAgent:
+        "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko)",
+    });
+    const context2 = await browser.newContext();
+    try {
+      const page1 = await context1.newPage();
+      const page2 = await context2.newPage();
+      await login(page1, "jdoe", "jdoe", "groups");
+      await page1.getByTestId("accountSecurity").click();
+      await expect(
+        page1.getByTestId("account-security/device-activity"),
+      ).toBeVisible();
+      await page1.getByTestId("account-security/device-activity").click();
+      await expect(page1.getByTestId("row-0")).toContainText("Current session");
+
+      await login(page2, "jdoe", "jdoe", "groups");
+      await page2.getByTestId("accountSecurity").click();
+      await expect(
+        page2.getByTestId("account-security/device-activity"),
+      ).toBeVisible();
+      await page2.getByTestId("account-security/device-activity").click();
+
+      await page2
+        .getByRole("button", { name: "Sign out", exact: true })
+        .click();
+      await page2.getByRole("button", { name: "Confirm" }).click();
+
+      // reload pages in browsers, one should stay logged in, the other should be logged out
+      await page1.reload();
+      await page2.reload();
+      await expect(
+        page1.getByRole("heading", { name: "Sign in to your account" }),
+      ).toBeVisible();
+      await expect(page2.getByTestId("accountSecurity")).toBeVisible();
+    } finally {
+      await context1.close();
+      await context2.close();
+    }
+  });
+
+  test("Sign out all devices", async ({ browser }) => {
+    const context1 = await browser.newContext({
+      userAgent:
+        "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko)",
+    });
+    const context2 = await browser.newContext();
+    try {
+      const page1 = await context1.newPage();
+      const page2 = await context2.newPage();
+      await login(page1, "jdoe", "jdoe", "groups");
+      await login(page2, "jdoe", "jdoe", "groups");
+
+      await page2.getByTestId("accountSecurity").click();
+      await page2.getByTestId("account-security/device-activity").click();
+
+      await page2
+        .getByRole("button", { name: "Sign out all devices", exact: true })
+        .click();
+      await page2.getByRole("button", { name: "Confirm" }).click();
+
+      // reload pages in browsers, one should stay logged in, the other should be logged out
+      await page1.reload();
+      // Reload in page2 should not be needed, as it should be logged out after clicking the button
+      await expect(
+        page1.getByRole("heading", { name: "Sign in to your account" }),
+      ).toBeVisible();
+      await expect(
+        page2.getByRole("heading", { name: "Sign in to your account" }),
+      ).toBeVisible();
+    } finally {
+      await context1.close();
+      await context2.close();
+    }
+  });
+});