On Tue, Mar 25, 2014 at 2:04 PM, Jon Robson <[email protected]> wrote:
> The danger with this is that "Main Page" and other forms will sneak in > again. It is much more natural to write > I am on the "San Francisco" page > This is why > I proposed automatically substituting ' ' for '_' in my change. I haven't had this argument in a while, but I actually think checking for URL correctness is almost never a very good test. Do y'all think we could just flat remove some of these? (It is the one in commons_steps.rb that is causing the immediate problem, but some of those others aren't too smart either.) $ grep -r "The URL of the page" * features/mainmenu.feature: Then The URL of the page should contain "Main_Page" features/mainmenu.feature: Then The URL of the page should contain "campaign=random" features/mainmenu.feature: Then The URL of the page should contain "Special:Nearby" features/mainmenu.feature: Then The URL of the page should contain "returnto=Special%3AWatchlist" features/mainmenu.feature: Then The URL of the page should contain "returnto=Special%3AUploads" features/mainmenu.feature: Then The URL of the page should contain "Special:MobileOptions" features/mainmenu.feature: Then The URL of the page should contain "Special:UserLogin" features/editor_wikitext_nosave.feature: And The URL of the page should contain "Nonexistent_page_ijewrcmhvg34773" features/editor_wikitext_nosave.feature: And The URL of the page should contain "Nonexistent_page_ijewrcmhvg34773" features/search.feature: And The URL of the page should contain "Main%20Page" features/search.feature: And The URL of the page should contain "Main%20Page" features/editor_ve.feature: And The URL of the page should contain "#/VisualEditor/1" features/step_definitions/common_steps.rb:Then(/^The URL of the page should contain "(.+)"$/) do |article| features/uploads_lead.feature: And The URL of the page should contain "Nonexistent_page_abc" features/uploads_lead.feature: And The URL of the page should contain "Nonexistent_page_abc"
_______________________________________________ Mobile-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mobile-l
