Zfilipin has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/87542


Change subject: Moving shared code to mediawiki-selenium Ruby gem
......................................................................

Moving shared code to mediawiki-selenium Ruby gem

Bug: 53579
Change-Id: If8475599b45d79e5916e05e64ab3be752d547d18
---
M tests/browser/Gemfile
M tests/browser/Gemfile.lock
M tests/browser/config/config.yml
M tests/browser/features/step_definitions/common_steps.rb
M tests/browser/features/support/env.rb
D tests/browser/features/support/sauce.rb
6 files changed, 5 insertions(+), 131 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TwnMainPage 
refs/changes/42/87542/1

diff --git a/tests/browser/Gemfile b/tests/browser/Gemfile
index 8332395..3f9e582 100755
--- a/tests/browser/Gemfile
+++ b/tests/browser/Gemfile
@@ -5,6 +5,7 @@
 
 gem 'cucumber'
 gem 'json'
+gem 'mediawiki-selenium'
 gem 'net-http-persistent'
 gem 'page-object'
 gem 'rspec-expectations'
diff --git a/tests/browser/Gemfile.lock b/tests/browser/Gemfile.lock
index 70d47da..1dd2ab7 100644
--- a/tests/browser/Gemfile.lock
+++ b/tests/browser/Gemfile.lock
@@ -21,6 +21,7 @@
       multi_json (~> 1.3)
     i18n (0.6.5)
     json (1.8.0)
+    mediawiki-selenium (0.1.5)
     multi_json (1.8.0)
     multi_test (0.0.2)
     net-http-persistent (2.9)
@@ -50,6 +51,7 @@
 DEPENDENCIES
   cucumber
   json
+  mediawiki-selenium
   net-http-persistent
   page-object
   rspec-expectations
diff --git a/tests/browser/config/config.yml b/tests/browser/config/config.yml
index ec397c5..6df0a31 100644
--- a/tests/browser/config/config.yml
+++ b/tests/browser/config/config.yml
@@ -1,5 +1,3 @@
-mediawiki_username: Kimono
-
 chrome:
   name: chrome
   platform: Linux
diff --git a/tests/browser/features/step_definitions/common_steps.rb 
b/tests/browser/features/step_definitions/common_steps.rb
index 6632e3a..757462f 100644
--- a/tests/browser/features/step_definitions/common_steps.rb
+++ b/tests/browser/features/step_definitions/common_steps.rb
@@ -2,7 +2,7 @@
 end
 
 Given(/^I am logged in$/) do
-       visit(LoginPage).login_with(@mediawiki_username, 
ENV['MEDIAWIKI_PASSWORD'])
+       visit(LoginPage).login_with(ENV['MEDIAWIKI_USER'], 
ENV['MEDIAWIKI_PASSWORD'])
        # Assert that login worked
        loggedin = [email protected]_script( "return mw.user.isAnon();" )
        loggedin.should be_true
diff --git a/tests/browser/features/support/env.rb 
b/tests/browser/features/support/env.rb
index d56c8c4..2281d94 100644
--- a/tests/browser/features/support/env.rb
+++ b/tests/browser/features/support/env.rb
@@ -1,118 +1 @@
-# before all
-require 'bundler/setup'
-require 'page-object'
-require 'page-object/page_factory'
-require 'watir-webdriver'
-require 'yaml'
-
-World(PageObject::PageFactory)
-
-def browser(environment, test_name, language)
-  if environment == :saucelabs
-    sauce_browser(test_name, language)
-  else
-    local_browser(language)
-  end
-end
-def environment
-  if ENV['BROWSER_LABEL'] and ENV['SAUCE_ONDEMAND_USERNAME'] and 
ENV['SAUCE_ONDEMAND_ACCESS_KEY']
-    :saucelabs
-  else
-    :local
-  end
-end
-def local_browser(language)
-  if ENV['BROWSER_LABEL']
-    browser_label = ENV['BROWSER_LABEL'].to_sym
-  else
-    browser_label = :firefox
-  end
-
-  if language == 'default'
-    Watir::Browser.new browser_label
-  else
-    if browser_label == :firefox
-      profile = Selenium::WebDriver::Firefox::Profile.new
-    elsif browser_label == :chrome
-      profile = Selenium::WebDriver::Chrome::Profile.new
-    else
-      raise "Changing default language is currently supported only for Firefox 
and Chrome!"
-    end
-    profile['intl.accept_languages'] = language
-    Watir::Browser.new browser_label, :profile => profile
-  end
-end
-def sauce_api(json)
-  %x{curl -H 'Content-Type:text/json' -s -X PUT -d '#{json}' 
http://#{ENV['SAUCE_ONDEMAND_USERNAME']}:#{ENV['SAUCE_ONDEMAND_ACCESS_KEY']}@saucelabs.com/rest/v1/#{ENV['SAUCE_ONDEMAND_USERNAME']}/jobs/#{$session_id}}
-end
-def sauce_browser(test_name, language)
-  config = YAML.load_file('config/config.yml')
-  browser_label = config[ENV['BROWSER_LABEL']]
-
-  if language == 'default'
-    caps = 
Selenium::WebDriver::Remote::Capabilities.send(browser_label['name'])
-  elsif browser_label['name'] == 'firefox'
-    profile = Selenium::WebDriver::Firefox::Profile.new
-    profile['intl.accept_languages'] = language
-    caps = Selenium::WebDriver::Remote::Capabilities.firefox(:firefox_profile 
=> profile)
-  elsif browser_label['name'] == 'chrome'
-    profile = Selenium::WebDriver::Chrome::Profile.new
-    profile['intl.accept_languages'] = language
-    caps = Selenium::WebDriver::Remote::Capabilities.chrome('chrome.profile' 
=> profile.as_json['zip'])
-  end
-
-  caps.platform = browser_label['platform']
-  caps.version = browser_label['version']
-  caps[:name] = "#{test_name} #{ENV['JOB_NAME']}##{ENV['BUILD_NUMBER']}"
-
-  require 'selenium/webdriver/remote/http/persistent' # http_client
-  browser = Watir::Browser.new(
-    :remote,
-    http_client: Selenium::WebDriver::Remote::Http::Persistent.new,
-    url: 
"http://#{ENV['SAUCE_ONDEMAND_USERNAME']}:#{ENV['SAUCE_ONDEMAND_ACCESS_KEY']}@ondemand.saucelabs.com:80/wd/hub",
-    desired_capabilities: caps)
-
-  browser.wd.file_detector = lambda do |args|
-    # args => ['/path/to/file']
-    str = args.first.to_s
-    str if File.exist?(str)
-  end
-
-  browser
-end
-
-def test_name(scenario)
-  if scenario.respond_to? :feature
-    "#{scenario.feature.name}: #{scenario.name}"
-  elsif scenario.respond_to? :scenario_outline
-    "#{scenario.scenario_outline.feature.name}: 
#{scenario.scenario_outline.name}: #{scenario.name}"
-  end
-end
-
-config = YAML.load_file('config/config.yml')
-mediawiki_username = config['mediawiki_username']
-
-Before('@login') do
-  puts "MEDIAWIKI_PASSWORD environment variable is not defined! Please export 
a value for that variable before proceeding." unless ENV['MEDIAWIKI_PASSWORD']
-end
-
-Before('@language') do |scenario|
-  @language = true
-  @scenario = scenario
-end
-
-Before do |scenario|
-  @config = config
-  @random_string = Random.new.rand.to_s
-  @mediawiki_username = mediawiki_username
-  @browser = browser(environment, test_name(scenario), 'default')
-  $session_id = @browser.driver.instance_variable_get(:@bridge).session_id
-end
-
-After do |scenario|
-  if environment == :saucelabs
-    sauce_api(%Q{{"passed": #{scenario.passed?}}})
-    sauce_api(%Q{{"public": true}})
-  end
-  @browser.close unless ENV['KEEP_BROWSER_OPEN'] == 'true'
-end
+require 'mediawiki/selenium'
diff --git a/tests/browser/features/support/sauce.rb 
b/tests/browser/features/support/sauce.rb
deleted file mode 100644
index 63eaeaa..0000000
--- a/tests/browser/features/support/sauce.rb
+++ /dev/null
@@ -1,10 +0,0 @@
-require 'cucumber/formatter/junit'
-
-module Cucumber::Formatter
-  class Sauce < Junit
-    def format_exception(exception)
-      sauce_job_page = "Sauce Labs job URL: 
http://saucelabs.com/jobs/#{$session_id}\n";
-      ([sauce_job_page] + ["#{exception.message} (#{exception.class})"] + 
exception.backtrace).join("\n")
-    end
-  end
-end

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If8475599b45d79e5916e05e64ab3be752d547d18
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TwnMainPage
Gerrit-Branch: master
Gerrit-Owner: Zfilipin <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to