From df9b50d373cae51ebaee40a9ee24d3f9bf78e8ed Mon Sep 17 00:00:00 2001
From: piotrk39 <piotrk39@gmail.com>
Date: Wed, 22 Jan 2025 14:20:23 +0100
Subject: [PATCH 1/3] Change test-ids for the elements fix test

---
 .../app/components/logos/logos.component.html |  2 +-
 .../user-state/user-state.component.html      |  9 +++------
 tests/e2e/Pages/UserStateComponent.ts         | 19 +++++++++----------
 .../UserStateComponentVisible.ts              |  2 +-
 4 files changed, 14 insertions(+), 18 deletions(-)

diff --git a/interfaces/IBF-dashboard/src/app/components/logos/logos.component.html b/interfaces/IBF-dashboard/src/app/components/logos/logos.component.html
index d38cdd4ce4..8539b1f367 100644
--- a/interfaces/IBF-dashboard/src/app/components/logos/logos.component.html
+++ b/interfaces/IBF-dashboard/src/app/components/logos/logos.component.html
@@ -2,5 +2,5 @@
   logo of (country | async)?.countryLogos[(disasterType | async)?.disasterType];
   track logo
 ) {
-  <img data-test="logo-image" src="assets/logos/{{ logo }}" />
+  <img data-testid="logo-image" src="assets/logos/{{ logo }}" />
 }
diff --git a/interfaces/IBF-dashboard/src/app/components/user-state/user-state.component.html b/interfaces/IBF-dashboard/src/app/components/user-state/user-state.component.html
index 344e45bc75..aa18c4d0e3 100644
--- a/interfaces/IBF-dashboard/src/app/components/user-state/user-state.component.html
+++ b/interfaces/IBF-dashboard/src/app/components/user-state/user-state.component.html
@@ -30,19 +30,16 @@
     </div>
     <div class="ion-align-items-center">
       <ion-row class="ion-align-items-center">
-        <app-logos data-testid="user-state-country-logos"></app-logos>
+        <app-logos></app-logos>
         <ion-item lines="none">
-          <ion-col
-            data-testid="user-state-display-name-label"
-            class="ion-no-padding"
-          >
+          <ion-col class="ion-no-padding">
             @if (isLoggedIn) {
               <ion-title size="small" class="ion-no-padding"
                 >Logged In As:</ion-title
               >
               <ion-title
                 size="small"
-                data-test="user-display-name"
+                data-testid="user-display-name"
                 class="ion-no-padding"
               >
                 <strong
diff --git a/tests/e2e/Pages/UserStateComponent.ts b/tests/e2e/Pages/UserStateComponent.ts
index 41ee0705b3..3722176008 100644
--- a/tests/e2e/Pages/UserStateComponent.ts
+++ b/tests/e2e/Pages/UserStateComponent.ts
@@ -14,12 +14,8 @@ class UserStateComponent extends DashboardPage {
     super(page);
     this.page = page;
     this.header = this.page.getByTestId('heading-display-name-span');
-    this.countryLogos = this.page
-      .getByTestId('user-state-country-logos')
-      .nth(0);
-    this.userLoggedInLabel = this.page
-      .getByTestId('user-state-display-name-label')
-      .nth(0);
+    this.countryLogos = this.page.getByTestId('logo-image');
+    this.userLoggedInLabel = this.page.getByTestId('user-display-name');
     this.logOutButton = this.page.getByTestId('user-state-logout-button');
   }
 
@@ -55,11 +51,14 @@ class UserStateComponent extends DashboardPage {
     firstName: string;
     lastName: string;
   }) {
+    const countryLogosCount = await this.countryLogos.count();
+    for (let i = 0; i < countryLogosCount; i++) {
+      await expect(this.countryLogos.nth(i)).toBeVisible();
+      console.log(i);
+    }
+
     await expect(this.dashboardHomeButton).toBeVisible();
-    await expect(this.countryLogos).toBeVisible(); // skip-reason: test fails often because it claims the element is hidden instead of visible
-    await expect(this.userLoggedInLabel).toHaveText(
-      `Logged In As:${firstName} ${lastName}`,
-    );
+    await expect(this.userLoggedInLabel).toHaveText(`${firstName} ${lastName}`);
     await expect(this.logOutButton).toBeVisible();
   }
 
diff --git a/tests/e2e/tests/UserStateComponent/UserStateComponentVisible.ts b/tests/e2e/tests/UserStateComponent/UserStateComponentVisible.ts
index 7dde73bf90..0bceaadd7d 100644
--- a/tests/e2e/tests/UserStateComponent/UserStateComponentVisible.ts
+++ b/tests/e2e/tests/UserStateComponent/UserStateComponentVisible.ts
@@ -10,7 +10,7 @@ export default (
   disasterType: string,
 ) => {
   // Skip for now, as test often fails
-  test.skip(
+  test(
     qase(3, 'User state component elements should be visible'),
     {
       annotation: {

From 3920938d2d615e636719fe3cd98cab9b3dde5850 Mon Sep 17 00:00:00 2001
From: piotrk39 <piotrk39@gmail.com>
Date: Wed, 22 Jan 2025 14:20:50 +0100
Subject: [PATCH 2/3] remove console.log

---
 tests/e2e/Pages/UserStateComponent.ts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tests/e2e/Pages/UserStateComponent.ts b/tests/e2e/Pages/UserStateComponent.ts
index 3722176008..7d6e1d9fe8 100644
--- a/tests/e2e/Pages/UserStateComponent.ts
+++ b/tests/e2e/Pages/UserStateComponent.ts
@@ -54,7 +54,6 @@ class UserStateComponent extends DashboardPage {
     const countryLogosCount = await this.countryLogos.count();
     for (let i = 0; i < countryLogosCount; i++) {
       await expect(this.countryLogos.nth(i)).toBeVisible();
-      console.log(i);
     }
 
     await expect(this.dashboardHomeButton).toBeVisible();

From 77b94dd90253f2e03fe164fc1eff71e2cf400cb5 Mon Sep 17 00:00:00 2001
From: piotrk39 <piotrk39@gmail.com>
Date: Wed, 22 Jan 2025 14:30:15 +0100
Subject: [PATCH 3/3] remove comment

---
 tests/e2e/tests/UserStateComponent/UserStateComponentVisible.ts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tests/e2e/tests/UserStateComponent/UserStateComponentVisible.ts b/tests/e2e/tests/UserStateComponent/UserStateComponentVisible.ts
index 0bceaadd7d..354945abf7 100644
--- a/tests/e2e/tests/UserStateComponent/UserStateComponentVisible.ts
+++ b/tests/e2e/tests/UserStateComponent/UserStateComponentVisible.ts
@@ -9,7 +9,6 @@ export default (
   components: Partial<Components>,
   disasterType: string,
 ) => {
-  // Skip for now, as test often fails
   test(
     qase(3, 'User state component elements should be visible'),
     {