Test stabilization - multiple tabs navigation

Author: soncuteanca

tests/tabs/test_navigation_multiple_tabs.py

@@ -1,6 +1,4 @@
 import logging
-import sys
-from os import environ
 
 import pytest
 from selenium.webdriver import Firefox
@@ -13,37 +11,53 @@ def test_case():
     return "134647"
 
 
-WIN_GHA = environ.get("GITHUB_ACTIONS") == "true" and sys.platform.startswith("win")
-
-
-@pytest.mark.skipif(WIN_GHA, reason="Test unstable in Windows Github Actions")
 def test_navigation_multiple_tabs(driver: Firefox):
-    """C134647 - Verify that Multiple Tabs can be closed via the context menu"""
-    # open 20 tabs
+    """C134647 - Verify that multiple tabs can be navigated via the scroll buttons"""
+
     tabs = TabBar(driver)
     num_tabs = 20
 
+    # Open multiple tabs
     for _ in range(num_tabs):
         tabs.new_tab_by_button()
 
-    # after opening 20 tabs, should be at the right most tab
     with driver.context(driver.CONTEXT_CHROME):
-        # get the last tab, verify that upon double clicking the left scroll position decreased
+        # Get the last tab's initial position (x-axis)
         last_tab = tabs.get_tab(21)
         original_location_pre_left = last_tab.location["x"]
+
+        # Scroll left via double click
         tabs.double_click("tab-scrollbox-left-button", labels=[])
-        new_location_post_left = last_tab.location["x"]
 
-        logging.info(f"The original position: {original_location_pre_left}")
-        logging.info(f"The new position: {new_location_post_left}")
-        assert new_location_post_left < original_location_pre_left
+        # Wait for left scroll to change position (x should increase)
+        def tab_scrolled_left(d):
+            current_pos = tabs.get_tab(21).location["x"]
+            return current_pos > original_location_pre_left
+
+        tabs.custom_wait(timeout=2).until(tab_scrolled_left)
 
-        # get the first tab, verify that upon clicking the right scroll position increased
+        # Log and assert the position after left scroll
+        new_location_post_left = tabs.get_tab(21).location["x"]
+        logging.info(f"Left scroll - original x: {original_location_pre_left}")
+        logging.info(f"Left scroll - new x: {new_location_post_left}")
+        assert new_location_post_left > original_location_pre_left
+
+        # Get the first tab's initial position (x-axis)
         first_tab = tabs.get_tab(1)
         original_location_pre_right = first_tab.location["x"]
+
+        # Scroll right via double click
         tabs.double_click("tab-scrollbox-right-button", labels=[])
-        new_location_post_right = first_tab.location["x"]
 
-        logging.info(f"The original position: {original_location_pre_right}")
-        logging.info(f"The new position: {new_location_post_right}")
-        assert new_location_post_right > original_location_pre_right
+        # Wait for right scroll to change position (x should decrease)
+        def tab_scrolled_right(d):
+            current_pos = tabs.get_tab(1).location["x"]
+            return current_pos < original_location_pre_right
+
+        tabs.custom_wait(timeout=2).until(tab_scrolled_right)
+
+        # Log and assert the position after right scroll
+        new_location_post_right = tabs.get_tab(1).location["x"]
+        logging.info(f"Right scroll - original x: {original_location_pre_right}")
+        logging.info(f"Right scroll - new x: {new_location_post_right}")
+        assert new_location_post_right < original_location_pre_right