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