jenkins-bot has submitted this change and it was merged.

Change subject: Hygiene: Remove undefined test and unused rule
......................................................................


Hygiene: Remove undefined test and unused rule

ran "make cucumber"
Not sure how this test in watchlist.feature didn't get deleted
It's covered in mainmenu.feature - probably a rebase issue.
The not logged in step is not used anywhere

Change-Id: I6cdf41f111a689c6a20d3cdffe575e0ab41423ac
---
M tests/browser/features/step_definitions/common_steps.rb
M tests/browser/features/watchlist.feature
2 files changed, 0 insertions(+), 11 deletions(-)

Approvals:
  Cmcmahon: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/tests/browser/features/step_definitions/common_steps.rb 
b/tests/browser/features/step_definitions/common_steps.rb
index 4e3c029..4818e7e 100644
--- a/tests/browser/features/step_definitions/common_steps.rb
+++ b/tests/browser/features/step_definitions/common_steps.rb
@@ -85,10 +85,6 @@
   end
 end
 
-Given /^I am not logged in$/ do
-  # nothing to do here, user in not logged in by default
-end
-
 Given(/^I am on the "(.+)" page$/) do |article|
   visit(ArticlePage, :using_params => {:article_name => article})
 end
diff --git a/tests/browser/features/watchlist.feature 
b/tests/browser/features/watchlist.feature
index b2f1035..512c6dd 100644
--- a/tests/browser/features/watchlist.feature
+++ b/tests/browser/features/watchlist.feature
@@ -1,13 +1,6 @@
 @en.m.wikipedia.beta.wmflabs.org @en.m.wikipedia.org @test2.m.wikipedia.org
 Feature: Manage Watchlist
 
-  Scenario: Clicking the watchlist item in main menu when anonymous
-    Given I am on the "Main Page" page
-      And I am not logged in
-    When I click on "Watchlist" in the main navigation menu
-    Then I receive watchlist message A watchlist helps you bookmark pages and 
keep track of changes to them.
-      And I receive watchlist message Log in to see it.
-
   @login
   Scenario: Add an article to the watchlist
     Given I am logged into the mobile website

-- 
To view, visit https://gerrit.wikimedia.org/r/118209
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6cdf41f111a689c6a20d3cdffe575e0ab41423ac
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <jrob...@wikimedia.org>
Gerrit-Reviewer: Awjrichards <aricha...@wikimedia.org>
Gerrit-Reviewer: Cmcmahon <cmcma...@wikimedia.org>
Gerrit-Reviewer: JGonera <jgon...@wikimedia.org>
Gerrit-Reviewer: Jdlrobson <jrob...@wikimedia.org>
Gerrit-Reviewer: Kaldari <rkald...@wikimedia.org>
Gerrit-Reviewer: MaxSem <maxsem.w...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to