Skip to content

Commit c53b2fd

Browse files
committed
fix clear history
1 parent 03b3a95 commit c53b2fd

File tree

3 files changed

+14
-18
lines changed

3 files changed

+14
-18
lines changed

modules/browser_object_panel_ui.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -135,12 +135,10 @@ def open_history_menu(self) -> BasePage:
135135
Opens the History menu
136136
"""
137137
self.open_panel_menu()
138-
self.element_visible("panel-main-view")
139138
# Bug 1974080
140139
if self.sys_platform() == "Windows":
141140
sleep(2)
142141
self.click_on("panel-ui-history")
143-
self.element_visible("panel-ui-history-view")
144142
return self
145143

146144
def select_clear_history_option(self, option: str) -> BasePage:
@@ -267,3 +265,10 @@ def clear_recent_history(self, execute=True) -> BasePage:
267265
self.click("clear_recent_history")
268266

269267
return self
268+
269+
@BasePage.context_chrome
270+
def confirm_history_clear(self):
271+
self.open_history_menu()
272+
self.expect_element_attribute_contains(
273+
"recent-history-content", "value", "(Empty)"
274+
)

modules/data/panel_ui.components.json

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
{
22
"context": "chrome",
3+
"do-not-cache": true,
34
"panel-ui-button": {
45
"selectorData": "PanelUI-menu-button",
56
"strategy": "id",
67
"groups": [
7-
"requiredForPage",
8-
"doNotCache"
8+
"requiredForPage"
99
]
1010
},
1111

@@ -42,9 +42,7 @@
4242
"fxa-sync-label": {
4343
"selectorData": "syncnow-label",
4444
"strategy": "class",
45-
"groups": [
46-
"doNotCache"
47-
]
45+
"groups": []
4846
},
4947

5048
"new-private-window-option": {
@@ -118,9 +116,7 @@
118116
"panel-ui-history-view": {
119117
"selectorData": "PanelUI-history",
120118
"strategy": "id",
121-
"groups": [
122-
"doNotCache"
123-
]
119+
"groups": []
124120
},
125121

126122
"panel-ui-history-recently-closed": {

tests/bookmarks_and_history/test_clear_all_history.py

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33

44
from modules.browser_object import PanelUi
55
from modules.page_object import GenericPage
6-
from modules.util import BrowserActions
76

87

98
@pytest.fixture()
@@ -20,16 +19,12 @@ def test_clear_all_history(driver: Firefox):
2019
"""
2120
C172045: Verify that the user can Clear all the History
2221
"""
22+
gen_page = GenericPage(driver)
2323
panel_ui = PanelUi(driver)
2424
panel_ui.open()
25-
gen_page = GenericPage(driver)
2625
panel_ui.open_history_menu()
27-
ba = BrowserActions(driver)
2826

2927
panel_ui.select_clear_history_option("Everything")
3028

31-
gen_page.get_element("clear-history-button").click()
32-
ba.switch_to_content_context()
33-
34-
panel_ui.open_history_menu()
35-
panel_ui.element_does_not_exist("bookmark-item")
29+
gen_page.click_on("clear-history-button")
30+
panel_ui.confirm_history_clear()

0 commit comments

Comments
 (0)