Cmcmahon has uploaded a new change for review.
https://gerrit.wikimedia.org/r/178923
Change subject: QA: no_javascript view no longer has watchlist available
......................................................................
QA: no_javascript view no longer has watchlist available
Update the test according to T78158
Change-Id: I3b7c9793dffffa56a5bb1bd7b55146381db107a9
---
M tests/browser/features/no_javascript_site.feature
M tests/browser/features/step_definitions/common_article_steps.rb
2 files changed, 1 insertion(+), 8 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/23/178923/1
diff --git a/tests/browser/features/no_javascript_site.feature
b/tests/browser/features/no_javascript_site.feature
index 6945cb0..63f7c2e 100644
--- a/tests/browser/features/no_javascript_site.feature
+++ b/tests/browser/features/no_javascript_site.feature
@@ -7,11 +7,9 @@
And I am on the "Main Page" page
# FIXME: Add scenario to check search actually works
- # FIXME: Add scenario to check watch star actually works
Scenario: Able to search in basic non-JavaScript site
When I click on "Random" in the main navigation menu
- Then I should see the watch star
- And I should see the search button
+ Then I should see the search button
# FIXME: Check that the edit button is invisible
# FIXME: Check that the upload button is invisible
@@ -21,7 +19,6 @@
Then I should see a link to "Home" in the main navigation menu
And I should see a link to "Random" in the main navigation menu
And I should see a link to "Settings" in the main navigation menu
- And I should see a link to "Watchlist" in the main navigation menu
And I should see a link to "Log in" in the main navigation menu
And I should not see a link to "Uploads" in the main navigation menu
diff --git a/tests/browser/features/step_definitions/common_article_steps.rb
b/tests/browser/features/step_definitions/common_article_steps.rb
index ed5b2e5..7f3a918 100644
--- a/tests/browser/features/step_definitions/common_article_steps.rb
+++ b/tests/browser/features/step_definitions/common_article_steps.rb
@@ -49,10 +49,6 @@
expect(on(ArticlePage).error_message).to match (error_message)
end
-Then(/^I should see the watch star$/) do
- expect(on(ArticlePage).watch_link_element).to be_visible
-end
-
Then(/^the text of the first heading should be "(.*)"$/) do |title|
on(ArticlePage) do |page|
page.wait_until do
--
To view, visit https://gerrit.wikimedia.org/r/178923
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3b7c9793dffffa56a5bb1bd7b55146381db107a9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Cmcmahon <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
