[QA] Making Visual Editor tests a bit more user-agnostic

[Antoine Musso]

Le 04/12/13 21:33, Jeff Hall a écrit :
> As I've been adding and updating automated tests for Visual Editor, I've
> noticed that even though we usually export a MEDIAWIKI_USER environment
> variable value, there is a hard-coded equivalent value defined in
> visual_editor_page.rb at line 5[1]
> 
> In order to move in the direction of making the tests more flexible,
> does anyone have concerns if I submit a change to modify that line from:
> 
> /page_url URL.url('User:Selenium_user')/
> 
> ...to:
> /
> //page_url URL.url("User:" + ENV['MEDIAWIKI_USER'])/
> 
> Let me know if you have any concerns with such a change, otherwise I'll
> submit a change via Gerrit.

I noticed earlier that the tests assume 'Selenium_user' to exist which
might not always be the case:  https://bugzilla.wikimedia.org/57103

I think your patch would fix it.


-- 
Antoine "hashar" Musso